Skip to content

Commit

Permalink
net: dsa: Make most functions take a dsa_port argument
Browse files Browse the repository at this point in the history 
In preparation for allowing platform data, and therefore no valid
device_node pointer, make most DSA functions takes a pointer to a
dsa_port structure whenever possible. While at it, introduce a
dsa_port_is_valid() helper function which checks whether port->dn is
NULL or not at the moment.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Florian Fainelli authored and David S. Miller committed Jan 26, 2017
1 parent 55ed0ce commit 293784a
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 36 deletions.
15 changes: 9 additions & 6 deletions net/dsa/dsa.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -110,8 +110,9 @@ dsa_switch_probe(struct device *parent, struct device *host_dev, int sw_addr,

/* basic switch operations **************************************************/
int dsa_cpu_dsa_setup(struct dsa_switch *ds, struct device *dev,
struct device_node *port_dn, int port)
struct dsa_port *dport, int port)
{
struct device_node *port_dn = dport->dn;
struct phy_device *phydev;
int ret, mode;

Expand Down Expand Up @@ -141,15 +142,15 @@ int dsa_cpu_dsa_setup(struct dsa_switch *ds, struct device *dev,

static int dsa_cpu_dsa_setups(struct dsa_switch *ds, struct device *dev)
{
struct device_node *port_dn;
struct dsa_port *dport;
int ret, port;

for (port = 0; port < DSA_MAX_PORTS; port++) {
if (!(dsa_is_cpu_port(ds, port) || dsa_is_dsa_port(ds, port)))
continue;

port_dn = ds->ports[port].dn;
ret = dsa_cpu_dsa_setup(ds, dev, port_dn, port);
dport = &ds->ports[port];
ret = dsa_cpu_dsa_setup(ds, dev, dport, port);
if (ret)
return ret;
}
Expand Down Expand Up @@ -364,8 +365,10 @@ dsa_switch_setup(struct dsa_switch_tree *dst, int index,
return ds;
}

void dsa_cpu_dsa_destroy(struct device_node *port_dn)
void dsa_cpu_dsa_destroy(struct dsa_port *port)
{
struct device_node *port_dn = port->dn;

if (of_phy_is_fixed_link(port_dn))
of_phy_deregister_fixed_link(port_dn);
}
Expand All @@ -389,7 +392,7 @@ static void dsa_switch_destroy(struct dsa_switch *ds)
for (port = 0; port < DSA_MAX_PORTS; port++) {
if (!(dsa_is_cpu_port(ds, port) || dsa_is_dsa_port(ds, port)))
continue;
dsa_cpu_dsa_destroy(ds->ports[port].dn);
dsa_cpu_dsa_destroy(&ds->ports[port]);

/* Clearing a bit which is not set does no harm */
ds->cpu_port_mask |= ~(1 << port);
Expand Down
61 changes: 33 additions & 28 deletions net/dsa/dsa2.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -78,14 +78,19 @@ static void dsa_dst_del_ds(struct dsa_switch_tree *dst,
kref_put(&dst->refcount, dsa_free_dst);
}

static bool dsa_port_is_dsa(struct device_node *port)
static bool dsa_port_is_valid(struct dsa_port *port)
{
return !!of_parse_phandle(port, "link", 0);
return !!port->dn;
}

static bool dsa_port_is_cpu(struct device_node *port)
static bool dsa_port_is_dsa(struct dsa_port *port)
{
return !!of_parse_phandle(port, "ethernet", 0);
return !!of_parse_phandle(port->dn, "link", 0);
}

static bool dsa_port_is_cpu(struct dsa_port *port)
{
return !!of_parse_phandle(port->dn, "ethernet", 0);
}

static bool dsa_ds_find_port(struct dsa_switch *ds,
Expand Down Expand Up @@ -119,15 +124,15 @@ static struct dsa_switch *dsa_dst_find_port(struct dsa_switch_tree *dst,

static int dsa_port_complete(struct dsa_switch_tree *dst,
struct dsa_switch *src_ds,
struct device_node *port,
struct dsa_port *port,
u32 src_port)
{
struct device_node *link;
int index;
struct dsa_switch *dst_ds;

for (index = 0;; index++) {
link = of_parse_phandle(port, "link", index);
link = of_parse_phandle(port->dn, "link", index);
if (!link)
break;

Expand All @@ -150,13 +155,13 @@ static int dsa_port_complete(struct dsa_switch_tree *dst,
*/
static int dsa_ds_complete(struct dsa_switch_tree *dst, struct dsa_switch *ds)
{
struct device_node *port;
struct dsa_port *port;
u32 index;
int err;

for (index = 0; index < DSA_MAX_PORTS; index++) {
port = ds->ports[index].dn;
if (!port)
port = &ds->ports[index];
if (!dsa_port_is_valid(port))
continue;

if (!dsa_port_is_dsa(port))
Expand Down Expand Up @@ -196,7 +201,7 @@ static int dsa_dst_complete(struct dsa_switch_tree *dst)
return 0;
}

static int dsa_dsa_port_apply(struct device_node *port, u32 index,
static int dsa_dsa_port_apply(struct dsa_port *port, u32 index,
struct dsa_switch *ds)
{
int err;
Expand All @@ -211,13 +216,13 @@ static int dsa_dsa_port_apply(struct device_node *port, u32 index,
return 0;
}

static void dsa_dsa_port_unapply(struct device_node *port, u32 index,
static void dsa_dsa_port_unapply(struct dsa_port *port, u32 index,
struct dsa_switch *ds)
{
dsa_cpu_dsa_destroy(port);
}

static int dsa_cpu_port_apply(struct device_node *port, u32 index,
static int dsa_cpu_port_apply(struct dsa_port *port, u32 index,
struct dsa_switch *ds)
{
int err;
Expand All @@ -234,21 +239,21 @@ static int dsa_cpu_port_apply(struct device_node *port, u32 index,
return 0;
}

static void dsa_cpu_port_unapply(struct device_node *port, u32 index,
static void dsa_cpu_port_unapply(struct dsa_port *port, u32 index,
struct dsa_switch *ds)
{
dsa_cpu_dsa_destroy(port);
ds->cpu_port_mask &= ~BIT(index);

}

static int dsa_user_port_apply(struct device_node *port, u32 index,
static int dsa_user_port_apply(struct dsa_port *port, u32 index,
struct dsa_switch *ds)
{
const char *name;
int err;

name = of_get_property(port, "label", NULL);
name = of_get_property(port->dn, "label", NULL);
if (!name)
name = "eth%d";

Expand All @@ -262,7 +267,7 @@ static int dsa_user_port_apply(struct device_node *port, u32 index,
return 0;
}

static void dsa_user_port_unapply(struct device_node *port, u32 index,
static void dsa_user_port_unapply(struct dsa_port *port, u32 index,
struct dsa_switch *ds)
{
if (ds->ports[index].netdev) {
Expand All @@ -274,7 +279,7 @@ static void dsa_user_port_unapply(struct device_node *port, u32 index,

static int dsa_ds_apply(struct dsa_switch_tree *dst, struct dsa_switch *ds)
{
struct device_node *port;
struct dsa_port *port;
u32 index;
int err;

Expand Down Expand Up @@ -308,8 +313,8 @@ static int dsa_ds_apply(struct dsa_switch_tree *dst, struct dsa_switch *ds)
}

for (index = 0; index < DSA_MAX_PORTS; index++) {
port = ds->ports[index].dn;
if (!port)
port = &ds->ports[index];
if (!dsa_port_is_valid(port))
continue;

if (dsa_port_is_dsa(port)) {
Expand All @@ -336,12 +341,12 @@ static int dsa_ds_apply(struct dsa_switch_tree *dst, struct dsa_switch *ds)

static void dsa_ds_unapply(struct dsa_switch_tree *dst, struct dsa_switch *ds)
{
struct device_node *port;
struct dsa_port *port;
u32 index;

for (index = 0; index < DSA_MAX_PORTS; index++) {
port = ds->ports[index].dn;
if (!port)
port = &ds->ports[index];
if (!dsa_port_is_valid(port))
continue;

if (dsa_port_is_dsa(port)) {
Expand Down Expand Up @@ -425,15 +430,15 @@ static void dsa_dst_unapply(struct dsa_switch_tree *dst)
dst->applied = false;
}

static int dsa_cpu_parse(struct device_node *port, u32 index,
static int dsa_cpu_parse(struct dsa_port *port, u32 index,
struct dsa_switch_tree *dst,
struct dsa_switch *ds)
{
enum dsa_tag_protocol tag_protocol;
struct net_device *ethernet_dev;
struct device_node *ethernet;

ethernet = of_parse_phandle(port, "ethernet", 0);
ethernet = of_parse_phandle(port->dn, "ethernet", 0);
if (!ethernet)
return -EINVAL;

Expand Down Expand Up @@ -466,13 +471,13 @@ static int dsa_cpu_parse(struct device_node *port, u32 index,

static int dsa_ds_parse(struct dsa_switch_tree *dst, struct dsa_switch *ds)
{
struct device_node *port;
struct dsa_port *port;
u32 index;
int err;

for (index = 0; index < DSA_MAX_PORTS; index++) {
port = ds->ports[index].dn;
if (!port)
port = &ds->ports[index];
if (!dsa_port_is_valid(port))
continue;

if (dsa_port_is_cpu(port)) {
Expand Down Expand Up @@ -533,7 +538,7 @@ static int dsa_parse_ports_dn(struct device_node *ports, struct dsa_switch *ds)
* to have access to a correct value, just like what
* net/dsa/dsa.c::dsa_switch_setup_one does.
*/
if (!dsa_port_is_cpu(port))
if (!dsa_port_is_cpu(&ds->ports[reg]))
ds->enabled_port_mask |= 1 << reg;
}

Expand Down
4 changes: 2 additions & 2 deletions net/dsa/dsa_priv.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,8 @@ struct dsa_slave_priv {

/* dsa.c */
int dsa_cpu_dsa_setup(struct dsa_switch *ds, struct device *dev,
struct device_node *port_dn, int port);
void dsa_cpu_dsa_destroy(struct device_node *port_dn);
struct dsa_port *dport, int port);
void dsa_cpu_dsa_destroy(struct dsa_port *dport);
const struct dsa_device_ops *dsa_resolve_tag_protocol(int tag_protocol);
int dsa_cpu_port_ethtool_setup(struct dsa_switch *ds);
void dsa_cpu_port_ethtool_restore(struct dsa_switch *ds);
Expand Down

0 comments on commit 293784a

Please sign in to comment.