From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 89D0C51A13F; Mon, 21 Sep 2026 23:43:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790034242; cv=none; b=b6JuHhKHHSYGYEKN5HSmJPw1n47qKu/x7gJPeNR5695RbA43ku0NdB6TGEQkl4sm6MjKDBUfDBU0daH+qRKqscm3TcqqJInKFuWIV7itqpYhLk9F+50tMvEpKG4152svW35WeCQAdQgc/LG70AUbHEUIdf90IYDuKvmz1XwgLzY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790034242; c=relaxed/simple; bh=I+VfiP8hGoJQQOdgUTc7/x7yfC/MnLJ0hRhTIx/inrI=; h=Subject:From:To:Cc:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=PD//cADC/VJyrvEmtcjXSk8tB8ZZHFRugQ48O31rjaYOlQayLC6ZtWW3ZEYUumdxrV6h/zF7kIM0uIRSEX8ufQkA26A8VVaSwN7h1BvNSBQV1eKHMcCtJqEHngqY1qUc+KfSTRo9KMRN48CfeSB2SdnzQN6Uj4PJGbvc9STcNGs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=kN2kcqeg; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="kN2kcqeg" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0A2B41F00893; Mon, 21 Sep 2026 23:43:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790034236; bh=ZG4q1bWR2eMGYdwDsubP8MPzWiDbvYTpOk4BFn7wEjk=; h=Subject:From:To:Cc:Date:In-Reply-To:References; b=kN2kcqegqTrBXWD/LP2j0qpShjoMhypaQ04GgiCeWD/cTHdjADw7IgIxgUCBUpSrJ HIUKVpRQAECQ3rnpwScVKxCNzCzfYKJ1CfG2sYvvYGAfWUvTzpv7DUZhoeUhbmkU36 OU7PMMYWLGmZEf15yOx7qdoxqLLuJkOXBUMLtNnjfe7sUe91tbTzFHq1DROx+sZe3I mQH6j5tPUvb/1Q8MbwVEqW291mEa1TGMzy+3NtxOn5nUVrkaufWAQFnWAyayA0MVCI l0yG5qwCy1TJCRlSX3FJOTXvKT8qMj9zPlM2UqZKxvoU759FW2nWTnQXI74cFI2y/9 Sk/mcIkUcFIPw== Subject: Re: [PATCH net-next v7 10/11] arm64: dts: rockchip: rk3568-photonicat: enable SGMII LAN port From: netdev-bot+sashiko@kernel.org To: coiaprant@gmail.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, heiko@sntech.de, vkoul@kernel.org, maxime.chevallier@bootlin.com, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com, prabhakar.mahadev-lad.rj@bp.renesas.com, romain.gantois@bootlin.com, hkallweit1@gmail.com, neil.armstrong@linaro.org, linux@armlinux.org.uk, shawn.lin@rock-chips.com, david@ixit.cz, netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-renesas-soc@vger.kernel.org Date: Mon, 21 Sep 2026 23:43:54 +0000 Message-ID: <179003423458.2160803.7280358019218159785@kernel.org> In-Reply-To: <20260917204624.2427486-11-coiaprant@gmail.com> References: <20260917204624.2427486-11-coiaprant@gmail.com> X-sashiko-severity: Low Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Thank you for your contribution! Sashiko AI review found more severe issue(s) in other patches of this series. This mail is only an FYI, in case you want to address the nit picks below when you respin. Low priority / nit picks for your consideration: - [Low] Unrelated functional DT change bundled into an "enable SGMII LAN port" patch: the hunk under `&mdio1` adds a `leds { led@1; led@2; }`… -- arm64: dts: rockchip: rk3568-photonicat: enable SGMII LAN port This enables GMAC0 on the Ariaboard Photonicat by adding the XPCS SGMII plumbing (combphy2 mac-sel, xpcs/xpcs_mii0, gmac0 properties) and describing the Motorcomm YT8521SC PHY on a new mdio0 node. It also adds a PHY LED description to the existing RGMII PHY on mdio1. > diff --git a/arch/arm64/boot/dts/rockchip/rk3568-photonicat.dts b/arch/arm64/boot/dts/rockchip/rk3568-photonicat.dts > index 58c1052ba8ef3..fdaa4a2a4328b 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3568-photonicat.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3568-photonicat.dts [ ... ] > @@ -350,6 +391,25 @@ rgmii_phy: ethernet-phy@3 { > reset-gpios = <&gpio4 RK_PC0 GPIO_ACTIVE_LOW>; > rx-internal-delay-ps = <1500>; > tx-internal-delay-ps = <1500>; > + > + leds { > + #address-cells = <1>; > + #size-cells = <0>; > + > + led@1 { > + reg = <1>; > + color = ; > + function = LED_FUNCTION_WAN; > + default-state = "keep"; > + }; > + > + led@2 { > + reg = <2>; > + color = ; > + function = LED_FUNCTION_WAN; > + default-state = "keep"; > + }; > + }; > }; > }; [Severity: Low] This isn't a bug, but could this hunk go into a separate patch? It changes the pre-existing RGMII WAN PHY (rgmii_phy under &mdio1), which is already enabled and working before this patch: arch/arm64/boot/dts/rockchip/rk3568-photonicat.dts: &gmac1 { phy-handle = <&rgmii_phy>; phy-mode = "rgmii-id"; status = "okay"; }; None of the SGMII enablement in this patch (&combphy2 rockchip,sgmii-mac-sel, &gmac0, &mdio0/sgmii_phy, &xpcs, &xpcs_mii0) depends on the mdio1 leds node. It is also not purely descriptive - drivers/net/phy/motorcomm.c implements the LED callbacks for this PHY (yt8521_led_hw_is_supported(), yt8521_led_hw_control_set(), yt8521_led_hw_control_get()), so PHY LED class devices get instantiated for the WAN PHY as a result of this change. Split into its own patch, the WAN LED description and the LAN port enablement can be bisected and reverted independently, and the subject line would reflect that the WAN PHY description changed too. The commit message mentions it: Also add LED configuration for the existing RGMII PHY on mdio1 for consistency. but the subject only mentions the SGMII LAN port. [ ... ] -- Sashiko AI review · https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260917204624.2427486-1-coiaprant%40gmail.com