[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 08/10] hw/sd/pl181: Replace disabled fprintf()s by trace e
From: |
Alistair Francis |
Subject: |
Re: [PATCH v4 08/10] hw/sd/pl181: Replace disabled fprintf()s by trace events |
Date: |
Mon, 6 Jul 2020 09:13:15 -0700 |
On Sun, Jul 5, 2020 at 1:51 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Convert disabled DPRINTF() to trace events and remove ifdef'ry.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
> hw/sd/pl181.c | 26 +++++++++-----------------
> hw/sd/trace-events | 10 ++++++++++
> 2 files changed, 19 insertions(+), 17 deletions(-)
>
> diff --git a/hw/sd/pl181.c b/hw/sd/pl181.c
> index f69488ebac..574500ce60 100644
> --- a/hw/sd/pl181.c
> +++ b/hw/sd/pl181.c
> @@ -17,15 +17,7 @@
> #include "qemu/module.h"
> #include "qemu/error-report.h"
> #include "qapi/error.h"
> -
> -//#define DEBUG_PL181 1
> -
> -#ifdef DEBUG_PL181
> -#define DPRINTF(fmt, ...) \
> -do { printf("pl181: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while(0)
> -#endif
> +#include "trace.h"
>
> #define PL181_FIFO_LEN 16
>
> @@ -158,7 +150,7 @@ static void pl181_fifo_push(PL181State *s, uint32_t value)
> n = (s->fifo_pos + s->fifo_len) & (PL181_FIFO_LEN - 1);
> s->fifo_len++;
> s->fifo[n] = value;
> - DPRINTF("FIFO push %08x\n", (int)value);
> + trace_pl181_fifo_push(value);
> }
>
> static uint32_t pl181_fifo_pop(PL181State *s)
> @@ -172,7 +164,7 @@ static uint32_t pl181_fifo_pop(PL181State *s)
> value = s->fifo[s->fifo_pos];
> s->fifo_len--;
> s->fifo_pos = (s->fifo_pos + 1) & (PL181_FIFO_LEN - 1);
> - DPRINTF("FIFO pop %08x\n", (int)value);
> + trace_pl181_fifo_pop(value);
> return value;
> }
>
> @@ -184,7 +176,7 @@ static void pl181_do_command(PL181State *s)
>
> request.cmd = s->cmd & PL181_CMD_INDEX;
> request.arg = s->cmdarg;
> - DPRINTF("Command %d %08x\n", request.cmd, request.arg);
> + trace_pl181_command_send(request.cmd, request.arg);
> rlen = sdbus_do_command(&s->sdbus, &request, response);
> if (rlen < 0)
> goto error;
> @@ -201,16 +193,16 @@ static void pl181_do_command(PL181State *s)
> s->response[2] = ldl_be_p(&response[8]);
> s->response[3] = ldl_be_p(&response[12]) & ~1;
> }
> - DPRINTF("Response received\n");
> + trace_pl181_command_response_pending();
> s->status |= PL181_STATUS_CMDRESPEND;
> } else {
> - DPRINTF("Command sent\n");
> + trace_pl181_command_sent();
> s->status |= PL181_STATUS_CMDSENT;
> }
> return;
>
> error:
> - DPRINTF("Timeout\n");
> + trace_pl181_command_timeout();
> s->status |= PL181_STATUS_CMDTIMEOUT;
> }
>
> @@ -262,11 +254,11 @@ static void pl181_fifo_run(PL181State *s)
> s->status |= PL181_STATUS_DATAEND;
> /* HACK: */
> s->status |= PL181_STATUS_DATABLOCKEND;
> - DPRINTF("Transfer Complete\n");
> + trace_pl181_fifo_transfer_complete();
> }
> if (s->datacnt == 0 && s->fifo_len == 0) {
> s->datactrl &= ~PL181_DATA_ENABLE;
> - DPRINTF("Data engine idle\n");
> + trace_pl181_data_engine_idle();
> } else {
> /* Update FIFO bits. */
> bits = PL181_STATUS_TXACTIVE | PL181_STATUS_RXACTIVE;
> diff --git a/hw/sd/trace-events b/hw/sd/trace-events
> index 5f09d32eb2..a87d7355fb 100644
> --- a/hw/sd/trace-events
> +++ b/hw/sd/trace-events
> @@ -62,3 +62,13 @@ milkymist_memcard_memory_write(uint32_t addr, uint32_t
> value) "addr 0x%08x value
> # pxa2xx_mmci.c
> pxa2xx_mmci_read(uint8_t size, uint32_t addr, uint32_t value) "size %d addr
> 0x%02x value 0x%08x"
> pxa2xx_mmci_write(uint8_t size, uint32_t addr, uint32_t value) "size %d addr
> 0x%02x value 0x%08x"
> +
> +# pl181.c
> +pl181_command_send(uint8_t cmd, uint32_t arg) "sending CMD%02d arg 0x%08"
> PRIx32
> +pl181_command_sent(void) "command sent"
> +pl181_command_response_pending(void) "response received"
> +pl181_command_timeout(void) "command timeouted"
> +pl181_fifo_push(uint32_t data) "FIFO push 0x%08" PRIx32
> +pl181_fifo_pop(uint32_t data) "FIFO pop 0x%08" PRIx32
> +pl181_fifo_transfer_complete(void) "FIFO transfer complete"
> +pl181_data_engine_idle(void) "data engine idle"
> --
> 2.21.3
>
>
- [PATCH v4 04/10] hw/arm/versatilepb: Comment to remember some IRQs lines are left unwired, (continued)
- [PATCH v4 04/10] hw/arm/versatilepb: Comment to remember some IRQs lines are left unwired, Philippe Mathieu-Daudé, 2020/07/05
- [PATCH v4 05/10] hw/sd/pl181: Use named GPIOs, Philippe Mathieu-Daudé, 2020/07/05
- [PATCH v4 06/10] hw/sd/pl181: Expose a SDBus and connect the SDCard to it, Philippe Mathieu-Daudé, 2020/07/05
- [PATCH v4 07/10] hw/sd/pl181: Do not create SD card within the SDHCI controller, Philippe Mathieu-Daudé, 2020/07/05
- [PATCH v4 08/10] hw/sd/pl181: Replace disabled fprintf()s by trace events, Philippe Mathieu-Daudé, 2020/07/05
- Re: [PATCH v4 08/10] hw/sd/pl181: Replace disabled fprintf()s by trace events,
Alistair Francis <=
- [PATCH v4 09/10] hw/sd/sdcard: make sd_data_ready() static, Philippe Mathieu-Daudé, 2020/07/05
- [PATCH v4 10/10] hw/sd: move sdcard legacy API to 'hw/sd/sdcard_legacy.h', Philippe Mathieu-Daudé, 2020/07/05
- Re: [PATCH v4 00/10] hw/sd: convert legacy SDHCI devices to the SDBus API, Peter Maydell, 2020/07/09