[Mesa-dev] [PATCH v4 0/4] Fix OpenGL 1.3 big-endian support in r600g

[Oded Gabbay]

On Tue, Apr 26, 2016 at 1:26 PM, Andy Furniss <adf.lists at gmail.com> wrote:
> Oded Gabbay wrote:
>
>> Oded Gabbay (4): r600g/radeonsi: send endian info to format
>> translation functions r600g: set endianess of 16/32-bit buffers
>> according to do_endian_swap r600g: use do_endian_swap in color
>> swapping functions r600g: use do_endian_swap in texture swapping
>> function
>
>
> I get a build fail on radeonsi since these were committed.
>
> ./autogen.sh --prefix=/usr --sysconfdir=/etc --enable-texture-float
> --enable-opencl --enable-omx --with-egl-platforms=x11,drm,wayland
> --with-gallium-drivers=radeonsi,swrast --enable-gbm
> --enable-shared-glapi --enable-glx-tls --with-dri-drivers= && make -j5
>
>
>   CC       si_state.lo
> si_state.c: In function ‘si_make_texture_descriptor’:
> si_state.c:2952:19: error: too few arguments to function
> ‘r600_translate_colorswap’
>    unsigned swap = r600_translate_colorswap(pipe_format);
>                    ^
> In file included from si_state.h:31:0,
>                  from si_pipe.h:29,
>                  from si_state.c:27:
> ../../../../src/gallium/drivers/radeon/r600_pipe_common.h:638:10: note:
> declared here
>  unsigned r600_translate_colorswap(enum pipe_format format, bool
> do_endian_swap);
>           ^
> Makefile:702: recipe for target 'si_state.lo' failed
> make[4]: *** [si_state.lo] Error 1
>

Thanks for spotting that.
I've pushed a fix now. With that, it builds radeonsi without errors.

Oded