Subject: Re: [PATCH v2 net-next 3/3] net: phy: nxp-c45-tja11xx: enable MDIO write access to the master/slave registers

From: "Russell King (Oracle)" <[email-masked]>
Date: Mon, 14 Jun 2021 13:51:09 +0100

On Mon, Jun 14, 2021 at 03:38:15PM +0300, Vladimir Oltean wrote:
> From: Vladimir Oltean <[email-masked]>
> 
> The SJA1110 switch integrates TJA1103 PHYs, but in SJA1110 switch rev B
> silicon, there is a bug in that the registers for selecting the 100base-T1
> autoneg master/slave roles are not writable.
> 
> To enable write access to the master/slave registers, these additional
> PHY writes are necessary during initialization.
> 
> The issue has been corrected in later SJA1110 silicon versions and is
> not present in the standalone PHY variants, but applying the workaround
> unconditionally in the driver should not do any harm.
> 
> Suggested-by: Radu Pirea (NXP OSS) <[email-masked]>
> Signed-off-by: Vladimir Oltean <[email-masked]>

Reviewed-by: Russell King (Oracle) <[email-masked]>

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!