[Buildroot] [PATCH] arch/Config.in.powerpc: Drop PPC601 support

[Buildroot] [PATCH] arch/Config.in.powerpc: Drop PPC601 support

[Michael Walle]

Linux support was removed in 5.10 [1]. Since no in-tree defconfig
depends on it, just remove it.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/powerpc?id=f0ed73f3fa2cdca65973659689ec9e46d99a5f60

Reported-by: Romain Naour <romain.naour@gmail.com>
Signed-off-by: Michael Walle <michael@walle.cc>
---
 Config.in.legacy       | 6 ++++++
 arch/Config.in.powerpc | 4 ----
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/Config.in.legacy b/Config.in.legacy
index 6d8acadf9c..0b963cba3a 100644
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -160,6 +160,12 @@ config BR2_PACKAGE_IPSEC_TOOLS
 	  This package has been removed as it has security issues and
 	  has been abandoned since 2014.
 
+config BR2_powerpc_601
+	bool "PowerPC 601 support removed"
+	select BR2_LEGACY
+	help
+	  The support for the PowerPC 601 processors has been removed.
+
 comment "Legacy options removed in 2020.11"
 
 config BR2_PACKAGE_OPENCV
diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
index 56c4b526ec..ba56c9c721 100644
--- a/arch/Config.in.powerpc
+++ b/arch/Config.in.powerpc
@@ -44,9 +44,6 @@ config BR2_powerpc_476fp
 config BR2_powerpc_505
 	bool "505"
 	depends on !BR2_ARCH_IS_64
-config BR2_powerpc_601
-	bool "601"
-	depends on !BR2_ARCH_IS_64
 config BR2_powerpc_602
 	bool "602"
 	depends on !BR2_ARCH_IS_64
@@ -176,7 +173,6 @@ config BR2_GCC_TARGET_CPU
 	default "476"		if BR2_powerpc_476
 	default "476fp"		if BR2_powerpc_476fp
 	default "505"		if BR2_powerpc_505
-	default "601"		if BR2_powerpc_601
 	default "602"		if BR2_powerpc_602
 	default "603"		if BR2_powerpc_603
 	default "603e"		if BR2_powerpc_603e
-- 
2.20.1

[Buildroot] [PATCH] arch/Config.in.powerpc: Drop PPC601 support

[Romain Naour]

Hello Michael,

Le 15/12/2020 ? 09:14, Michael Walle a ?crit?:
> Linux support was removed in 5.10 [1]. Since no in-tree defconfig
> depends on it, just remove it.

Thanks for the patch.

I'm ok with this patch but I would like a feedback from maintainers because
PPC601 removal from the latest kernel doesn't mean that we have to remove it
from Buildroot immediately.
OTOH who is still using PPC601 in 2020 with the latest Buildroot ?
Also, since there was no kernel in-tree defconfig for this target I'm not sure
it worth the effort.

Best regards,
Romain

> 
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/powerpc?id=f0ed73f3fa2cdca65973659689ec9e46d99a5f60
> 
> Reported-by: Romain Naour <romain.naour@gmail.com>
> Signed-off-by: Michael Walle <michael@walle.cc>
> ---
>  Config.in.legacy       | 6 ++++++
>  arch/Config.in.powerpc | 4 ----
>  2 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 6d8acadf9c..0b963cba3a 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -160,6 +160,12 @@ config BR2_PACKAGE_IPSEC_TOOLS
>  	  This package has been removed as it has security issues and
>  	  has been abandoned since 2014.
>  
> +config BR2_powerpc_601
> +	bool "PowerPC 601 support removed"
> +	select BR2_LEGACY
> +	help
> +	  The support for the PowerPC 601 processors has been removed.
> +
>  comment "Legacy options removed in 2020.11"
>  
>  config BR2_PACKAGE_OPENCV
> diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
> index 56c4b526ec..ba56c9c721 100644
> --- a/arch/Config.in.powerpc
> +++ b/arch/Config.in.powerpc
> @@ -44,9 +44,6 @@ config BR2_powerpc_476fp
>  config BR2_powerpc_505
>  	bool "505"
>  	depends on !BR2_ARCH_IS_64
> -config BR2_powerpc_601
> -	bool "601"
> -	depends on !BR2_ARCH_IS_64
>  config BR2_powerpc_602
>  	bool "602"
>  	depends on !BR2_ARCH_IS_64
> @@ -176,7 +173,6 @@ config BR2_GCC_TARGET_CPU
>  	default "476"		if BR2_powerpc_476
>  	default "476fp"		if BR2_powerpc_476fp
>  	default "505"		if BR2_powerpc_505
> -	default "601"		if BR2_powerpc_601
>  	default "602"		if BR2_powerpc_602
>  	default "603"		if BR2_powerpc_603
>  	default "603e"		if BR2_powerpc_603e
> 

[Buildroot] [PATCH] arch/Config.in.powerpc: Drop PPC601 support

[Michael Walle]

