Re: [PATCH net-next] of: of_mdio: Check if MDIO bus controller is available

2016-05-01 Thread David Miller
From: Florian Fainelli Date: Thu, 28 Apr 2016 14:55:10 -0700 > Add a check whether the 'struct device_node' pointer passed to > of_mdiobus_register() is an available (aka enabled) node in the Device > Tree. > > Rationale for doing this are cases where an Ethernet MAC provides a MDIO > bus contro

Re: [PATCH net-next] of: of_mdio: Check if MDIO bus controller is available

2016-04-28 Thread Andrew Lunn
> Fair enough, I will submit something after re-spining this patch to use > -ENODEV, which I agree is a better return code. Did you want me to > remove that blurb from the commit message? Blurb looks good. More blurb is better than less... Andrew

Re: [PATCH net-next] of: of_mdio: Check if MDIO bus controller is available

2016-04-28 Thread Florian Fainelli
On 28/04/16 15:12, Andrew Lunn wrote: > On Thu, Apr 28, 2016 at 02:55:10PM -0700, Florian Fainelli wrote: >> Add a check whether the 'struct device_node' pointer passed to >> of_mdiobus_register() is an available (aka enabled) node in the Device >> Tree. >> >> Rationale for doing this are cases whe

Re: [PATCH net-next] of: of_mdio: Check if MDIO bus controller is available

2016-04-28 Thread Andrew Lunn
On Thu, Apr 28, 2016 at 02:55:10PM -0700, Florian Fainelli wrote: > Add a check whether the 'struct device_node' pointer passed to > of_mdiobus_register() is an available (aka enabled) node in the Device > Tree. > > Rationale for doing this are cases where an Ethernet MAC provides a MDIO > bus con

Re: [PATCH net-next] of: of_mdio: Check if MDIO bus controller is available

2016-04-28 Thread Andrew Lunn
On Thu, Apr 28, 2016 at 02:55:10PM -0700, Florian Fainelli wrote: > Add a check whether the 'struct device_node' pointer passed to > of_mdiobus_register() is an available (aka enabled) node in the Device > Tree. > > Rationale for doing this are cases where an Ethernet MAC provides a MDIO > bus con

[PATCH net-next] of: of_mdio: Check if MDIO bus controller is available

2016-04-28 Thread Florian Fainelli
Add a check whether the 'struct device_node' pointer passed to of_mdiobus_register() is an available (aka enabled) node in the Device Tree. Rationale for doing this are cases where an Ethernet MAC provides a MDIO bus controller and node, and an additional Ethernet MAC might be connecting its PHY/s