[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 4/4] hw/sensor/tmp105: Convert printf() to trace event, add tr
From: |
Bernhard Beschow |
Subject: |
[PATCH v2 4/4] hw/sensor/tmp105: Convert printf() to trace event, add tracing for read/write access |
Date: |
Sun, 3 Nov 2024 15:33:30 +0100 |
printf() unconditionally prints to the console which disturbs `-serial stdio`.
Fix that by converting into a trace event. While at it, add some tracing for
read and write access.
Fixes: 7e7c5e4c1ba5 "Nokia N800 machine support (ARM)."
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
meson.build | 1 +
hw/sensor/trace.h | 1 +
hw/sensor/tmp105.c | 7 ++++++-
hw/sensor/trace-events | 6 ++++++
4 files changed, 14 insertions(+), 1 deletion(-)
create mode 100644 hw/sensor/trace.h
create mode 100644 hw/sensor/trace-events
diff --git a/meson.build b/meson.build
index 2c9086a3fe..19df8d7125 100644
--- a/meson.build
+++ b/meson.build
@@ -3476,6 +3476,7 @@ if have_system
'hw/s390x',
'hw/scsi',
'hw/sd',
+ 'hw/sensor',
'hw/sh4',
'hw/sparc',
'hw/sparc64',
diff --git a/hw/sensor/trace.h b/hw/sensor/trace.h
new file mode 100644
index 0000000000..e4721560b0
--- /dev/null
+++ b/hw/sensor/trace.h
@@ -0,0 +1 @@
+#include "trace/trace-hw_sensor.h"
diff --git a/hw/sensor/tmp105.c b/hw/sensor/tmp105.c
index 9d7b911f59..ef2824f3e1 100644
--- a/hw/sensor/tmp105.c
+++ b/hw/sensor/tmp105.c
@@ -27,6 +27,7 @@
#include "qapi/visitor.h"
#include "qemu/module.h"
#include "hw/registerfields.h"
+#include "trace.h"
FIELD(CONFIG, SHUTDOWN_MODE, 0, 1)
FIELD(CONFIG, THERMOSTAT_MODE, 1, 1)
@@ -150,17 +151,21 @@ static void tmp105_read(TMP105State *s)
s->buf[s->len++] = ((uint16_t) s->limit[1]) >> 0;
break;
}
+
+ trace_tmp105_read(s->i2c.address, s->pointer);
}
static void tmp105_write(TMP105State *s)
{
+ trace_tmp105_write(s->i2c.address, s->pointer);
+
switch (s->pointer & 3) {
case TMP105_REG_TEMPERATURE:
break;
case TMP105_REG_CONFIG:
if (FIELD_EX8(s->buf[0] & ~s->config, CONFIG, SHUTDOWN_MODE)) {
- printf("%s: TMP105 shutdown\n", __func__);
+ trace_tmp105_write_shutdown(s->i2c.address);
}
s->config = FIELD_DP8(s->buf[0], CONFIG, ONE_SHOT, 0);
s->faults = tmp105_faultq[FIELD_EX8(s->config, CONFIG, FAULT_QUEUE)];
diff --git a/hw/sensor/trace-events b/hw/sensor/trace-events
new file mode 100644
index 0000000000..a3fe54fa6d
--- /dev/null
+++ b/hw/sensor/trace-events
@@ -0,0 +1,6 @@
+# See docs/devel/tracing.rst for syntax documentation.
+
+# tmp105.c
+tmp105_read(uint8_t dev, uint8_t addr) "device: 0x%02x, addr: 0x%02x"
+tmp105_write(uint8_t dev, uint8_t addr) "device: 0x%02x, addr 0x%02x"
+tmp105_write_shutdown(uint8_t dev) "device: 0x%02x"
--
2.47.0
- [PATCH v2 0/4] Trivial ARM changes, Bernhard Beschow, 2024/11/03
- [PATCH v2 1/4] hw/rtc/ds1338: Trace send and receive operations, Bernhard Beschow, 2024/11/03
- [PATCH v2 2/4] hw/timer/imx_gpt: Convert DPRINTF to trace events, Bernhard Beschow, 2024/11/03
- [PATCH v2 3/4] hw/watchdog/wdt_imx2: Remove redundant assignment, Bernhard Beschow, 2024/11/03
- [PATCH v2 4/4] hw/sensor/tmp105: Convert printf() to trace event, add tracing for read/write access,
Bernhard Beschow <=
- Re: [PATCH v2 0/4] Trivial ARM changes, Peter Maydell, 2024/11/04