On Mon, May 25, 2020 at 11:25 PM Philippe Mathieu-Daudé <[email protected]> wrote: > > Replace some debug printf() calls by qemu_log_mask(LOG_UNIMP), > and add a new one in cirrus_linear_bitblt_read(). > > Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Alistair Francis <[email protected]> Alistair > --- > hw/display/cirrus_vga.c | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c > index 33ccdde000..f9f837b850 100644 > --- a/hw/display/cirrus_vga.c > +++ b/hw/display/cirrus_vga.c > @@ -35,6 +35,7 @@ > #include "qemu/osdep.h" > #include "qemu/module.h" > #include "qemu/units.h" > +#include "qemu/log.h" > #include "sysemu/reset.h" > #include "qapi/error.h" > #include "trace.h" > @@ -905,9 +906,8 @@ static int cirrus_bitblt_cputovideo(CirrusVGAState * s) > static int cirrus_bitblt_videotocpu(CirrusVGAState * s) > { > /* XXX */ > -#ifdef DEBUG_BITBLT > - printf("cirrus: bitblt (video to cpu) is not implemented yet\n"); > -#endif > + qemu_log_mask(LOG_UNIMP, > + "cirrus: bitblt (video to cpu) is not implemented\n"); > return 0; > } > > @@ -989,9 +989,8 @@ static void cirrus_bitblt_start(CirrusVGAState * s) > cirrus_blt_mode & (CIRRUS_BLTMODE_MEMSYSSRC | > CIRRUS_BLTMODE_MEMSYSDEST)) > == (CIRRUS_BLTMODE_MEMSYSSRC | CIRRUS_BLTMODE_MEMSYSDEST)) { > -#ifdef DEBUG_BITBLT > - printf("cirrus: bitblt - memory-to-memory copy is requested\n"); > -#endif > + qemu_log_mask(LOG_UNIMP, > + "cirrus: bitblt - memory-to-memory copy requested\n"); > goto bitblt_ignore; > } > > @@ -2412,6 +2411,9 @@ static uint64_t cirrus_linear_bitblt_read(void *opaque, > > /* XXX handle bitblt */ > (void)s; > + qemu_log_mask(LOG_UNIMP, > + "cirrus: linear bitblt is not implemented\n"); > + > return 0xff; > } > > -- > 2.21.3 > >
