diff mbox series

[1/2] ARM: bcm: Add selects for the Cortex-A7 BCMBCA

Message ID 20240917-bcm-arm-bcm6846-v1-1-236e29661f4c@linaro.org (mailing list archive)
State New
Headers show
Series ARM: bcm: Make BCM6846 bootable on mainline | expand

Commit Message

Linus Walleij Sept. 17, 2024, 7:18 p.m. UTC
The Cortex-A7 versions of BCMBCA have most of the stuff
that the Cortex-A9 version has and it can be selected
independently with the Cortex-A9 disabled, so make sure
it will also select all needed features.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
 arch/arm/mach-bcm/Kconfig | 5 +++++
 1 file changed, 5 insertions(+)

Comments

William Zhang Sept. 19, 2024, 1:04 a.m. UTC | #1
Hi Linus,

On 09/17/2024 12:18 PM, Linus Walleij wrote:
> The Cortex-A7 versions of BCMBCA have most of the stuff
> that the Cortex-A9 version has and it can be selected
> independently with the Cortex-A9 disabled, so make sure
> it will also select all needed features.
> 
Did you use the multi_v7_defconfig? I tested all the BCMBCA
A7 chips and there is no extra config needed if use the
multi_v7 config.

BTW, ARM_GLOBAL_TIMER, HAVE_ARM_TWD and HAVE_ARM_SCU do not
apply to A7.  A7 has integrated L2 controller and does not
need this external CACHE_L2X0 config.
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
>   arch/arm/mach-bcm/Kconfig | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> index 7318d8789e24..39d1d713efe0 100644
> --- a/arch/arm/mach-bcm/Kconfig
> +++ b/arch/arm/mach-bcm/Kconfig
> @@ -217,6 +217,11 @@ if ARCH_BCMBCA
>   
>   config ARCH_BCMBCA_CORTEXA7
>   	bool "Cortex-A7 SoCs"
> +	select ARCH_HAS_RESET_CONTROLLER
> +	select ARM_GLOBAL_TIMER
> +	select CACHE_L2X0
> +	select HAVE_ARM_TWD if SMP
> +	select HAVE_ARM_SCU if SMP
>   	help
>   	  Say Y if you intend to run the kernel on a Broadcom Broadband ARM A7
>   	  based chipset.
>
diff mbox series

Patch

diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
index 7318d8789e24..39d1d713efe0 100644
--- a/arch/arm/mach-bcm/Kconfig
+++ b/arch/arm/mach-bcm/Kconfig
@@ -217,6 +217,11 @@  if ARCH_BCMBCA
 
 config ARCH_BCMBCA_CORTEXA7
 	bool "Cortex-A7 SoCs"
+	select ARCH_HAS_RESET_CONTROLLER
+	select ARM_GLOBAL_TIMER
+	select CACHE_L2X0
+	select HAVE_ARM_TWD if SMP
+	select HAVE_ARM_SCU if SMP
 	help
 	  Say Y if you intend to run the kernel on a Broadcom Broadband ARM A7
 	  based chipset.