On Wed, Feb 3, 2021 at 7:48 PM Gerd Hoffmann <kra...@redhat.com> wrote:
> > +static int
> > +virtio_gpu_get_flags(void *opaque)
> > +{
> > + VirtIOGPUBase *g = opaque;
> > + int flags = GRAPHIC_FLAGS_NONE;
> > +
> > + if (virtio_gpu_virgl_enabled(g->conf))
> > + flags |= GRAPHIC_FLAGS_GL;
> > +
> > + if (virtio_gpu_dmabuf_enabled(g->conf))
> > + flags |= GRAPHIC_FLAGS_DMABUF;
>
> fbe6ba76ac01 ui: add an optional get_flags callback to GraphicHwOps
> ERROR: braces {} are necessary for all arms of this statement
> #50: FILE: hw/display/virtio-gpu-base.c:123:
> + if (virtio_gpu_virgl_enabled(g->conf))
> [...]
>
> ERROR: braces {} are necessary for all arms of this statement
> #53: FILE: hw/display/virtio-gpu-base.c:126:
> + if (virtio_gpu_dmabuf_enabled(g->conf))
> [...]
>
> total: 2 errors, 0 warnings, 68 lines checked
>
If you queued the series, do you mind squashing a style fix? Otherwise I
can resend.
Thanks
