[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 03/11] sdcard: add a trace event for command
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v4 03/11] sdcard: add a trace event for command responses |
Date: |
Mon, 26 Feb 2018 18:14:53 -0800 |
On Thu, Feb 15, 2018 at 2:05 PM, Philippe Mathieu-Daudé <address@hidden> wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> hw/sd/sd.c | 27 ++++++++++++++++++++++++---
> hw/sd/trace-events | 1 +
> 2 files changed, 25 insertions(+), 3 deletions(-)
>
> diff --git a/hw/sd/sd.c b/hw/sd/sd.c
> index 72e9b47e34..8f72cde534 100644
> --- a/hw/sd/sd.c
> +++ b/hw/sd/sd.c
> @@ -153,6 +153,27 @@ static const char *sd_state_name(enum SDCardStates state)
> return state_name[state];
> }
>
> +static const char *sd_response_name(sd_rsp_type_t rsp)
> +{
> + static const char *response_name[] = {
> + [sd_r0] = "RESP#0 (no response)",
> + [sd_r1] = "RESP#1 (normal cmd)",
> + [sd_r2_i] = "RESP#2 (CID reg)",
> + [sd_r2_s] = "RESP#2 (CSD reg)",
> + [sd_r3] = "RESP#3 (OCR reg)",
> + [sd_r6] = "RESP#6 (RCA)",
> + [sd_r7] = "RESP#7 (operating voltage)",
> + };
> + if (rsp == sd_illegal) {
> + return "ILLEGAL RESP";
> + }
> + if (rsp == sd_r1b) {
> + rsp = sd_r1;
> + }
> + assert(rsp <= ARRAY_SIZE(response_name));
> + return response_name[rsp];
> +}
> +
> static uint8_t sd_get_dat_lines(SDState *sd)
> {
> return sd->enable ? sd->dat_lines : 0;
> @@ -1596,10 +1617,12 @@ send_response:
>
> case sd_r0:
> case sd_illegal:
> - default:
> rsplen = 0;
> break;
> + default:
> + g_assert_not_reached();
> }
> + trace_sdcard_response(sd_response_name(rtype), rsplen);
>
> if (rtype != sd_illegal) {
> /* Clear the "clear on valid command" status bits now we've
> @@ -1616,8 +1639,6 @@ send_response:
> DPRINTF(" %02x", response[i]);
> }
> DPRINTF(" state %d\n", sd->state);
> - } else {
> - DPRINTF("No response %d\n", sd->state);
> }
> #endif
>
> diff --git a/hw/sd/trace-events b/hw/sd/trace-events
> index 75dac5a2cd..b2aa19ec0d 100644
> --- a/hw/sd/trace-events
> +++ b/hw/sd/trace-events
> @@ -26,6 +26,7 @@ sdhci_capareg(const char *desc, uint16_t val) "%s: %u"
> # hw/sd/sd.c
> sdcard_normal_command(uint8_t cmd, uint32_t arg, const char *state) "CMD%d
> arg 0x%08x (state %s)"
> sdcard_app_command(uint8_t acmd, uint32_t arg) "ACMD%d arg 0x%08x"
> +sdcard_response(const char *rspdesc, int rsplen) "%s (sz:%d)"
> sdcard_read_block(uint64_t addr, uint32_t len) "addr 0x%" PRIx64 " size 0x%x"
> sdcard_write_block(uint64_t addr, uint32_t len) "addr 0x%" PRIx64 " size
> 0x%x"
>
> --
> 2.16.1
>
>
- [Qemu-devel] [PATCH v4 00/11] SDCard: housekeeping, add tracing (part 4), Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 01/11] sdcard: reorder SDState struct members, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 02/11] sdcard: replace DPRINTF() by trace events, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 03/11] sdcard: add a trace event for command responses, Philippe Mathieu-Daudé, 2018/02/15
- Re: [Qemu-devel] [PATCH v4 03/11] sdcard: add a trace event for command responses,
Alistair Francis <=
- [Qemu-devel] [PATCH v4 04/11] sdcard: replace fprintf() by qemu_hexdump(), Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 05/11] sdcard: add more trace events, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 06/11] sdcard: do not trace CMD55 when expecting ACMD, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 07/11] sdcard: define SDMMC_CMD_MAX instead of using the magic '64', Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 08/11] sdcard: display command name when tracing CMD/ACMD, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 09/11] sdcard: display protocol used when tracing, Philippe Mathieu-Daudé, 2018/02/15
- [Qemu-devel] [PATCH v4 10/11] sdcard: use G_BYTE from cutils, Philippe Mathieu-Daudé, 2018/02/15