Re: [RFC PATCH] drivers: net: mdio_bus: try indirect clause 45 regs access

2020-05-17 Thread Baruch Siach
Hi Andrew, On Sun, May 17 2020, Andrew Lunn wrote: >> > I don't think this should be done at mdiobus level; I think this is a >> > layering violation. It needs to happen at the PHY level because the >> > indirect C45 access via C22 registers is specific to PHYs. >> > >> > It also needs to check i

Re: [RFC PATCH] drivers: net: mdio_bus: try indirect clause 45 regs access

2020-05-17 Thread Andrew Lunn
> > I don't think this should be done at mdiobus level; I think this is a > > layering violation. It needs to happen at the PHY level because the > > indirect C45 access via C22 registers is specific to PHYs. > > > > It also needs to check in the general case that the PHY does indeed > > support t

Re: [RFC PATCH] drivers: net: mdio_bus: try indirect clause 45 regs access

2020-05-17 Thread Baruch Siach
Hi Russell, On Sun, May 17 2020, Russell King - ARM Linux admin wrote: > On Sun, May 17, 2020 at 01:20:56PM +0300, Baruch Siach wrote: >> When the MDIO bus does not support directly clause 45 access, fallback >> to indirect registers access method to read/write clause 45 registers >> using clause

Re: [RFC PATCH] drivers: net: mdio_bus: try indirect clause 45 regs access

2020-05-17 Thread Russell King - ARM Linux admin
On Sun, May 17, 2020 at 01:20:56PM +0300, Baruch Siach wrote: > When the MDIO bus does not support directly clause 45 access, fallback > to indirect registers access method to read/write clause 45 registers > using clause 22 registers. > > Signed-off-by: Baruch Siach > --- > > Is that the right

[RFC PATCH] drivers: net: mdio_bus: try indirect clause 45 regs access

2020-05-17 Thread Baruch Siach
When the MDIO bus does not support directly clause 45 access, fallback to indirect registers access method to read/write clause 45 registers using clause 22 registers. Signed-off-by: Baruch Siach --- Is that the right course? Currently, this does not really work on the my target machine, which