All of 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.