arm64: Kconfig: provide a top-level switch for Microchip platforms

[PATCH] arm64: Kconfig: provide a top-level switch for Microchip platforms

Posted by Bartosz Golaszewski 1 month, 2 weeks ago

Microchip is the only platform that doesn't have a top-level switch for
disabling all SoC families. Align it with other platforms and update the
arm64 defconfig to keep the default config the same.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
---
 arch/arm64/Kconfig.platforms | 10 ++++------
 arch/arm64/configs/defconfig |  1 +
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 54eb1d7fd419b..72c812e76b0b1 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -188,16 +188,15 @@ config ARCH_MESON
 	  This enables support for the arm64 based Amlogic SoCs
 	  such as the s905, S905X/D, S912, A113X/D or S905X/D2
 
-menu "Microchip SoC support"
+menuconfig ARCH_MICROCHIP
+	bool "Microchip SoC support"
 
-config ARCH_MICROCHIP
-	bool
+if ARCH_MICROCHIP
 
 config ARCH_LAN969X
 	bool "Microchip LAN969X SoC family"
 	select PINCTRL
 	select DW_APB_TIMER_OF
-	select ARCH_MICROCHIP
 	help
 	  This enables support for the Microchip LAN969X ARMv8-based
 	  SoC family of TSN-capable gigabit switches.
@@ -212,7 +211,6 @@ config ARCH_SPARX5
 	bool "Microchip Sparx5 SoC family"
 	select PINCTRL
 	select DW_APB_TIMER_OF
-	select ARCH_MICROCHIP
 	help
 	  This enables support for the Microchip Sparx5 ARMv8-based
 	  SoC family of TSN-capable gigabit switches.
@@ -223,7 +221,7 @@ config ARCH_SPARX5
 	  security through TCAM-based frame processing using versatile
 	  content aware processor (VCAP).
 
-endmenu
+endif
 
 config ARCH_MMP
 	bool "Marvell MMP SoC Family"
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index d1e894b988f99..580b7ab2ea05f 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -57,6 +57,7 @@ CONFIG_ARCH_HISI=y
 CONFIG_ARCH_KEEMBAY=y
 CONFIG_ARCH_MEDIATEK=y
 CONFIG_ARCH_MESON=y
+CONFIG_ARCH_MICROCHIP=y
 CONFIG_ARCH_MVEBU=y
 CONFIG_ARCH_NXP=y
 CONFIG_ARCH_LAYERSCAPE=y
-- 
2.47.3

Re: [PATCH] arm64: Kconfig: provide a top-level switch for Microchip platforms

Posted by Krzysztof Kozlowski 1 month, 2 weeks ago

On 26/02/2026 15:36, Bartosz Golaszewski wrote:
> Microchip is the only platform that doesn't have a top-level switch for
> disabling all SoC families. Align it with other platforms and update the
> arm64 defconfig to keep the default config the same.
> 
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> ---
>  arch/arm64/Kconfig.platforms | 10 ++++------
>  arch/arm64/configs/defconfig |  1 +
>  2 files changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index 54eb1d7fd419b..72c812e76b0b1 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -188,16 +188,15 @@ config ARCH_MESON
>  	  This enables support for the arm64 based Amlogic SoCs
>  	  such as the s905, S905X/D, S912, A113X/D or S905X/D2
>  
> -menu "Microchip SoC support"
> +menuconfig ARCH_MICROCHIP
> +	bool "Microchip SoC support"
>  
> -config ARCH_MICROCHIP
> -	bool
> +if ARCH_MICROCHIP

This will surprise all out-of-tree non-defconfig users, because their
boards will stop working (drivers depend on ARCH_LAN969X|ARCH_SPARX5
which will be now disabled), but I like the consistency.

I guess this will go via Microchip arm64 soc tree.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

Best regards,
Krzysztof

Re: [PATCH] arm64: Kconfig: provide a top-level switch for Microchip platforms

Posted by Bartosz Golaszewski 1 month, 2 weeks ago

On Thu, Feb 26, 2026 at 3:49 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 26/02/2026 15:36, Bartosz Golaszewski wrote:
> > Microchip is the only platform that doesn't have a top-level switch for
> > disabling all SoC families. Align it with other platforms and update the
> > arm64 defconfig to keep the default config the same.
> >
> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> > ---
> >  arch/arm64/Kconfig.platforms | 10 ++++------
> >  arch/arm64/configs/defconfig |  1 +
> >  2 files changed, 5 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> > index 54eb1d7fd419b..72c812e76b0b1 100644
> > --- a/arch/arm64/Kconfig.platforms
> > +++ b/arch/arm64/Kconfig.platforms
> > @@ -188,16 +188,15 @@ config ARCH_MESON
> >         This enables support for the arm64 based Amlogic SoCs
> >         such as the s905, S905X/D, S912, A113X/D or S905X/D2
> >
> > -menu "Microchip SoC support"
> > +menuconfig ARCH_MICROCHIP
> > +     bool "Microchip SoC support"
> >
> > -config ARCH_MICROCHIP
> > -     bool
> > +if ARCH_MICROCHIP
>
> This will surprise all out-of-tree non-defconfig users, because their
> boards will stop working (drivers depend on ARCH_LAN969X|ARCH_SPARX5
> which will be now disabled), but I like the consistency.
>

I'm not following, ARCH_SPARX5 remains enabled in defconfig and
ARCH_LAN969X was already disabled by default.

> I guess this will go via Microchip arm64 soc tree.
>

I think this should be picked up directly into the ARM SoC tree to
avoid conflicts later into the cycle.

Bartosz

> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
>
> Best regards,
> Krzysztof

Re: [PATCH] arm64: Kconfig: provide a top-level switch for Microchip platforms

Posted by Krzysztof Kozlowski 1 month, 2 weeks ago

On 27/02/2026 09:42, Bartosz Golaszewski wrote:
> On Thu, Feb 26, 2026 at 3:49 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 26/02/2026 15:36, Bartosz Golaszewski wrote:
>>> Microchip is the only platform that doesn't have a top-level switch for
>>> disabling all SoC families. Align it with other platforms and update the
>>> arm64 defconfig to keep the default config the same.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
>>> ---
>>>  arch/arm64/Kconfig.platforms | 10 ++++------
>>>  arch/arm64/configs/defconfig |  1 +
>>>  2 files changed, 5 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>>> index 54eb1d7fd419b..72c812e76b0b1 100644
>>> --- a/arch/arm64/Kconfig.platforms
>>> +++ b/arch/arm64/Kconfig.platforms
>>> @@ -188,16 +188,15 @@ config ARCH_MESON
>>>         This enables support for the arm64 based Amlogic SoCs
>>>         such as the s905, S905X/D, S912, A113X/D or S905X/D2
>>>
>>> -menu "Microchip SoC support"
>>> +menuconfig ARCH_MICROCHIP
>>> +     bool "Microchip SoC support"
>>>
>>> -config ARCH_MICROCHIP
>>> -     bool
>>> +if ARCH_MICROCHIP
>>
>> This will surprise all out-of-tree non-defconfig users, because their
>> boards will stop working (drivers depend on ARCH_LAN969X|ARCH_SPARX5
>> which will be now disabled), but I like the consistency.
>>
> 
> I'm not following, ARCH_SPARX5 remains enabled in defconfig and
> ARCH_LAN969X was already disabled by default.

That's why I said non-defconfig.


Best regards,
Krzysztof