riscv defconfig CONFIG_PM/macb/generic PHY regression in v5.18-rc1

[Conor.Dooley at microchip.com]

On 05/04/2022 15:04, Andrew Lunn wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
>> [ 2.818894] macb 20112000.ethernet eth0: PHY [20112000.ethernet-ffffffff:09] driver [Generic PHY] (irq=POLL)
>> [ 2.828915] macb 20112000.ethernet eth0: configuring for phy/sgmii link mode
>> [11.045411] macb 20112000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>> [11.053247] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> 
> You have a multi-part link. You need that the PHY reports the line
> side is up. Put some printk in genphy_update_link() and look at
> phydev->link. You also need that the SGMII link between the PHY and
> the SoC is up. That is a bit harder to see, but try adding #define
> DEBUG at the top of phylink.c and phy.c so you get additional debug
> prints for the state machines.

Tracked the state of phydev->link in genphy_update_link, never saw a
value other than 0.

Using the debug prints in phylink.c I got the following:
[    3.230364] macb 20112000.ethernet eth0: PHY [20112000.ethernet-ffffffff:09] driver [Vitesse VSC8662] (irq=POLL)
[    3.240682] macb 20112000.ethernet eth0: phy: sgmii setting supported 0000000,00000000,000042ff advertising 0000000,00000000,000042ff
[    3.252783] macb 20112000.ethernet eth0: configuring for phy/sgmii link mode
[    3.259892] macb 20112000.ethernet eth0: major config sgmii
[    3.265526] macb 20112000.ethernet eth0: phylink_mac_config: mode=phy/sgmii/Unknown/Unknown adv=0000000,00000000,00000000 pause=00 link=0 an=0
[    3.279249] macb 20112000.ethernet eth0: phy link down sgmii/Unknown/Unknown/off

I couldn't see any prints out of phy.c

Thanks,
Conor.