[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aLrmEzyxzo1DRBNG@quicinc.com>
Date: Fri, 5 Sep 2025 19:00:59 +0530
From: Mohd Ayaan Anwar <quic_mohdayaa@...cinc.com>
To: "Russell King (Oracle)" <linux@...linux.org.uk>
CC: Andrew Lunn <andrew@...n.ch>, Heiner Kallweit <hkallweit1@...il.com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>,
Andrew Lunn
<andrew+netdev@...n.ch>,
"David S. Miller" <davem@...emloft.net>,
Eric
Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-stm32@...md-mailman.stormreply.com>,
Maxime Coquelin
<mcoquelin.stm32@...il.com>, <netdev@...r.kernel.org>,
Paolo Abeni
<pabeni@...hat.com>
Subject: Re: [PATCH net-next v2 00/11] net: stmmac: mdio cleanups
On Thu, Sep 04, 2025 at 01:10:42PM +0100, Russell King (Oracle) wrote:
> On Wed, Sep 03, 2025 at 01:38:57PM +0100, Russell King (Oracle) wrote:
> Hi,
>
> Clean up the stmmac MDIO code:
> - provide an address register formatter to avoid repeated code
> - provide a common function to wait for the busy bit to clear
> - pre-compute the CR field (mdio clock divider)
> - move address formatter into read/write functions
> - combine the read/write functions into a common accessor function
> - move runtime PM handling into common accessor function
> - rename register constants to better reflect manufacturer names
> - move stmmac_clk_csr_set() into stmmac_mdio
> - make stmmac_clk_csr_set() return the CR field value and remove
> priv->clk_csr
> - clean up if() range tests in stmmac_clk_csr_set()
> - use STMMAC_CSR_xxx definitions in initialisers
>
> Untested on hardware; would be grateful for any testing people can do.
>
Picked this series on top of net-next and was able to test on the
Qualcomm QCS9100 Ride R3 board with the AQR115C Phy. No issues seen with
C45 MDIO operations, so:
Tested-by: Mohd Ayaan Anwar <quic_mohdayaa@...cinc.com>
Ayaan
> v2: add "Return:" to patch 1 and 9
>
> drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c | 5 +-
> .../net/ethernet/stmicro/stmmac/dwmac-loongson.c | 3 +-
> drivers/net/ethernet/stmicro/stmmac/stmmac.h | 2 +-
> drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 82 -----
> drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c | 345 ++++++++++++---------
> drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c | 5 +-
> 6 files changed, 207 insertions(+), 235 deletions(-)
>
> --
> RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
> FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
>
Powered by blists - more mailing lists