motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-11 03:24:58 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

hw/gpio/imx_gpio: Turn DPRINTF() into trace events

Browse source 
While at it add a trace event for input GPIO events.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
Message-ID: <20250111183711.2338-14-shentey@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
This commit is contained in:
Bernhard Beschow 2025-01-11 19:37:11 +01:00 • committed by Philippe Mathieu-Daudé
parent e589c0ea9c
commit e3778c8499
2 changed files with 12 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

18
hw/gpio/imx_gpio.c
View file

@ -24,6 +24,7 @@
#include "migration/vmstate.h" #include "migration/vmstate.h"
#include "qemu/log.h" #include "qemu/log.h"
#include "qemu/module.h" #include "qemu/module.h"
#include "trace.h"
#ifndef DEBUG_IMX_GPIO #ifndef DEBUG_IMX_GPIO
#define DEBUG_IMX_GPIO 0 #define DEBUG_IMX_GPIO 0
@ -34,14 +35,6 @@ typedef enum IMXGPIOLevel {
IMX_GPIO_LEVEL_HIGH = 1, IMX_GPIO_LEVEL_HIGH = 1,
} IMXGPIOLevel; } IMXGPIOLevel;
#define DPRINTF(fmt, args...) \
do { \
if (DEBUG_IMX_GPIO) { \
fprintf(stderr, "[%s]%s: " fmt , TYPE_IMX_GPIO, \
__func__, ##args); \
} \
} while (0)
static const char *imx_gpio_reg_name(uint32_t reg) static const char *imx_gpio_reg_name(uint32_t reg)
{ {
switch (reg) { switch (reg) {
@ -111,6 +104,8 @@ static void imx_gpio_set(void *opaque, int line, int level)
IMXGPIOState *s = IMX_GPIO(opaque); IMXGPIOState *s = IMX_GPIO(opaque);
IMXGPIOLevel imx_level = level ? IMX_GPIO_LEVEL_HIGH : IMX_GPIO_LEVEL_LOW; IMXGPIOLevel imx_level = level ? IMX_GPIO_LEVEL_HIGH : IMX_GPIO_LEVEL_LOW;
trace_imx_gpio_set(DEVICE(s)->canonical_path, line, imx_level);
imx_gpio_set_int_line(s, line, imx_level); imx_gpio_set_int_line(s, line, imx_level);
/* this is an input signal, so set PSR */ /* this is an input signal, so set PSR */
@ -200,7 +195,8 @@ static uint64_t imx_gpio_read(void *opaque, hwaddr offset, unsigned size)
break; break;
} }
DPRINTF("(%s) = 0x%" PRIx32 "\n", imx_gpio_reg_name(offset), reg_value); trace_imx_gpio_read(DEVICE(s)->canonical_path, imx_gpio_reg_name(offset),
reg_value);
return reg_value; return reg_value;
} }
@ -210,8 +206,8 @@ static void imx_gpio_write(void *opaque, hwaddr offset, uint64_t value,
{ {
IMXGPIOState *s = IMX_GPIO(opaque); IMXGPIOState *s = IMX_GPIO(opaque);
DPRINTF("(%s, value = 0x%" PRIx32 ")\n", imx_gpio_reg_name(offset), trace_imx_gpio_write(DEVICE(s)->canonical_path, imx_gpio_reg_name(offset),
(uint32_t)value); value);
switch (offset) { switch (offset) {
case DR_ADDR: case DR_ADDR:

5
hw/gpio/trace-events
View file

@ -1,5 +1,10 @@
# See docs/devel/tracing.rst for syntax documentation. # See docs/devel/tracing.rst for syntax documentation.
# imx_gpio.c
imx_gpio_read(const char *id, const char *reg, uint32_t value) "%s:[%s] -> 0x%" PRIx32
imx_gpio_write(const char *id, const char *reg, uint32_t value) "%s:[%s] <- 0x%" PRIx32
imx_gpio_set(const char *id, int line, int level) "%s:[%d] <- %d"
# npcm7xx_gpio.c # npcm7xx_gpio.c
npcm7xx_gpio_read(const char *id, uint64_t offset, uint64_t value) " %s offset: 0x%04" PRIx64 " value 0x%08" PRIx64 npcm7xx_gpio_read(const char *id, uint64_t offset, uint64_t value) " %s offset: 0x%04" PRIx64 " value 0x%08" PRIx64
npcm7xx_gpio_write(const char *id, uint64_t offset, uint64_t value) "%s offset: 0x%04" PRIx64 " value 0x%08" PRIx64 npcm7xx_gpio_write(const char *id, uint64_t offset, uint64_t value) "%s offset: 0x%04" PRIx64 " value 0x%08" PRIx64