Re: [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello,

Let's add some PowerPC folks into the loop. Joel, Cédric, could you
have a look at the below patch, and how to handle VMX support in the
pixman package?

It feels a bit unfortunate to unconditionally disable VMX support:
there is some logic in the pixman configure script to detect the
availability of VMX, but apparently this logic assumes it works on
powerpc64le, even though at build time it breaks.

Could you have a look and give us your advice?

Thanks a lot!

Thomas Petazzoni

On Fri,  8 Jul 2022 16:21:50 +0200
Bernd Kuhls <bernd.kuhls@t-online.de> wrote:

> Disable vmx support to fix build error:
> 
> pixman-vmx.c:268:5: note: in expansion of macro 'LOAD_VECTOR'
>   268 |     LOAD_VECTOR (src);
>       |     ^~~~~~~~~~~
> pixman-vmx.c:268:5: error: cannot convert a value of type 'int' to
>  vector type '__vector(4) unsigned int' which has different size
> 
> Fixes:
> http://autobuild.buildroot.net/results/fda886768fce25ccd9b52b635ff5b13da7ba2d0c/
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/pixman/pixman.mk | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/package/pixman/pixman.mk b/package/pixman/pixman.mk
> index 64302fa49a..0f00a20cd8 100644
> --- a/package/pixman/pixman.mk
> +++ b/package/pixman/pixman.mk
> @@ -21,7 +21,8 @@ PIXMAN_AUTORECONF = YES
>  # don't build gtk based demos
>  PIXMAN_CONF_OPTS = \
>  	--disable-gtk \
> -	--disable-loongson-mmi
> +	--disable-loongson-mmi \
> +	--disable-vmx
>  
>  # The ARM SIMD code from pixman requires a recent enough ARM core, but
>  # there is a runtime CPU check that makes sure it doesn't get used if



-- 
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