Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] configs/qemu_ppc64*: remove binutils version from defconfigs
@ 2023-08-03 21:38 Romain Naour
  2023-08-04 20:19 ` Thomas Petazzoni via buildroot
  0 siblings, 1 reply; 2+ messages in thread
From: Romain Naour @ 2023-08-03 21:38 UTC (permalink / raw)
  To: buildroot; +Cc: Romain Naour, Cédric Le Goater, Joel Stanley

Buildroot 2022.05 use binutils 2.37 by default, but the binutils
version was downgraded to the previous binutils version in qemu_ppc64*
defconfigs due to a bug in binutils 2.37 [1].

Later when binutils 2.36 has been removed the binutils version has
been updated to 2.38 (even though it was already the default version
selected by Buildroot at that time) [2].

Since then, several binutils release has been added and the binutils
version 2.38 has been removed recently [3].

Since the initial bug is gone with the removal of binutils 2.37,
we can safely remove the binutils version from qemu_ppc64 defconfigs.

[1] 1e2fe860f3a261fc877591c056808d6fb30e25d4
[2] e461c9adc82a889deb6e259ce811193cb8d00f36
[3] 1391c99d627d27c55bec99072b3ae4d78be02c74

Fixes:
https://gitlab.com/buildroot.org/toolchains-builder/-/jobs/4798047373

Signed-off-by: Romain Naour <romain.naour@gmail.com>
Cc: Cédric Le Goater <clg@kaod.org>
Cc: Joel Stanley <joel@jms.id.au>
---
 configs/qemu_ppc64_e5500_defconfig      | 1 -
 configs/qemu_ppc64_pseries_defconfig    | 1 -
 configs/qemu_ppc64le_powernv8_defconfig | 1 -
 configs/qemu_ppc64le_pseries_defconfig  | 1 -
 4 files changed, 4 deletions(-)

diff --git a/configs/qemu_ppc64_e5500_defconfig b/configs/qemu_ppc64_e5500_defconfig
index 0ab7e8e76f..6839f2f0ba 100644
--- a/configs/qemu_ppc64_e5500_defconfig
+++ b/configs/qemu_ppc64_e5500_defconfig
@@ -3,7 +3,6 @@ BR2_powerpc64=y
 BR2_powerpc_e5500=y
 
 # System
-BR2_BINUTILS_VERSION_2_38_X=y
 BR2_SYSTEM_DHCP="eth0"
 BR2_TARGET_GENERIC_GETTY_PORT="ttyS0"
 
diff --git a/configs/qemu_ppc64_pseries_defconfig b/configs/qemu_ppc64_pseries_defconfig
index 92fe3a11f6..896d209675 100644
--- a/configs/qemu_ppc64_pseries_defconfig
+++ b/configs/qemu_ppc64_pseries_defconfig
@@ -3,7 +3,6 @@ BR2_powerpc64=y
 BR2_powerpc_power7=y
 
 # System
-BR2_BINUTILS_VERSION_2_38_X=y
 BR2_SYSTEM_DHCP="eth0"
 BR2_TARGET_GENERIC_GETTY_PORT="hvc0"
 
diff --git a/configs/qemu_ppc64le_powernv8_defconfig b/configs/qemu_ppc64le_powernv8_defconfig
index cb086ba248..e23fcd4695 100644
--- a/configs/qemu_ppc64le_powernv8_defconfig
+++ b/configs/qemu_ppc64le_powernv8_defconfig
@@ -3,7 +3,6 @@ BR2_powerpc64le=y
 BR2_powerpc_power8=y
 
 # System
-BR2_BINUTILS_VERSION_2_38_X=y
 BR2_TARGET_GENERIC_GETTY_PORT="hvc0"
 BR2_SYSTEM_DHCP="eth0"
 
diff --git a/configs/qemu_ppc64le_pseries_defconfig b/configs/qemu_ppc64le_pseries_defconfig
index 001d614c3b..fa8e2a09d3 100644
--- a/configs/qemu_ppc64le_pseries_defconfig
+++ b/configs/qemu_ppc64le_pseries_defconfig
@@ -3,7 +3,6 @@ BR2_powerpc64le=y
 BR2_powerpc_power8=y
 
 # System
-BR2_BINUTILS_VERSION_2_38_X=y
 BR2_TARGET_GENERIC_GETTY_PORT="hvc0"
 BR2_SYSTEM_DHCP="eth0"
 
-- 
2.41.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Buildroot] [PATCH] configs/qemu_ppc64*: remove binutils version from defconfigs
  2023-08-03 21:38 [Buildroot] [PATCH] configs/qemu_ppc64*: remove binutils version from defconfigs Romain Naour
@ 2023-08-04 20:19 ` Thomas Petazzoni via buildroot
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-08-04 20:19 UTC (permalink / raw)
  To: Romain Naour; +Cc: Joel Stanley, Cédric Le Goater, buildroot

On Thu,  3 Aug 2023 23:38:14 +0200
Romain Naour <romain.naour@gmail.com> wrote:

> Buildroot 2022.05 use binutils 2.37 by default, but the binutils
> version was downgraded to the previous binutils version in qemu_ppc64*
> defconfigs due to a bug in binutils 2.37 [1].
> 
> Later when binutils 2.36 has been removed the binutils version has
> been updated to 2.38 (even though it was already the default version
> selected by Buildroot at that time) [2].
> 
> Since then, several binutils release has been added and the binutils
> version 2.38 has been removed recently [3].
> 
> Since the initial bug is gone with the removal of binutils 2.37,
> we can safely remove the binutils version from qemu_ppc64 defconfigs.
> 
> [1] 1e2fe860f3a261fc877591c056808d6fb30e25d4
> [2] e461c9adc82a889deb6e259ce811193cb8d00f36
> [3] 1391c99d627d27c55bec99072b3ae4d78be02c74
> 
> Fixes:
> https://gitlab.com/buildroot.org/toolchains-builder/-/jobs/4798047373
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> Cc: Cédric Le Goater <clg@kaod.org>
> Cc: Joel Stanley <joel@jms.id.au>
> ---
>  configs/qemu_ppc64_e5500_defconfig      | 1 -
>  configs/qemu_ppc64_pseries_defconfig    | 1 -
>  configs/qemu_ppc64le_powernv8_defconfig | 1 -
>  configs/qemu_ppc64le_pseries_defconfig  | 1 -
>  4 files changed, 4 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-08-04 20:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-03 21:38 [Buildroot] [PATCH] configs/qemu_ppc64*: remove binutils version from defconfigs Romain Naour
2023-08-04 20:19 ` Thomas Petazzoni via buildroot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox