cpmac: fix wrong MDIO bus identifier
authorFlorian Fainelli <florian@openwrt.org>
Tue, 4 Aug 2009 10:52:41 +0000 (10:52 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 5 Aug 2009 19:20:57 +0000 (12:20 -0700)
This patch fixes the wrong MDIO bus identifier which was
set to 0 unconditionaly, suitable for external switches while
it is actually 1 for PHYs different than external switches
which are autodetected.

Signed-off-by: Florian Fainelli <florian@openwrt.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/cpmac.c

index fd5e32cbcb8778bb7ed02fdd0d7bf6ec6d1e9163..c951dd43f29beb8b6d4648f1ac940dc4c0c9b565 100644 (file)
@@ -1109,7 +1109,7 @@ static int external_switch;
 static int __devinit cpmac_probe(struct platform_device *pdev)
 {
        int rc, phy_id;
-       char *mdio_bus_id = "0";
+       char mdio_bus_id[BUS_ID_SIZE];
        struct resource *mem;
        struct cpmac_priv *priv;
        struct net_device *dev;
@@ -1127,7 +1127,7 @@ static int __devinit cpmac_probe(struct platform_device *pdev)
 
        if (phy_id == PHY_MAX_ADDR) {
                if (external_switch || dumb_switch) {
-                       mdio_bus_id = 0; /* fixed phys bus */
+                       strncpy(mdio_bus_id, "0", BUS_ID_SIZE); /* fixed phys bus */
                        phy_id = pdev->id;
                } else {
                        dev_err(&pdev->dev, "no PHY present\n");
@@ -1254,7 +1254,7 @@ int __devinit cpmac_init(void)
        }
 
        cpmac_mii->phy_mask = ~(mask | 0x80000000);
-       snprintf(cpmac_mii->id, MII_BUS_ID_SIZE, "0");
+       snprintf(cpmac_mii->id, MII_BUS_ID_SIZE, "1");
 
        res = mdiobus_register(cpmac_mii);
        if (res)