diff mbox series

[5/9] arch/powerpc: Add Power9 CPU type

Message ID 20220729000904.1295295-6-joel@jms.id.au
State Accepted
Headers show
Series powerpc: Fix ppc64le configurations | expand

Commit Message

Joel Stanley July 29, 2022, 12:09 a.m. UTC
The Power9 CPU was released by IBM in 2016. It is a 64-bit CPU
implementing ISA v3.0. It runs in big and little endian modes, and
supports VSX and AltiVec.

Power9 is supported as of GCC 6.1 and Binutils 2.26.

Signed-off-by: Joel Stanley <joel@jms.id.au>
---
 arch/Config.in.powerpc | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Cédric Le Goater July 29, 2022, 6:53 a.m. UTC | #1
On 7/29/22 02:09, Joel Stanley wrote:
> The Power9 CPU was released by IBM in 2016. It is a 64-bit CPU
> implementing ISA v3.0. It runs in big and little endian modes, and
> supports VSX and AltiVec.
> 
> Power9 is supported as of GCC 6.1 and Binutils 2.26.

Ah. That's what I was asking for !

Reviewed-by: Cédric Le Goater <clg@kaod.org>

Thanks,

C.

> 
> Signed-off-by: Joel Stanley <joel@jms.id.au>
> ---
>   arch/Config.in.powerpc | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
> index 7c6ae84348e9..4c633c338d86 100644
> --- a/arch/Config.in.powerpc
> +++ b/arch/Config.in.powerpc
> @@ -149,6 +149,12 @@ config BR2_powerpc_power8
>   	depends on BR2_ARCH_IS_64
>   	select BR2_POWERPC_CPU_HAS_ALTIVEC
>   	select BR2_POWERPC_CPU_HAS_VSX
> +config BR2_powerpc_power9
> +	bool "power9"
> +	depends on BR2_ARCH_IS_64
> +	select BR2_POWERPC_CPU_HAS_ALTIVEC
> +	select BR2_POWERPC_CPU_HAS_VSX
> +	select BR2_ARCH_NEEDS_GCC_AT_LEAST_6
>   endchoice
>   
>   if BR2_POWERPC_CPU_HAS_SPE
> @@ -228,6 +234,7 @@ config BR2_GCC_TARGET_CPU
>   	default "power6"	if BR2_powerpc_power6
>   	default "power7"	if BR2_powerpc_power7
>   	default "power8"	if BR2_powerpc_power8
> +	default "power9"	if BR2_powerpc_power9
>   	default "powerpc64"	if BR2_generic_powerpc64
>   	default "powerpc"	if BR2_generic_powerpc
>
Yann E. MORIN July 29, 2022, 9:42 p.m. UTC | #2
Joel, All,

On 2022-07-29 09:39 +0930, Joel Stanley spake thusly:
> The Power9 CPU was released by IBM in 2016. It is a 64-bit CPU
> implementing ISA v3.0. It runs in big and little endian modes, and
> supports VSX and AltiVec.
> 
> Power9 is supported as of GCC 6.1 and Binutils 2.26.
> 
> Signed-off-by: Joel Stanley <joel@jms.id.au>

