[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-trivial] [PATCH] virtio-gpu: replace PIXMAN_* by
From: |
Sahid Orentino Ferdjaoui |
Subject: |
Re: [Qemu-devel] [Qemu-trivial] [PATCH] virtio-gpu: replace PIXMAN_* by PIXMAN_BE_* |
Date: |
Thu, 6 Apr 2017 12:43:36 +0200 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Mon, Apr 03, 2017 at 09:28:02AM -0300, Philippe Mathieu-Daudé wrote:
> On 04/03/2017 08:40 AM, Laurent Vivier wrote:
> > This avoids a "#ifdef HOST_WORDS_BIGENDIAN" and this is the purpose
> > of PIXMAN_BE_* macros.
> >
> > Signed-off-by: Laurent Vivier <address@hidden>
>
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
>
Reviewed-by: Sahid Orentino Ferdjaoui <address@hidden>
>
> > ---
> > hw/display/virtio-gpu.c | 35 ++++++++---------------------------
> > 1 file changed, 8 insertions(+), 27 deletions(-)
> >
> > diff --git a/hw/display/virtio-gpu.c b/hw/display/virtio-gpu.c
> > index 9b530ab..7befcef 100644
> > --- a/hw/display/virtio-gpu.c
> > +++ b/hw/display/virtio-gpu.c
> > @@ -258,41 +258,22 @@ void virtio_gpu_get_display_info(VirtIOGPU *g,
> > static pixman_format_code_t get_pixman_format(uint32_t virtio_gpu_format)
> > {
> > switch (virtio_gpu_format) {
> > -#ifdef HOST_WORDS_BIGENDIAN
> > case VIRTIO_GPU_FORMAT_B8G8R8X8_UNORM:
> > - return PIXMAN_b8g8r8x8;
> > + return PIXMAN_BE_b8g8r8x8;
> > case VIRTIO_GPU_FORMAT_B8G8R8A8_UNORM:
> > - return PIXMAN_b8g8r8a8;
> > + return PIXMAN_BE_b8g8r8a8;
> > case VIRTIO_GPU_FORMAT_X8R8G8B8_UNORM:
> > - return PIXMAN_x8r8g8b8;
> > + return PIXMAN_BE_x8r8g8b8;
> > case VIRTIO_GPU_FORMAT_A8R8G8B8_UNORM:
> > - return PIXMAN_a8r8g8b8;
> > + return PIXMAN_BE_a8r8g8b8;
> > case VIRTIO_GPU_FORMAT_R8G8B8X8_UNORM:
> > - return PIXMAN_r8g8b8x8;
> > + return PIXMAN_BE_r8g8b8x8;
> > case VIRTIO_GPU_FORMAT_R8G8B8A8_UNORM:
> > - return PIXMAN_r8g8b8a8;
> > + return PIXMAN_BE_r8g8b8a8;
> > case VIRTIO_GPU_FORMAT_X8B8G8R8_UNORM:
> > - return PIXMAN_x8b8g8r8;
> > + return PIXMAN_BE_x8b8g8r8;
> > case VIRTIO_GPU_FORMAT_A8B8G8R8_UNORM:
> > - return PIXMAN_a8b8g8r8;
> > -#else
> > - case VIRTIO_GPU_FORMAT_B8G8R8X8_UNORM:
> > - return PIXMAN_x8r8g8b8;
> > - case VIRTIO_GPU_FORMAT_B8G8R8A8_UNORM:
> > - return PIXMAN_a8r8g8b8;
> > - case VIRTIO_GPU_FORMAT_X8R8G8B8_UNORM:
> > - return PIXMAN_b8g8r8x8;
> > - case VIRTIO_GPU_FORMAT_A8R8G8B8_UNORM:
> > - return PIXMAN_b8g8r8a8;
> > - case VIRTIO_GPU_FORMAT_R8G8B8X8_UNORM:
> > - return PIXMAN_x8b8g8r8;
> > - case VIRTIO_GPU_FORMAT_R8G8B8A8_UNORM:
> > - return PIXMAN_a8b8g8r8;
> > - case VIRTIO_GPU_FORMAT_X8B8G8R8_UNORM:
> > - return PIXMAN_r8g8b8x8;
> > - case VIRTIO_GPU_FORMAT_A8B8G8R8_UNORM:
> > - return PIXMAN_r8g8b8a8;
> > -#endif
> > + return PIXMAN_BE_a8b8g8r8;
> > default:
> > return 0;
> > }
> >