From: Gabor Juhos Date: Tue, 17 Dec 2013 15:12:52 +0000 (+0000) Subject: ar71xx: ag71xx: use mdio bus name in ar7240_probe messages X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=ea89cc9250dc5f303925d2d91cbad8bb69b9d924;p=openwrt%2Fstaging%2Fnbd.git ar71xx: ag71xx: use mdio bus name in ar7240_probe messages The ar7240_probe function uses the network device name in the kernel log messages, however the name is not yet initialized when the ar7240_probe function is called. Use the mdio bus name in the messages to avoid ugly log lines like the following one: eth%d: Found an AR7240/AR9330 built-in switch Reported-by: Ronald Wahl Signed-off-by: Gabor Juhos SVN-Revision: 39116 --- diff --git a/target/linux/ar71xx/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_ar7240.c b/target/linux/ar71xx/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_ar7240.c index 9a51f786f9..3fa6cca36a 100644 --- a/target/linux/ar71xx/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_ar7240.c +++ b/target/linux/ar71xx/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_ar7240.c @@ -1043,7 +1043,7 @@ static struct ar7240sw *ar7240_probe(struct ag71xx *ag) if ((phy_id1 != AR7240_PHY_ID1 || phy_id2 != AR7240_PHY_ID2) && (phy_id1 != AR934X_PHY_ID1 || phy_id2 != AR934X_PHY_ID2)) { pr_err("%s: unknown phy id '%04x:%04x'\n", - ag->dev->name, phy_id1, phy_id2); + dev_name(&mii->dev), phy_id1, phy_id2); return NULL; } @@ -1074,7 +1074,7 @@ static struct ar7240sw *ar7240_probe(struct ag71xx *ag) AR934X_OPER_MODE0_PHY_MII_EN); } else { pr_err("%s: invalid PHY interface mode\n", - ag->dev->name); + dev_name(&mii->dev)); goto err_free; } @@ -1087,7 +1087,7 @@ static struct ar7240sw *ar7240_probe(struct ag71xx *ag) } } else { pr_err("%s: unsupported chip, ctrl=%08x\n", - ag->dev->name, ctrl); + dev_name(&mii->dev), ctrl); goto err_free; } @@ -1098,7 +1098,7 @@ static struct ar7240sw *ar7240_probe(struct ag71xx *ag) if (register_switch(&as->swdev, ag->dev) < 0) goto err_free; - pr_info("%s: Found an %s\n", ag->dev->name, swdev->name); + pr_info("%s: Found an %s\n", dev_name(&mii->dev), swdev->name); /* initialize defaults */ for (i = 0; i < AR7240_MAX_VLANS; i++)