On Thu, Feb 04, 2021 at 02:21:02AM +0400, Marc-André Lureau wrote:
> 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.
Whatever is easier for you (note there are more codestyle warnings in
following patches, see patchew report).
take care,
Gerd
