Re: [U-Boot] net, phy: wrong 1000BASE detection with a lan9303 switch

Dear Heiko,

In message 51DD3D92.8050701@denx.de you wrote:

Would this be an acceptable patch for it:

...

@@ -300,6 +300,10 @@ int genphy_parse_link(struct phy_device *phydev) * both PHYs in the link */ gblpa = phy_read(phydev, MDIO_DEVAD_NONE, MII_STAT1000);

•                   if (gblpa < 0) {
  

•                           printf ("Could not read MII_STAT1000. Ignoring gigabit capability\n");
  

•                           gblpa = 0;
  

•                   }
                     gblpa &= phy_read(phydev,
                                     MDIO_DEVAD_NONE, MII_CTRL1000) << 2;
  

Well, this other phy_read() probably needs exactly the same error handling - and I doubt if we should actually try riding the reg when the first one failed already?

Best regards,

Wolfgang Denk