* [Buildroot] [PATCH] package/gnu-efi: disable on mips64el
@ 2023-09-30 10:23 Thomas Petazzoni via buildroot
2023-09-30 17:20 ` Yann E. MORIN
2023-10-13 6:21 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-09-30 10:23 UTC (permalink / raw)
To: Buildroot List; +Cc: Fabrice Fontaine, Thomas Petazzoni
The gnu-efi package was enabled on mips64el by commit
11b347c03ab0b38389ec11b48f63236a3b4d6547 ("package/gnu-efi: add
mips64el support"). However, it has been failing to build for a long
time, and nobody bothered fixing it:
gnu-efi-3.0.15//gnuefi/crt0-efi-mips64el.S:71: Error: cannot represent BFD_RELOC_16 relocation in this object file format
Even reverting back to gnu-efi 3.0.10, which was the version used at
the time of 11b347c03ab0b38389ec11b48f63236a3b4d6547, does not fix the
issue. We tested updating to the latest gnu-efi version, 3.0.17, and
the problem still exists.
Since EFI on MIPS is extremely niche, we don't really want to invest
the time to fix this issue, so let's disable it again. If someone
cares enough, it can be fixed and re-enabled.
Fixes:
http://autobuild.buildroot.net/results/d5938edcf50ebb7fdcec148d73f402845079779d/
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
package/gnu-efi/Config.in | 1 -
package/gnu-efi/gnu-efi.mk | 2 --
2 files changed, 3 deletions(-)
diff --git a/package/gnu-efi/Config.in b/package/gnu-efi/Config.in
index 8e35fddbcb..d456a23837 100644
--- a/package/gnu-efi/Config.in
+++ b/package/gnu-efi/Config.in
@@ -4,7 +4,6 @@ config BR2_PACKAGE_GNU_EFI_ARCH_SUPPORTS
default y if BR2_aarch64
default y if BR2_aarch64_be
default y if BR2_i386
- default y if BR2_mips64el
default y if BR2_x86_64
config BR2_PACKAGE_GNU_EFI
diff --git a/package/gnu-efi/gnu-efi.mk b/package/gnu-efi/gnu-efi.mk
index 93b5888370..686a14be01 100644
--- a/package/gnu-efi/gnu-efi.mk
+++ b/package/gnu-efi/gnu-efi.mk
@@ -24,8 +24,6 @@ else ifeq ($(BR2_arm)$(BR2_armeb),y)
GNU_EFI_PLATFORM = arm
else ifeq ($(BR2_aarch64)$(BR2_aarch64_be),y)
GNU_EFI_PLATFORM = aarch64
-else ifeq ($(BR2_mips64el),y)
-GNU_EFI_PLATFORM = mips64el
endif
GNU_EFI_MAKE_OPTS = \
--
2.41.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Buildroot] [PATCH] package/gnu-efi: disable on mips64el
2023-09-30 10:23 [Buildroot] [PATCH] package/gnu-efi: disable on mips64el Thomas Petazzoni via buildroot
@ 2023-09-30 17:20 ` Yann E. MORIN
2023-10-13 6:21 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Yann E. MORIN @ 2023-09-30 17:20 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: Fabrice Fontaine, Buildroot List
Thomas, All,
On 2023-09-30 12:23 +0200, Thomas Petazzoni via buildroot spake thusly:
> The gnu-efi package was enabled on mips64el by commit
> 11b347c03ab0b38389ec11b48f63236a3b4d6547 ("package/gnu-efi: add
> mips64el support"). However, it has been failing to build for a long
> time, and nobody bothered fixing it:
>
> gnu-efi-3.0.15//gnuefi/crt0-efi-mips64el.S:71: Error: cannot represent BFD_RELOC_16 relocation in this object file format
>
> Even reverting back to gnu-efi 3.0.10, which was the version used at
> the time of 11b347c03ab0b38389ec11b48f63236a3b4d6547, does not fix the
> issue. We tested updating to the latest gnu-efi version, 3.0.17, and
> the problem still exists.
>
> Since EFI on MIPS is extremely niche, we don't really want to invest
> the time to fix this issue, so let's disable it again. If someone
> cares enough, it can be fixed and re-enabled.
>
> Fixes:
>
> http://autobuild.buildroot.net/results/d5938edcf50ebb7fdcec148d73f402845079779d/
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Applied to master, thanks.
Regards,
Yann E. MORIN.
> ---
> package/gnu-efi/Config.in | 1 -
> package/gnu-efi/gnu-efi.mk | 2 --
> 2 files changed, 3 deletions(-)
>
> diff --git a/package/gnu-efi/Config.in b/package/gnu-efi/Config.in
> index 8e35fddbcb..d456a23837 100644
> --- a/package/gnu-efi/Config.in
> +++ b/package/gnu-efi/Config.in
> @@ -4,7 +4,6 @@ config BR2_PACKAGE_GNU_EFI_ARCH_SUPPORTS
> default y if BR2_aarch64
> default y if BR2_aarch64_be
> default y if BR2_i386
> - default y if BR2_mips64el
> default y if BR2_x86_64
>
> config BR2_PACKAGE_GNU_EFI
> diff --git a/package/gnu-efi/gnu-efi.mk b/package/gnu-efi/gnu-efi.mk
> index 93b5888370..686a14be01 100644
> --- a/package/gnu-efi/gnu-efi.mk
> +++ b/package/gnu-efi/gnu-efi.mk
> @@ -24,8 +24,6 @@ else ifeq ($(BR2_arm)$(BR2_armeb),y)
> GNU_EFI_PLATFORM = arm
> else ifeq ($(BR2_aarch64)$(BR2_aarch64_be),y)
> GNU_EFI_PLATFORM = aarch64
> -else ifeq ($(BR2_mips64el),y)
> -GNU_EFI_PLATFORM = mips64el
> endif
>
> GNU_EFI_MAKE_OPTS = \
> --
> 2.41.0
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/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. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Buildroot] [PATCH] package/gnu-efi: disable on mips64el
2023-09-30 10:23 [Buildroot] [PATCH] package/gnu-efi: disable on mips64el Thomas Petazzoni via buildroot
2023-09-30 17:20 ` Yann E. MORIN
@ 2023-10-13 6:21 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2023-10-13 6:21 UTC (permalink / raw)
To: Thomas Petazzoni via buildroot; +Cc: Fabrice Fontaine, Thomas Petazzoni
>>>>> "Thomas" == Thomas Petazzoni via buildroot <buildroot@buildroot.org> writes:
> The gnu-efi package was enabled on mips64el by commit
> 11b347c03ab0b38389ec11b48f63236a3b4d6547 ("package/gnu-efi: add
> mips64el support"). However, it has been failing to build for a long
> time, and nobody bothered fixing it:
> gnu-efi-3.0.15//gnuefi/crt0-efi-mips64el.S:71: Error: cannot represent BFD_RELOC_16 relocation in this object file format
> Even reverting back to gnu-efi 3.0.10, which was the version used at
> the time of 11b347c03ab0b38389ec11b48f63236a3b4d6547, does not fix the
> issue. We tested updating to the latest gnu-efi version, 3.0.17, and
> the problem still exists.
> Since EFI on MIPS is extremely niche, we don't really want to invest
> the time to fix this issue, so let's disable it again. If someone
> cares enough, it can be fixed and re-enabled.
> Fixes:
> http://autobuild.buildroot.net/results/d5938edcf50ebb7fdcec148d73f402845079779d/
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Committed to 2023.02.x and 2023.08.x, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-10-13 6:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-30 10:23 [Buildroot] [PATCH] package/gnu-efi: disable on mips64el Thomas Petazzoni via buildroot
2023-09-30 17:20 ` Yann E. MORIN
2023-10-13 6:21 ` Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox