[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 08/14] hw/display/dpcd: Convert debug printf()s to trace events
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH 08/14] hw/display/dpcd: Convert debug printf()s to trace events |
Date: |
Tue, 26 May 2020 08:22:46 +0200 |
Convert DPRINTF() to trace events and remove ifdef'ry.
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
---
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
- [PATCH 03/14] hw/display/cirrus_vga: Convert debug printf() to trace event, (continued)
- [PATCH 03/14] hw/display/cirrus_vga: Convert debug printf() to trace event, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 04/14] hw/display/cirrus_vga: Use qemu_log_mask(UNIMP) instead of debug printf, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 06/14] hw/display/cirrus_vga: Convert debug printf() to trace event, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 07/14] hw/display/dpcd: Fix memory region size, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 05/14] hw/display/cirrus_vga: Use qemu_log_mask(ERROR) instead of debug printf, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 08/14] hw/display/dpcd: Convert debug printf()s to trace events,
Philippe Mathieu-Daudé <=
- [PATCH 09/14] hw/display/xlnx_dp: Replace disabled DPRINTF() by error_report(), Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 10/14] hw/display/vmware_vga: Replace printf() calls by qemu_log_mask(ERROR), Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 11/14] hw/display/vmware_vga: Let the PCI device own its I/O MemoryRegion, Philippe Mathieu-Daudé, 2020/05/26