From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E3FF0C88E50 for ; Fri, 11 Sep 2026 11:26:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version: Content-Transfer-Encoding:Content-Type:References:In-Reply-To:Message-ID:Date :Cc:To:From:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Tat9mq7+c+2wVgjCeNs6A4pXm6W+juN5KefobLoQo9k=; b=B0Ceml6IDcC4rhCNGB4vI78Mfo rZKcRMwxeHKGYX6QOXbYyq0j0nYhGEkv1yRL0QEpekcRFIBwBI0d06yUi/ug9hbDmYMxDSks50+I9 f8wVH6hjU5487h47WLJ66kaHQDzunYeMx5dUd05oXz8ZOEyeWFV5MIAyrBw5kypbDrrMWG9xdG6E6 1lBH0o7aguWdGtnGGDvFiB3kWZLPO+X1Q47OZ7rUZcJ/dSBN45NHmcaR2PgIjAC3RaOIY67Jkotle oXo8kIMMB4x1IiiXX/PC4gRypoiAK3TWKDiTrNBYNvvrYZjEA+U53yHljqBVoiUcUUFua5vXTcjVD AzC8GyAw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x4zOF-0000000GUcg-2Sxi; Fri, 11 Sep 2026 11:25:55 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x4zNf-0000000GUOC-17bL for linux-arm-kernel@lists.infradead.org; Fri, 11 Sep 2026 11:25:22 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 09EED41885; Fri, 11 Sep 2026 11:25:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4FBC91F0089B; Fri, 11 Sep 2026 11:25:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1789125918; bh=Tat9mq7+c+2wVgjCeNs6A4pXm6W+juN5KefobLoQo9k=; h=Subject:From:To:Cc:Date:In-Reply-To:References; b=JG3FUD5WC0loPVITE99b8rSpPe+3pzbh5U+mDnRuYFnzgrcGV9L5vTYpNxPM8r2yy 61P4MOcbwypVzEjjeviACdXahJxdqE8IGiXHqknf1gkEuk8i5q2hCQVOVpYPVc77dn FWadNFYGbMuMGRNwUob2IB3myd2Zi9fM/xoq2kWgegnUVNpLZVRIBG72w7aDnHHRkQ dFXDIHjdb8Xtp++DVFQft/+Gm0ihNoHsy8TMG9YKOADk6cdikRnX65RNrdivuu3Wk9 mohBN1gswc2zLba12y3ePu1+VAmc4BGP8BEk6uKboTmXI0JK927sS2ieOOvP/4N6V0 J8X7fa+IqlHrw== Subject: Re: [PATCH net-next v2 6/9] net: stmmac: qcom-ethqos: warn about legacy RGMII PHY modes From: netdev-bot+sashiko@kernel.org To: mohd.anwar@oss.qualcomm.com Cc: andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, afd@ti.com, andrew@lunn.ch, hkallweit1@gmail.com, linux@armlinux.org.uk, richardcochran@gmail.com, andersson@kernel.org, konradybcio@kernel.org, alexandre.torgue@foss.st.com, peppe.cavallaro@st.com, joabreu@synopsys.com, maxime.chevallier@bootlin.com, mcoquelin.stm32@gmail.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Date: Fri, 11 Sep 2026 11:25:16 +0000 Message-ID: <178912591683.219967.17377991068527504766@kernel.org> In-Reply-To: <20260908-shikra_ethernet-v2-6-bbe3389d0652@oss.qualcomm.com> References: <20260908-shikra_ethernet-v2-6-bbe3389d0652@oss.qualcomm.com> X-sashiko-severity: Medium Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit MIME-Version: 1.0 X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider: Critical: 0 · High: 0 · Medium: 1 · Low: 1 - [Medium] Unactionable boot-time warning on every in-tree qcom ethqos platform: the new dev_warn() in qcom_ethqos_probe() (dwmac-qcom-… - [Low] Commit message / code mismatch: the changelog of "net: stmmac: qcom-ethqos: warn about legacy RGMII PHY modes" states that "existing… -- net: stmmac: qcom-ethqos: warn about legacy RGMII PHY modes This adds a probe-time dev_warn() in qcom_ethqos_probe() when the ethernet node uses one of the older RGMII phy-mode values, suggesting a move to "rgmii-id" with PHY-provided delays. PHY_INTERFACE_MODE_RGMII_ID is split out of the shared case group so that it does not warn. > diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c > index 53fb9de4d92c7..a92bf4a6a2dbd 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c > +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c > @@ -764,9 +764,11 @@ static int qcom_ethqos_probe(struct platform_device *pdev) > ethqos->phy_mode = plat_dat->phy_interface; > switch (ethqos->phy_mode) { > case PHY_INTERFACE_MODE_RGMII: > - case PHY_INTERFACE_MODE_RGMII_ID: > case PHY_INTERFACE_MODE_RGMII_RXID: > case PHY_INTERFACE_MODE_RGMII_TXID: > + dev_warn(dev, "legacy RGMII phy-mode detected; consider upgrading to a newer DTB\n"); [Severity: Low] The changelog says: However, existing DTBs use "rgmii" and "rgmii-txid" and must remain supported for backwards compatibility. Warn when either of these legacy PHY modes is used [...] but the warning in qcom_ethqos_probe() is attached to a group of three case labels, so PHY_INTERFACE_MODE_RGMII_RXID also warns. Is "rgmii-rxid" intended to be declared legacy as well? The driver does treat these modes differently, for example in ethqos_rgmii_macro_init(): if (ethqos->phy_mode == PHY_INTERFACE_MODE_RGMII_TXID) phase_shift = 0; else phase_shift = RGMII_CONFIG2_TX_CLK_PHASE_SHIFT_EN; so which phy-modes are being deprecated seems worth stating exactly in the commit message. [Severity: Medium] Is there an upstream DTB that this warning can point people at? No in-tree device tree for this driver uses "rgmii-id", and this series does not touch arch/ at all, so every in-tree board hits the new warning on each probe: arch/arm64/boot/dts/qcom/qcs404-evb-4000.dts phy-mode = "rgmii"; arch/arm64/boot/dts/qcom/sa8155p-adp.dts phy-mode = "rgmii"; arch/arm64/boot/dts/qcom/sa8540p-ride.dts phy-mode = "rgmii-txid"; (both MACs) The binding example in Documentation/devicetree/bindings/net/qcom,ethqos.yaml also still uses: phy-mode = "rgmii"; and the binding carries no phy-mode deprecation note, so the documented ABI still recommends what the driver now calls legacy. Converting those device trees is also the only way the advocated PHY-provided-delay path gets exercised, since ethqos_set_clk_tx_rate() stops doubling the link clock only for PHY_INTERFACE_MODE_RGMII_ID. Would it make sense to convert the in-tree DTs (and the binding example/text) in the same series, or to hold off on the warning until a migration target exists? > + fallthrough; > + case PHY_INTERFACE_MODE_RGMII_ID: > plat_dat->fix_mac_speed = ethqos_fix_mac_speed_rgmii; > break; > case PHY_INTERFACE_MODE_2500BASEX: -- Sashiko AI review · https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260908-shikra_ethernet-v2-0-bbe3389d0652%40oss.qualcomm.com