On Mon, May 25, 2020 at 11:25 PM Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > > Convert DPRINTF() to trace events and remove ifdef'ry. > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > hw/display/dpcd.c | 16 +++------------- > hw/display/trace-events | 4 ++++ > 2 files changed, 7 insertions(+), 13 deletions(-) > > diff --git a/hw/display/dpcd.c b/hw/display/dpcd.c > index 0c1b7b35fb..64463654a1 100644 > --- a/hw/display/dpcd.c > +++ b/hw/display/dpcd.c > @@ -32,16 +32,7 @@ > #include "hw/misc/auxbus.h" > #include "migration/vmstate.h" > #include "hw/display/dpcd.h" > - > -#ifndef DEBUG_DPCD > -#define DEBUG_DPCD 0 > -#endif > - > -#define DPRINTF(fmt, ...) do { > \ > - if (DEBUG_DPCD) { > \ > - qemu_log("dpcd: " fmt, ## __VA_ARGS__); > \ > - } > \ > -} while (0) > +#include "trace.h" > > #define DPCD_READABLE_AREA 0x600 > > @@ -70,8 +61,8 @@ static uint64_t dpcd_read(void *opaque, hwaddr offset, > unsigned size) > offset); > ret = 0; > } > + trace_dpcd_read(offset, ret); > > - DPRINTF("read 0x%" PRIX8 " @0x%" HWADDR_PRIX "\n", ret, offset); > return ret; > } > > @@ -80,8 +71,7 @@ static void dpcd_write(void *opaque, hwaddr offset, > uint64_t value, > { > DPCDState *e = DPCD(opaque); > > - DPRINTF("write 0x%" PRIX8 " @0x%" HWADDR_PRIX "\n", (uint8_t)value, > offset); > - > + trace_dpcd_write(offset, value); > if (offset < DPCD_READABLE_AREA) { > e->dpcd_info[offset] = value; > } else { > diff --git a/hw/display/trace-events b/hw/display/trace-events > index bb089a5f5e..72d4c9812c 100644 > --- a/hw/display/trace-events > +++ b/hw/display/trace-events > @@ -157,3 +157,7 @@ artist_draw_line(unsigned int start_x, unsigned int > start_y, unsigned int end_x, > # cg3.c > cg3_read(uint32_t addr, uint32_t val, unsigned size) "read > addr:0x%06"PRIx32" val:0x%08"PRIx32" size:%u" > cg3_write(uint32_t addr, uint32_t val, unsigned size) "write > addr:0x%06"PRIx32" val:0x%08"PRIx32" size:%u" > + > +# dpcd.c > +dpcd_read(uint32_t addr, uint8_t val) "read addr:0x%"PRIx32" val:0x%02x" > +dpcd_write(uint32_t addr, uint8_t val) "write addr:0x%"PRIx32" val:0x%02x" > -- > 2.21.3 > >