Re: mv88e6240 configuration broken for B850v3

Russell King (Oracle)
2021-12-07T14:53:48Z

On Tue, Dec 07, 2021 at 04:37:05PM +0200, Vladimir Oltean wrote:
> Although it isn't strictly true that "[ if ] there is a non-software
> method to handle the status updates, then we don't do any forcing in
> the mac_link_*() methods", because technically we still do in
> mac_link_down, due to the " || mode == MLO_AN_FIXED" condition plus the
> way in which we are called, it's just that we undo it later.

Fixed links are a "software method" as Andrew has already pointed
out - there is a software hook that can be used to retrieve the
link status, potentially also updating the other link parameters
as well.

Fixed links have never been "these are the only parameters you will
ever get and the link will always be forced up". That is not new.
Have a look at drivers/net/phy/fixed_phy.c and its link_update()
method, which can change any parameter of the fixed-link it desires.
There is also fixed_phy_change_carrier() which allows one to force
the link state of a fixed-link.

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