From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [78.32.30.218]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9407C296BA7; Fri, 23 Jan 2026 17:27:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=78.32.30.218 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769189225; cv=none; b=fgaXQlkvohzcGUgdOEUP16EwFFJCuOIOZ0ii6yaxNSfXlU58kty9tGAM2DveBKaQgNk6+GyNkuMYjSGar3XYkgxnYXzTxNewAq5W4i8cN35ikyip+j+FAZSos9xYe+1sOxldPN7rB9BixEL7yheRMtujF7ZXYpIMVrC3SFkNsck= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769189225; c=relaxed/simple; bh=G/8OCQ/8COWTW4EpiiDpBWGvKl0QjVJqCfmQ1t0wbGk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=oFww1EDg0ydWh5VxptX/8opl+3PpUl3qMul8mYN+5YdO9Oveq8t11NsLaQ+B8r570rxXvdhE36eS+BxUcuT6b6ODMN3rGDF4pS84xpRN6eWGKSqMutO8sIEFAIW5CsGiYbuPlpcretd57/VhXBco4GSyB9pRiCsp/mKmogNYqK4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=armlinux.org.uk; spf=none smtp.mailfrom=armlinux.org.uk; dkim=pass (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b=WbE7SAtu; arc=none smtp.client-ip=78.32.30.218 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=armlinux.org.uk Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="WbE7SAtu" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=ATqUy5/Sp0hLV7kTQCotJYn2id+kgnBDhi2Bfu1dxMw=; b=WbE7SAtucfXWKA5HnfaYgijWAA 7L0Et33+VW7YE0lTPtIg91YcJfGT/yqR4XWo1hOw3m7+aVG65JgF7eEHkVJ2uucFrEz9d3yS0RouW pT+Tki0zV5+Z5Jy3b1t3Syzoa/LXYZcZEwiBOglSDeN7qf+1Dhgk0Mv06lfV2fsphs/ho1GU7SiPk 7Lup2A1+JyjmwnTix8YwXKrcGF4aBOn7KZ/kjR695XoY5LJlDU4d/Hqm0YJmltDvCL7s3/OAaJ++m ZpZK6f8PfdvQq5KpcTqsfmSNWwQZtHE4V5fTLx3oUaU2CZ+leNEvhXGF4mPe+8IT6GbIMaS6sLY5/ Fp2PCgHA==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:42282) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1vjKvr-000000002Mn-2kBY; Fri, 23 Jan 2026 17:26:51 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.98.2) (envelope-from ) id 1vjKvn-000000002GB-2S0Y; Fri, 23 Jan 2026 17:26:47 +0000 Date: Fri, 23 Jan 2026 17:26:47 +0000 From: "Russell King (Oracle)" To: Mohd Ayaan Anwar Cc: Andrew Lunn , Heiner Kallweit , Alexandre Torgue , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Konrad Dybcio , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, Maxime Coquelin , Neil Armstrong , netdev@vger.kernel.org, Paolo Abeni , Vinod Koul Subject: Re: [PATCH net-next v2 00/14] net: stmmac: SerDes, PCS, BASE-X, and inband goodies Message-ID: References: Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: Russell King (Oracle) On Fri, Jan 23, 2026 at 07:05:00PM +0530, Mohd Ayaan Anwar wrote: > Hello Russell, > On Fri, Jan 23, 2026 at 09:52:00AM +0000, Russell King (Oracle) wrote: > > This is the v1 submission: if it doesn't get tested but review goes > > well, it'll end up in net-next and mainline without testing on the > > affected hardware! > > > > Mentioned previously, I've been trying to sort out the PCS support in > > stmmac, and this series represents the current state of play. > > > > Previous posted patches centred around merely getting autonegotiation > > to be configured correctly, to a point where the manual configuration > > can be removed from the qcom-ethqos driver. The qcom-ethqos driver > > uses both SGMII and 2500BASE-X, manually configuring the dwmac's > > integrated PCS appropriately. > > > > Thank you for CC'ing me on this series. Sorry, but I have been M.I.A. > for the past couple of months due to some health issues, which caused a > backlog at work that I had to power through. I haven't been able to > monitor the mailing list for stmmac patches. Sorry to hear that, but if it's any consolation, you're not alone. On new year's eve, I had three teeth extracted, including one that was laying horizontally in the palate of the mouth buried in bone, and needed in bone graft (modern bone grafts are quite different from what you'd expect btw.) It's been quite sore/painful as it heals. > I tested v1 last night and just picked up v2. Here are my observations > and logs (phylink logs are enabled). I haven't had time to debug the > issues, but they are not seen on the net-next tree. One thing that I > remember from our last discussion is the need to test with comma > detection enabled; I will test that next. > > Tested on the QCS9100 Ride R3 board with 2X AQR115C PHYs. I have one > more board that I can test next week (IQ8275, which has a single > QCA8081 PHY, but that is limited to 2.5G because the PHY switches its > mode according to the speed). Thanks for testing! Given the results you've given, my suggestion would be that the following patches are probably the most risky: Patch 2 "net: stmmac: qcom-ethqos: convert to set_clk_tx_rate() method" This changes the way the clock is configured. It would be worth testing that and giving a tested-by for the first two patches if that's successful. Patch 6 "net: stmmac: qcom-ethqos: convert to dwmac generic SerDes support" This changes how the SerDes is handled, which is a significant change. It's possible that the PHY calibration needs some other state to be appropriately configured, and that's not happening in the same order. Either of these two would account for what appears to be an unstable SerDes link. As for the NULL pointer deref, I'll look at that in a bit (waiting for a build to complete so I can hopefully pinpoint where the oops is happening.) -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!