Re: [PATCH net-next v5 09/11] net: stmmac: dwmac-rk: add SGMII support for RK3568
From: sashiko-bot@kernel.org
Date: 2026-09-14 19:44:27
Also in:
linux-phy
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Medium] DWMAC_ROCKCHIP selects PCS_XPCS_ROCKCHIP without inheriting its dependencies, causing Kconfig warnings. -- commit a034d04f6974ebd2055a96c9de8c8e79f795fefe Author: Coia Prant [off-list ref] net: stmmac: dwmac-rk: add SGMII support for RK3568 This commit adds SGMII support for the RK3568 SoC in the dwmac-rk driver. It introduces callbacks to configure the GMAC GRF register for SGMII mode, implements PCS initialization to create the XPCS, and configures phylink for in-band negotiation. Link: https://dl.radxa.com/rock3/docs/hw/datasheet/Rockchip%20RK3568%20TRM%20Part1%20V1.1-20210301.pdf (Page 386)
quoted hunk ↗ jump to hunk
diff --git a/drivers/net/ethernet/stmicro/stmmac/Kconfig b/drivers/net/ethernet/stmicro/stmmac/Kconfig index ab3c4cf964239..c6559e749c0ce 100644 --- a/drivers/net/ethernet/stmicro/stmmac/Kconfig +++ b/drivers/net/ethernet/stmicro/stmmac/Kconfig@@ -170,6 +170,7 @@ config DWMAC_ROCKCHIP default ARCH_ROCKCHIP depends on OF && (ARCH_ROCKCHIP || COMPILE_TEST) select MFD_SYSCON + select PCS_XPCS_ROCKCHIP
[Severity: Medium] Does selecting PCS_XPCS_ROCKCHIP under config DWMAC_ROCKCHIP require inheriting its dependencies? If a kernel is built with DWMAC_ROCKCHIP enabled, but GENERIC_PHY or PM_GENERIC_DOMAINS is disabled, this select statement forces PCS_XPCS_ROCKCHIP to be enabled without its underlying dependencies. This can cause Kconfig unmet direct dependencies warnings at build time and potentially lead to silent build failures. Would it be better to add the required dependencies to DWMAC_ROCKCHIP? -- Sashiko AI review · https://sashiko.dev/#/patchset/20260913185849.907479-1-coiaprant@gmail.com?part=9