[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/14] sdhci: convert the DPRINT() calls into tr
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 11/14] sdhci: convert the DPRINT() calls into trace events |
Date: |
Thu, 14 Dec 2017 15:19:51 -0300 |
On Thu, Dec 14, 2017 at 2:54 PM, Alistair Francis
<address@hidden> wrote:
> On Wed, Dec 13, 2017 at 11:58 AM, Philippe Mathieu-Daudé
> <address@hidden> wrote:
>> zero-initialize ADMADescr 'dscr' in sdhci_do_adma() to avoid:
>>
>> hw/sd/sdhci.c: In function ‘sdhci_do_adma’:
>> hw/sd/sdhci.c:714:29: error: ‘dscr.addr’ may be used uninitialized in this
>> function [-Werror=maybe-uninitialized]
>> trace_sdhci_adma("link", s->admasysaddr);
>> ^
>>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>
> Acked-by: Alistair Francis <address@hidden>
Thanks, note this patch has two incorrect uint64_t string format, I'll
add your Acked-by in the respin with "%x" replaced by PRIx64.
>> +# hw/sd/sdhci.c
>> +sdhci_set_inserted(const char *level) "card state changed: %s"
>> +sdhci_send_command(uint8_t cmd, uint32_t arg) "sending CMD%02u ARG[0x%08x]"
>> +sdhci_error(const char *msg) "%s"
>> +sdhci_response4(uint32_t r0) "Response: RSPREG[31..0]=0x%08x"
>> +sdhci_response16(uint32_t r3, uint32_t r2, uint32_t r1, uint32_t r0)
>> "Response received: RSPREG[127..96]=0x%08x, RSPREG[95..64]=0x%08x,
>> RSPREG[63..32]=0x%08x, RSPREG[31..0]=0x%08x"
>> +sdhci_end_transfer(uint8_t cmd, uint32_t arg) "Automatically issue CMD%02u
>> 0x%08x"
>> +sdhci_adma(const char *desc, uint32_t sysad) "ADMA %s: admasysaddr=0x%"
>> PRIx32
>> +sdhci_adma_loop(uint64_t addr, uint16_t length, uint8_t attr) "ADMA loop:
>> addr=0x%08" HWADDR_PRIx ", len=%d, attr=0x%x"
^ incorrect uint64_t format
>> +sdhci_adma_transfer_completed(void) "ADMA transfer completed"
>> +sdhci_access(const char *access, unsigned int size, uint64_t offset, const
>> char *dir, uint64_t val, uint64_t val2) "%s %ub: addr[0x%04" HWADDR_PRIx "]
>> %s %" PRIu64 "(0x%" PRIx64 ")"
>> +sdhci_read_dataport(uint16_t data_count) "all %u bytes of data have been
>> read from input buffer"
>> +sdhci_write_dataport(uint16_t data_count) "write buffer filled with %u
>> bytes of data"
- Re: [Qemu-devel] [PATCH 08/14] sdhci: refactor common sysbus/pci class_init() into sdhci_class_init(), (continued)
- [Qemu-devel] [PATCH 09/14] sdhci: refactor common sysbus/pci unrealize() into sdhci_unrealizefn(), Philippe Mathieu-Daudé, 2017/12/13
- [Qemu-devel] [PATCH 11/14] sdhci: convert the DPRINT() calls into trace events, Philippe Mathieu-Daudé, 2017/12/13
- [Qemu-devel] [PATCH 12/14] sdhci: add a trace event for the LED control, Philippe Mathieu-Daudé, 2017/12/13
- [Qemu-devel] [PATCH 13/14] sdhci: add sdhci_init_capareg() to initialize the CAPAB register, Philippe Mathieu-Daudé, 2017/12/13
- [Qemu-devel] [PATCH 14/14] sdhci: add a "dma-memory" property, Philippe Mathieu-Daudé, 2017/12/13