All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/igt-gpu-tools: depends on little endian
@ 2026-01-02 23:40 Bernd Kuhls
  2026-02-03  8:19 ` Thomas Petazzoni via buildroot
  0 siblings, 1 reply; 2+ messages in thread
From: Bernd Kuhls @ 2026-01-02 23:40 UTC (permalink / raw)
  To: buildroot

Buildroot commit f9a91bf60e35617157b950eea05fc6f6144d650c bumped igt-gpu-
tools to 2.3 including upstream commit
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/commit/b542242f5b116e3b554b4068ef5dfa4451075b2b
which limits the package to little endian archs causing build errors on
big endian archs:

In file included from ../lib/igt_panthor.c:5:
../lib/igt_panthor.h:61:2: error: #error "big endian not supported"
   61 | #error "big endian not supported"

Fixes:
https://autobuild.buildroot.org/results/e3faeba1bff3d0e781032ddc9a760f34a9bf5714/

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
 package/igt-gpu-tools/Config.in | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/package/igt-gpu-tools/Config.in b/package/igt-gpu-tools/Config.in
index 184acb4cf1..87268969a1 100644
--- a/package/igt-gpu-tools/Config.in
+++ b/package/igt-gpu-tools/Config.in
@@ -1,5 +1,10 @@
+config BR2_PACKAGE_IGT_GPU_TOOLS_ARCH_SUPPORTS
+	bool
+	default y if BR2_ENDIAN = "LITTLE"
+
 config BR2_PACKAGE_IGT_GPU_TOOLS
 	bool "igt-gpu-tools"
+	depends on BR2_PACKAGE_IGT_GPU_TOOLS_ARCH_SUPPORTS
 	depends on BR2_USE_MMU # fork()
 	depends on BR2_ENABLE_LOCALE
 	depends on !BR2_STATIC_LIBS
@@ -35,6 +40,7 @@ config BR2_PACKAGE_IGT_GPU_TOOLS_TESTS
 endif
 
 comment "igt-gpu-tools needs udev /dev management and toolchain w/ NPTL, wchar, dynamic library, locale, headers >= 4.11"
+	depends on BR2_PACKAGE_IGT_GPU_TOOLS_ARCH_SUPPORTS
 	depends on BR2_USE_MMU
 	depends on !BR2_PACKAGE_HAS_UDEV || BR2_STATIC_LIBS || \
 		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_USE_WCHAR || \
-- 
2.47.3

_______________________________________________
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 1/1] package/igt-gpu-tools: depends on little endian
  2026-01-02 23:40 [Buildroot] [PATCH 1/1] package/igt-gpu-tools: depends on little endian Bernd Kuhls
@ 2026-02-03  8:19 ` Thomas Petazzoni via buildroot
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni via buildroot @ 2026-02-03  8:19 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: buildroot

Hello Bernd,

Thanks for the patch!

On Sat, Jan 03, 2026 at 12:40:05AM +0100, Bernd Kuhls wrote:
> Buildroot commit f9a91bf60e35617157b950eea05fc6f6144d650c bumped igt-gpu-
> tools to 2.3 including upstream commit
> https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/commit/b542242f5b116e3b554b4068ef5dfa4451075b2b

This wasn't the commit introducing the problematic change, so I've
fixed that up with the correct commit, and applied.

However, I've also filed an issue an upstream, because it looks quite
intentional that this change adding Panthor tests prevents the whole
project from building on big endian architectures:

  https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/issues/200

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
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:[~2026-02-03  8:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-02 23:40 [Buildroot] [PATCH 1/1] package/igt-gpu-tools: depends on little endian Bernd Kuhls
2026-02-03  8: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.