[PATCH 06/10] net/farady: Helper functions to create or destroy MDIO interface

Benjamin Herrenschmidt benh at kernel.crashing.org
Fri Jul 1 12:59:49 AEST 2016


On Thu, 2016-06-30 at 23:19 +0930, Joel Stanley wrote:
> +       priv->mii_bus->name = "ftgmac100_mdio";
> > +       snprintf(priv->mii_bus->id, MII_BUS_ID_SIZE,
> "ftgmac100_mii");
> 
> If we have two ftgmac100 drivers loaded (such as on the ast2500-evb),
> when the second goes to create the mdiobus kobject it will have the
> same name. We need to make this name unique (or fix up the mdiobus
> code).
> 
> You can see a backtrace here:
> 
>   https://github.com/openbmc/linux/issues/87

Traditional way:

	snprintf(priv->mii_bus->id, MII_BUS_ID_SIZE, "%s-%d",
		 priv->pdev->name, priv->pdev->id);


IE, the name here isn't the same as mii_bus->name which is a user
visible name, but the actual *device* name (platform dev) and an id.

Cheers,
Ben.



More information about the openbmc mailing list