[PATCH net-next 03/10] net: mdio: mux-bcm-iproc: Separate C22 and C45 transactions
Florian Fainelli
f.fainelli at gmail.com
Fri Jan 13 08:13:51 AEDT 2023
On 1/12/23 07:15, Michael Walle wrote:
> From: Andrew Lunn <andrew at lunn.ch>
>
> The MDIO mux broadcom iproc can perform both C22 and C45 transfers.
> Create separate functions for each and register the C45 versions using
> the new API calls.
>
> Signed-off-by: Andrew Lunn <andrew at lunn.ch>
> Signed-off-by: Michael Walle <michael at walle.cc>
> ---
> Apparently, in the c45 case, the reg value including the MII_ADDR_C45
> bit is written to the hardware. Looks weird, that a "random" software
> bit is written to a register. Florian is that correct? Also, with this
> patch this flag isn't set anymore.
We should be masking the MII_ADDR_C45 bit because the MDIO_ADDR_OFFSET
only defines bits 0 through 20 as being read/write and bits above being
read-only. In practice, this is probably not making any difference or harm.
--
Florian
More information about the Linux-aspeed
mailing list