Reviewed-by: Yann E. MORIN <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  arch/Config.in.powerpc | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
> index 7c6ae84348e9..4c633c338d86 100644
> --- a/arch/Config.in.powerpc
> +++ b/arch/Config.in.powerpc
> @@ -149,6 +149,12 @@ config BR2_powerpc_power8
>  	depends on BR2_ARCH_IS_64
>  	select BR2_POWERPC_CPU_HAS_ALTIVEC
>  	select BR2_POWERPC_CPU_HAS_VSX
> +config BR2_powerpc_power9
> +	bool "power9"
> +	depends on BR2_ARCH_IS_64
> +	select BR2_POWERPC_CPU_HAS_ALTIVEC
> +	select BR2_POWERPC_CPU_HAS_VSX
> +	select BR2_ARCH_NEEDS_GCC_AT_LEAST_6
>  endchoice
>  
>  if BR2_POWERPC_CPU_HAS_SPE
> @@ -228,6 +234,7 @@ config BR2_GCC_TARGET_CPU
>  	default "power6"	if BR2_powerpc_power6
>  	default "power7"	if BR2_powerpc_power7
>  	default "power8"	if BR2_powerpc_power8
> +	default "power9"	if BR2_powerpc_power9
>  	default "powerpc64"	if BR2_generic_powerpc64
>  	default "powerpc"	if BR2_generic_powerpc
>  
> -- 
> 2.35.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Arnout Vandecappelle Sept. 18, 2022, 10:43 a.m. UTC | #3
On 29/07/2022 02:09, Joel Stanley wrote:
> The Power9 CPU was released by IBM in 2016. It is a 64-bit CPU
> implementing ISA v3.0. It runs in big and little endian modes, and
> supports VSX and AltiVec.
> 
> Power9 is supported as of GCC 6.1 and Binutils 2.26.
> 
> Signed-off-by: Joel Stanley <joel@jms.id.au>

  Applied to master, after removing the !64 condition, thanks.

  I didn't continue with the rest of the series - there are merge conflicts with 
the earlier changes I did. Can you please rebase and resend? And also take into 
account the comments on the first three patches of the series of course.

  Regards,
  Arnout

> ---
>   arch/Config.in.powerpc | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
> index 7c6ae84348e9..4c633c338d86 100644
> --- a/arch/Config.in.powerpc
> +++ b/arch/Config.in.powerpc
> @@ -149,6 +149,12 @@ config BR2_powerpc_power8
>   	depends on BR2_ARCH_IS_64
>   	select BR2_POWERPC_CPU_HAS_ALTIVEC
>   	select BR2_POWERPC_CPU_HAS_VSX
> +config BR2_powerpc_power9
> +	bool "power9"
> +	depends on BR2_ARCH_IS_64
> +	select BR2_POWERPC_CPU_HAS_ALTIVEC
> +	select BR2_POWERPC_CPU_HAS_VSX
> +	select BR2_ARCH_NEEDS_GCC_AT_LEAST_6
>   endchoice
>   
>   if BR2_POWERPC_CPU_HAS_SPE
> @@ -228,6 +234,7 @@ config BR2_GCC_TARGET_CPU
>   	default "power6"	if BR2_powerpc_power6
>   	default "power7"	if BR2_powerpc_power7
>   	default "power8"	if BR2_powerpc_power8
> +	default "power9"	if BR2_powerpc_power9
>   	default "powerpc64"	if BR2_generic_powerpc64
>   	default "powerpc"	if BR2_generic_powerpc
>
diff mbox series

Patch

diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
index 7c6ae84348e9..4c633c338d86 100644
--- a/arch/Config.in.powerpc
+++ b/arch/Config.in.powerpc
@@ -149,6 +149,12 @@  config BR2_powerpc_power8
 	depends on BR2_ARCH_IS_64
 	select BR2_POWERPC_CPU_HAS_ALTIVEC
 	select BR2_POWERPC_CPU_HAS_VSX
+config BR2_powerpc_power9
+	bool "power9"
+	depends on BR2_ARCH_IS_64
+	select BR2_POWERPC_CPU_HAS_ALTIVEC
+	select BR2_POWERPC_CPU_HAS_VSX
+	select BR2_ARCH_NEEDS_GCC_AT_LEAST_6
 endchoice
 
 if BR2_POWERPC_CPU_HAS_SPE
@@ -228,6 +234,7 @@  config BR2_GCC_TARGET_CPU
 	default "power6"	if BR2_powerpc_power6
 	default "power7"	if BR2_powerpc_power7
 	default "power8"	if BR2_powerpc_power8
+	default "power9"	if BR2_powerpc_power9
 	default "powerpc64"	if BR2_generic_powerpc64
 	default "powerpc"	if BR2_generic_powerpc