On Thu, Dec 14, 2017 at 2:54 PM, Alistair Francis
<alistair.fran...@xilinx.com> wrote:
> On Wed, Dec 13, 2017 at 11:58 AM, Philippe Mathieu-Daudé
> <f4...@amsat.org> 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é <f4...@amsat.org>
>
> Acked-by: Alistair Francis <alistair.fran...@xilinx.com>
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"