Am 2020-12-15 11:06, schrieb Romain Naour:
> Hello Michael,
> 
> Le 15/12/2020 ? 09:14, Michael Walle a ?crit?:
>> Linux support was removed in 5.10 [1]. Since no in-tree defconfig
>> depends on it, just remove it.
> 
> Thanks for the patch.
> 
> I'm ok with this patch but I would like a feedback from maintainers 
> because
> PPC601 removal from the latest kernel doesn't mean that we have to 
> remove it
> from Buildroot immediately.
> OTOH who is still using PPC601 in 2020 with the latest Buildroot ?
> Also, since there was no kernel in-tree defconfig for this target I'm 
> not sure
> it worth the effort.

So the other possibility would be to make 5.9 the last kernel for ppc601
and 5.10 for ppc40x. But that would mean we also have to keep the 5.9 
one
indefinitely  right?

-michael

[Buildroot] [PATCH] arch/Config.in.powerpc: Drop PPC601 support

[Romain Naour]

Le 15/12/2020 ? 11:15, Michael Walle a ?crit?:
> Am 2020-12-15 11:06, schrieb Romain Naour:
>> Hello Michael,
>>
>> Le 15/12/2020 ? 09:14, Michael Walle a ?crit?:
>>> Linux support was removed in 5.10 [1]. Since no in-tree defconfig
>>> depends on it, just remove it.
>>
>> Thanks for the patch.
>>
>> I'm ok with this patch but I would like a feedback from maintainers because
>> PPC601 removal from the latest kernel doesn't mean that we have to remove it
>> from Buildroot immediately.
>> OTOH who is still using PPC601 in 2020 with the latest Buildroot ?
>> Also, since there was no kernel in-tree defconfig for this target I'm not sure
>> it worth the effort.
> 
> So the other possibility would be to make 5.9 the last kernel for ppc601
> and 5.10 for ppc40x. But that would mean we also have to keep the 5.9 one
> indefinitely? right?

Buildroot doesn't have to keep indefinitely a support for an unmaintained cpu
architecture or package.
At some point it's removed (see bfin, avr32 removal etc...).
Except if there is someone that really want to keep it and help us to take care
of this support.

There is a similar issue with ppc w/ SPE that was deprecated in gcc 8 and
removed in gcc 9:

https://git.buildroot.net/buildroot/commit/?id=96e80ad214ced86c802d561946fa7da88c1595eb

Should we keep indefinitely gcc 8 for ppc w/ spe ?
Yes and no because we know that ArcturusNetworks people (maybe other) are still
using ppc w/ spe and they have sent patches for arcturus_ucp1020_defconfig.
But gcc-8 is not maintained anymore.

Best regards,
Romain

> 
> -michael

[Buildroot] [PATCH] arch/Config.in.powerpc: Drop PPC601 support

[Yann E. MORIN]

Michael, All,

On 2020-12-15 09:14 +0100, Michael Walle spake thusly:
> Linux support was removed in 5.10 [1]. Since no in-tree defconfig
> depends on it, just remove it.
> 
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/powerpc?id=f0ed73f3fa2cdca65973659689ec9e46d99a5f60
> 
> Reported-by: Romain Naour <romain.naour@gmail.com>
> Signed-off-by: Michael Walle <michael@walle.cc>

Applied to master, thanks.

I've just re-ordered the legacy entry: we usually add the new legacy
entries at the to of the list.

Thanks!

Regards,
Yann E. MORIN.

> ---
>  Config.in.legacy       | 6 ++++++
>  arch/Config.in.powerpc | 4 ----
>  2 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 6d8acadf9c..0b963cba3a 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -160,6 +160,12 @@ config BR2_PACKAGE_IPSEC_TOOLS
>  	  This package has been removed as it has security issues and
>  	  has been abandoned since 2014.
>  
> +config BR2_powerpc_601
> +	bool "PowerPC 601 support removed"
> +	select BR2_LEGACY
> +	help
> +	  The support for the PowerPC 601 processors has been removed.
> +
>  comment "Legacy options removed in 2020.11"
>  
>  config BR2_PACKAGE_OPENCV
> diff --git a/arch/Config.in.powerpc b/arch/Config.in.powerpc
> index 56c4b526ec..ba56c9c721 100644
> --- a/arch/Config.in.powerpc
> +++ b/arch/Config.in.powerpc
> @@ -44,9 +44,6 @@ config BR2_powerpc_476fp
>  config BR2_powerpc_505
>  	bool "505"
>  	depends on !BR2_ARCH_IS_64
> -config BR2_powerpc_601
> -	bool "601"
> -	depends on !BR2_ARCH_IS_64
>  config BR2_powerpc_602
>  	bool "602"
>  	depends on !BR2_ARCH_IS_64
> @@ -176,7 +173,6 @@ config BR2_GCC_TARGET_CPU
>  	default "476"		if BR2_powerpc_476
>  	default "476fp"		if BR2_powerpc_476fp
>  	default "505"		if BR2_powerpc_505
> -	default "601"		if BR2_powerpc_601
>  	default "602"		if BR2_powerpc_602
>  	default "603"		if BR2_powerpc_603
>  	default "603e"		if BR2_powerpc_603e
> -- 
> 2.20.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'