[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