summaryrefslogtreecommitdiffstats
path: root/net/dsa/slave.c
diff options
context:
space:
mode:
authorAndrew Lunn <andrew@lunn.ch>2014-10-19 16:41:47 +0200
committerDavid S. Miller <davem@davemloft.net>2014-10-19 12:46:31 -0400
commit228b16cb13dfa2f77354a9b144a07e86e2dda01c (patch)
tree0ade083774b11e5e3874a8b1c57d48d93792993a /net/dsa/slave.c
parenta6d4518da371b0d21512ef0f641928d6f99ee70f (diff)
downloadop-kernel-dev-228b16cb13dfa2f77354a9b144a07e86e2dda01c.zip
op-kernel-dev-228b16cb13dfa2f77354a9b144a07e86e2dda01c.tar.gz
Net: DSA: Fix checking for get_phy_flags function
The check for the presence or not of the optional switch function get_phy_flags() called the function, rather than checked to see if it is a NULL pointer. This causes a derefernce of a NULL pointer on all switch chips except the sf2, the only switch to implement this call. Signed-off-by: Andrew Lunn <andrew@lunn.ch> Fixes: 6819563e646a ("net: dsa: allow switch drivers to specify phy_device::dev_flags") Cc: Florian Fainelli <f.fainelli@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa/slave.c')
-rw-r--r--net/dsa/slave.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index a851e9f..6d18174 100644
--- a/net/dsa/slave.c
+++ b/net/dsa/slave.c
@@ -599,7 +599,7 @@ dsa_slave_create(struct dsa_switch *ds, struct device *parent,
netif_carrier_off(slave_dev);
if (p->phy != NULL) {
- if (ds->drv->get_phy_flags(ds, port))
+ if (ds->drv->get_phy_flags)
p->phy->dev_flags |= ds->drv->get_phy_flags(ds, port);
phy_attach(slave_dev, dev_name(&p->phy->dev),
OpenPOWER on IntegriCloud