diff mbox series

[V3,1/3] arm64: add config for Broadcom BCM4908 SoCs

Message ID 20201112072133.17039-1-zajec5@gmail.com (mailing list archive)
State New, archived
Headers show
Series [V3,1/3] arm64: add config for Broadcom BCM4908 SoCs | expand

Commit Message

Rafał Miłecki Nov. 12, 2020, 7:21 a.m. UTC 
From: Rafał Miłecki <rafal@milecki.pl>

Add ARCH_BCM4908 config that can be used for compiling DTS files.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
 arch/arm64/Kconfig.platforms | 8 ++++++++
 arch/arm64/configs/defconfig | 1 +
 2 files changed, 9 insertions(+)

Comments

Baruch Siach Nov. 12, 2020, 7:34 a.m. UTC | #1 
Hi Rafał,

On Thu, Nov 12 2020, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> Add ARCH_BCM4908 config that can be used for compiling DTS files.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
>  arch/arm64/Kconfig.platforms | 8 ++++++++
>  arch/arm64/configs/defconfig | 1 +
>  2 files changed, 9 insertions(+)
>
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index 6f2494dd6d60..d1c02a4a93c8 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -43,6 +43,14 @@ config ARCH_BCM2835
>  	  This enables support for the Broadcom BCM2837 and BCM2711 SoC.
>  	  These SoCs are used in the Raspberry Pi 3 and 4 devices.
>  
> +config ARCH_BCM4908
> +	bool "Broadcom BCM4908 family"
> +	select GPIOLIB
> +	help
> +	  This enables support for the Broadcom BCM4906, BCM4908 and
> +	  BCM49408 SoCs. These SoCs use Cortex-B53 cores and can be

Is Cortex-B53 an official name? I see reference to Brahma-B53 in other
places like Documentation/arm64/silicon-errata.rst.

baruch

> +	  found in home routers.
> +
>  config ARCH_BCM_IPROC
>  	bool "Broadcom iProc SoC Family"
>  	select COMMON_CLK_IPROC
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 17a2df6a263e..ead6fa942994 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -32,6 +32,7 @@ CONFIG_ARCH_AGILEX=y
>  CONFIG_ARCH_SUNXI=y
>  CONFIG_ARCH_ALPINE=y
>  CONFIG_ARCH_BCM2835=y
> +CONFIG_ARCH_BCM4908=y
>  CONFIG_ARCH_BCM_IPROC=y
>  CONFIG_ARCH_BERLIN=y
>  CONFIG_ARCH_BRCMSTB=y
Rafał Miłecki Nov. 12, 2020, 7:53 a.m. UTC | #2 
On 12.11.2020 08:34, Baruch Siach wrote:
> On Thu, Nov 12 2020, Rafał Miłecki wrote:
>> From: Rafał Miłecki <rafal@milecki.pl>
>>
>> Add ARCH_BCM4908 config that can be used for compiling DTS files.
>>
>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>> ---
>>   arch/arm64/Kconfig.platforms | 8 ++++++++
>>   arch/arm64/configs/defconfig | 1 +
>>   2 files changed, 9 insertions(+)
>>
>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>> index 6f2494dd6d60..d1c02a4a93c8 100644
>> --- a/arch/arm64/Kconfig.platforms
>> +++ b/arch/arm64/Kconfig.platforms
>> @@ -43,6 +43,14 @@ config ARCH_BCM2835
>>   	  This enables support for the Broadcom BCM2837 and BCM2711 SoC.
>>   	  These SoCs are used in the Raspberry Pi 3 and 4 devices.
>>   
>> +config ARCH_BCM4908
>> +	bool "Broadcom BCM4908 family"
>> +	select GPIOLIB
>> +	help
>> +	  This enables support for the Broadcom BCM4906, BCM4908 and
>> +	  BCM49408 SoCs. These SoCs use Cortex-B53 cores and can be
> 
> Is Cortex-B53 an official name? I see reference to Brahma-B53 in other
> places like Documentation/arm64/silicon-errata.rst.

That should be Brahma-B53. Good catch.

Initially I didn't know "Brahma" name and thought (Brahma-) B53 is just
minor variation of Cortex-B53. I even used:
compatible = "brcm,cortex-b53", "arm,cortex-a53";

It's clarified now but I forgot to update config help.
diff mbox series

Patch

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 6f2494dd6d60..d1c02a4a93c8 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -43,6 +43,14 @@  config ARCH_BCM2835
 	  This enables support for the Broadcom BCM2837 and BCM2711 SoC.
 	  These SoCs are used in the Raspberry Pi 3 and 4 devices.
 
+config ARCH_BCM4908
+	bool "Broadcom BCM4908 family"
+	select GPIOLIB
+	help
+	  This enables support for the Broadcom BCM4906, BCM4908 and
+	  BCM49408 SoCs. These SoCs use Cortex-B53 cores and can be
+	  found in home routers.
+
 config ARCH_BCM_IPROC
 	bool "Broadcom iProc SoC Family"
 	select COMMON_CLK_IPROC
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 17a2df6a263e..ead6fa942994 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -32,6 +32,7 @@  CONFIG_ARCH_AGILEX=y
 CONFIG_ARCH_SUNXI=y
 CONFIG_ARCH_ALPINE=y
 CONFIG_ARCH_BCM2835=y
+CONFIG_ARCH_BCM4908=y
 CONFIG_ARCH_BCM_IPROC=y
 CONFIG_ARCH_BERLIN=y
 CONFIG_ARCH_BRCMSTB=y