On 21/9/2023 10:55 pm, Russell King (Oracle) wrote:
On Thu, Sep 21, 2023 at 04:41:20PM +0200, Andrew Lunn wrote:
On Thu, Sep 21, 2023 at 03:12:19PM +0100, Russell King (Oracle) wrote:
On Thu, Sep 21, 2023 at 03:21:00PM +0200, Andrew Lunn wrote:
Hi Andrew,
After conducting a comprehensive study, it seems that implementing
out-of-band for all link modes might not be feasible. I may have missed some
key aspects during my analysis.
Would you be open to sharing a high-level idea of how we could potentially
make this feasible? Your insights would be greatly appreciated.
stmmac_mac_link_up() gets passed interface, speed and duplex. That
tells you what the PHY has negotiated. Is there anything else you need
to know?
The problem is... the stmmac driver is utter bollocks - that information
is *not* passed to the BSP. Instead, stmmac parse and store information
such as the PHY interface mode at initialisation time. BSPs also re-
parse and store e.g. the PHY interface mode at initialisation time.
The driver ignores what it gets from phylink.
The driver is basically utter crap. That's an area I _had_ patches to
clean up. I no longer do. stmmac is crap crap crap and will stay crap
until they become more receptive to patches to fix it, even if the
patches are not 100% to their liking but are in fact correct. Maybe
if I ever decide to touch that driver in the future. Which I doubt
given my recent experience.
Hi Russell
You pointed out the current proposal will break stuff. Do you see a
way forward for this patchset which does not first involve actually
cleaning up of this driver?
As I said in one of my replies, it would really help if the author can
provide a table showing what is attempting to be achieved here. With
that, we should be able to work out exactly what is required, what
needs to change in stmmac, etc.
Thank you, Russell and Andrew for the comments.
What I'm trying to achieve here is to enable interface mode switching
between 2500basex and SGMII interfaces for Intel platforms.
I did based on the DM7052 SFP module and BCM84881 PHY to make the necessary
changes in the new version of my patch series.
In the new patch series, the 'allow_switch_interface' flag was introduced,
based on the 'allow_switch_interface' flag, the interface mode is
configured to PHY_INTERFACE_MODE_NA within the 'phylink_validate_phy'
function. This setting allows all ethtool link modes that are supported and
advertised will be published. Then interface mode switching occurs based on
the selection of different link modes.
During the interface mode switching, the code will go through the
`phylink_major_config` function and based on the interface mode to select
PCS and PCS negotiation mode to configure PCS. Then, the MAC driver will
perform SerDes configuration according to the interface mode.
I did rewrite the description for the new patch series, hoping that it is
clear to describe the whole intention of the changes.
