Thanks and sorry for missing this in the original commit. Acked-by: Dongwon Kim <[email protected]>
> -----Original Message----- > From: Philippe Mathieu-Daudé <[email protected]> > Sent: Friday, May 17, 2024 8:02 AM > To: [email protected] > Cc: Cédric Le Goater <[email protected]>; Kim, Dongwon > <[email protected]>; Marc-André Lureau > <[email protected]>; Philippe Mathieu-Daudé > <[email protected]> > Subject: [PULL 1/5] ui/console: Only declare variable fence_fd when > CONFIG_GBM is defined > > From: Cédric Le Goater <[email protected]> > > This to avoid a build breakage : > > ../ui/gtk-egl.c: In function ‘gd_egl_draw’: > ../ui/gtk-egl.c:73:9: error: unused variable ‘fence_fd’ > [-Werror=unused-variable] > 73 | int fence_fd; > | ^~~~~~~~ > > Fixes: fa6426805b12 ("ui/console: Use qemu_dmabuf_set_..() helpers instead") > Cc: Dongwon Kim <[email protected]> > Cc: Marc-André Lureau <[email protected]> > Signed-off-by: Cédric Le Goater <[email protected]> > Reviewed-by: Philippe Mathieu-Daudé <[email protected]> > Message-ID: <[email protected]> > Signed-off-by: Philippe Mathieu-Daudé <[email protected]> > --- > ui/gtk-egl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c index 0473f689c9..9831c10e1b 100644 > --- a/ui/gtk-egl.c > +++ b/ui/gtk-egl.c > @@ -68,9 +68,9 @@ void gd_egl_draw(VirtualConsole *vc) > GdkWindow *window; > #ifdef CONFIG_GBM > QemuDmaBuf *dmabuf = vc->gfx.guest_fb.dmabuf; > + int fence_fd; > #endif > int ww, wh, ws; > - int fence_fd; > > if (!vc->gfx.gls) { > return; > -- > 2.41.0
