Re: [PATCH v4 net-next 3/8] net: phy: bcm84881: move the in-band capability check where it belongs
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
Date: 2022-11-23 13:31:20
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
Date: 2022-11-23 13:31:20
On Wed, Nov 23, 2022 at 12:08:11PM +0000, Russell King (Oracle) wrote:
On Tue, Nov 22, 2022 at 09:36:59PM +0200, Vladimir Oltean wrote:quoted
I think we're in agreement, but please let's wait until tomorrow, I need to take a break for today.I think we do have a sort of agreement... but lets give this a go. The following should be sufficient for copper SFP modules using the 88E1111 PHY. However, I haven't build-tested this patch yet. Reading through the documentation has brought up some worms in this area. :( It may be worth printing the fiber page BMCR and extsr at various strategic points in this driver and reporting back if things don't seem to be working right for your modules. In the mean time, I'll try to see how the modules in the Honeycomb appear to be setup at power-up and after the driver has configured the PHY... assuming I left both MicroUSBs connected and the board has a network connection via the main ethernet jack.
Unfortunately, I don't have a SFP with an 88e1111 plugged in, only the bcm84881, so I can't test my patch remotely. However, it builds fine when the appropriate TIMEOUT definition is added. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!