mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2] ARM: configs: sama5: enable current Microchip KSZ DSA symbols
@ 2026-08-14  7:53 Pengpeng Hou
  2026-09-19 14:38 ` claudiu beznea
  0 siblings, 1 reply; 3+ messages in thread
From: Pengpeng Hou @ 2026-08-14  7:53 UTC (permalink / raw)
  To: Nicolas Ferre, Alexandre Belloni, Claudiu Beznea
  Cc: Russell King, linux-arm-kernel, linux-kernel, Pengpeng Hou

The Microchip KSZ DSA family now uses NET_DSA_MICROCHIP_KSZ_COMMON
for the common provider and NET_DSA_MICROCHIP_KSZ_SPI for SPI-attached
switches.

sama5_defconfig still enables the removed KSZ9477-specific symbols even
though SAMA5 device trees contain KSZ SPI switch consumers. Replace the
stale entries with the current provider symbols.

Assisted-by: Codex:gpt-5
Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
---
Changes since v1: https://lore.kernel.org/all/20260624081216.75559-1-pengpeng@iscas.ac.cn/
- no source-code changes
- rebase on the current SAMA5 defconfig
- route the revision to the AT91 SoC maintainers
- add the coding-assistant disclosure

The Kconfig/provider mapping was reviewed statically; no SAMA5 kernel build
was performed.

 arch/arm/configs/sama5_defconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
index bd7f0b5f7d66..46e7f554faa1 100644
--- a/arch/arm/configs/sama5_defconfig
+++ b/arch/arm/configs/sama5_defconfig
@@ -70,8 +70,8 @@ CONFIG_BLK_DEV_SD=y
 # CONFIG_BLK_DEV_BSG is not set
 # CONFIG_SCSI_LOWLEVEL is not set
 CONFIG_NETDEVICES=y
-CONFIG_NET_DSA_MICROCHIP_KSZ9477=m
-CONFIG_NET_DSA_MICROCHIP_KSZ9477_SPI=m
+CONFIG_NET_DSA_MICROCHIP_KSZ_COMMON=m
+CONFIG_NET_DSA_MICROCHIP_KSZ_SPI=m
 # CONFIG_NET_VENDOR_BROADCOM is not set
 CONFIG_MACB=y
 # CONFIG_NET_VENDOR_CIRRUS is not set
-- 
2.50.1 (Apple Git-155)


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] ARM: configs: sama5: enable current Microchip KSZ DSA symbols
  2026-08-14  7:53 [PATCH v2] ARM: configs: sama5: enable current Microchip KSZ DSA symbols Pengpeng Hou
@ 2026-09-19 14:38 ` claudiu beznea
  2026-09-20  3:09   ` Pengpeng Hou
  0 siblings, 1 reply; 3+ messages in thread
From: claudiu beznea @ 2026-09-19 14:38 UTC (permalink / raw)
  To: Pengpeng Hou, Nicolas Ferre, Alexandre Belloni
  Cc: Russell King, linux-arm-kernel, linux-kernel



On 8/14/26 10:53, Pengpeng Hou wrote:
> The Microchip KSZ DSA family now uses NET_DSA_MICROCHIP_KSZ_COMMON
> for the common provider and NET_DSA_MICROCHIP_KSZ_SPI for SPI-attached
> switches.
> 
> sama5_defconfig still enables the removed KSZ9477-specific symbols even
> though SAMA5 device trees contain KSZ SPI switch consumers. Replace the
> stale entries with the current provider symbols.
> 
> Assisted-by: Codex:gpt-5
> Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>

Fixes: 07bca160469b ("net: dsa: microchip: common menuconfig for ksz series switch")
Fixes: 4658f2fe8fbc ("net: dsa: microchip: common ksz_spi_probe for ksz switches")

Reviewed-by: Claudiu Beznea <claudiu.beznea@tuxon.dev>

> ---
> Changes since v1: https://lore.kernel.org/all/20260624081216.75559-1-pengpeng@iscas.ac.cn/
> - no source-code changes
> - rebase on the current SAMA5 defconfig
> - route the revision to the AT91 SoC maintainers
> - add the coding-assistant disclosure
> 
> The Kconfig/provider mapping was reviewed statically; no SAMA5 kernel build
> was performed.
> 
>   arch/arm/configs/sama5_defconfig | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
> index bd7f0b5f7d66..46e7f554faa1 100644
> --- a/arch/arm/configs/sama5_defconfig
> +++ b/arch/arm/configs/sama5_defconfig
> @@ -70,8 +70,8 @@ CONFIG_BLK_DEV_SD=y
>   # CONFIG_BLK_DEV_BSG is not set
>   # CONFIG_SCSI_LOWLEVEL is not set
>   CONFIG_NETDEVICES=y
> -CONFIG_NET_DSA_MICROCHIP_KSZ9477=m
> -CONFIG_NET_DSA_MICROCHIP_KSZ9477_SPI=m
> +CONFIG_NET_DSA_MICROCHIP_KSZ_COMMON=m
> +CONFIG_NET_DSA_MICROCHIP_KSZ_SPI=m
>   # CONFIG_NET_VENDOR_BROADCOM is not set
>   CONFIG_MACB=y
>   # CONFIG_NET_VENDOR_CIRRUS is not set


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] ARM: configs: sama5: enable current Microchip KSZ DSA symbols
  2026-09-19 14:38 ` claudiu beznea
@ 2026-09-20  3:09   ` Pengpeng Hou
  0 siblings, 0 replies; 3+ messages in thread
From: Pengpeng Hou @ 2026-09-20  3:09 UTC (permalink / raw)
  To: claudiu.beznea
  Cc: nicolas.ferre, alexandre.belloni, linux, linux-arm-kernel,
	linux-kernel, hppiscas

Hi Claudiu,

I checked both changes. The first removed the KSZ9477 family option in
favour of the common option, and the second renamed the SPI provider.
Please include both tags with the patch:

Fixes: 07bca160469b ("net: dsa: microchip: common menuconfig for ksz series switch")
Fixes: 4658f2fe8fbc ("net: dsa: microchip: common ksz_spi_probe for ksz switches")

Thanks for the review.

Pengpeng


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-09-20  3:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-08-14  7:53 [PATCH v2] ARM: configs: sama5: enable current Microchip KSZ DSA symbols Pengpeng Hou
2026-09-19 14:38 ` claudiu beznea
2026-09-20  3:09   ` Pengpeng Hou

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®