Re: [PATCH 05/12] net: phy: add phy_id_broken support

From: Florian Fainelli
Date: Wed Apr 05 2023 - 08:30:28 EST




On 4/5/2023 5:27 AM, Andrew Lunn wrote:
On Wed, Apr 05, 2023 at 11:26:56AM +0200, Marco Felsch wrote:
Some phy's don't report the correct phy-id, e.g. the TJA1102 dual-port
report 0 for the 2nd port. To fix this a driver needs to supply the
phyid instead and tell the phy framework to not try to readout the
phyid. The latter case is done via the new 'phy_id_broken' flag which
tells the phy framework to skip phyid readout for the corresponding phy.

In general, we try to avoid work around for broken hardware in the
core. Please try to solve this within nxp-tja11xx.c.

Agreed, and one way to solve working around broken PHY identification registers is to provide them through the compatible string via "ethernet-phyAAAA.BBBB". This forces the PHY library not to read from those registers yet instantiate the PHY device and force it to bind to a certain phy_driver.
--
Florian