[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression) |
Date: |
Tue, 16 Jul 2013 07:54:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 |
Am 12.07.2013 18:48, schrieb Stefan Weil:
> Fix a number of warnings for 32 bit builds (tested on MingW and Linux):
>
> CC hw/ide/macio.o
> qemu/hw/ide/macio.c: In function 'pmac_ide_atapi_transfer_cb':
> qemu/hw/ide/macio.c:134:9: error: format '%lx' expects argument of type 'long
> unsigned int', but argument 3 has type 'hwaddr' [-Werror=format]
> qemu/hw/ide/macio.c: In function 'pmac_ide_transfer_cb':
> qemu/hw/ide/macio.c:215:5: error: format '%ld' expects argument of type 'long
> int', but argument 5 has type 'int64_t' [-Werror=format]
> qemu/hw/ide/macio.c:222:9: error: format '%lx' expects argument of type 'long
> unsigned int', but argument 3 has type 'hwaddr' [-Werror=format]
> qemu/hw/ide/macio.c:264:9: error: format '%lx' expects argument of type 'long
> unsigned int', but argument 3 has type 'hwaddr' [-Werror=format]
> cc1: all warnings being treated as errors
> make: *** [hw/ide/macio.o] Error 1
>
> Signed-off-by: Stefan Weil <address@hidden>
> ---
>
>
>
> Hi Anthony,
>
> the patch fixes a build regression which was introduced today.
> Could you please apply it without waiting for the next pull requests?
>
> Thanks,
> Stefan
>
>
>
> hw/ide/macio.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/hw/ide/macio.c b/hw/ide/macio.c
> index 38ad924..ef4ba2b 100644
> --- a/hw/ide/macio.c
> +++ b/hw/ide/macio.c
> @@ -131,7 +131,7 @@ static void pmac_ide_atapi_transfer_cb(void *opaque, int
> ret)
> int sector_num = (s->lba << 2) + (s->io_buffer_index >> 9);
> int nsector = io->len >> 9;
>
> - MACIO_DPRINTF("precopying unaligned %d bytes to %#lx\n",
> + MACIO_DPRINTF("precopying unaligned %d bytes to %#" HWADDR_PRIx "\n",
> unaligned, io->addr + io->len - unaligned);
>
> bdrv_read(s->bs, sector_num + nsector, io->remainder, 1);
> @@ -212,14 +212,15 @@ static void pmac_ide_transfer_cb(void *opaque, int ret)
> s->nsector -= n;
> }
>
> - MACIO_DPRINTF("remainder: %d io->len: %d nsector: %d sector_num: %ld\n",
> + MACIO_DPRINTF("remainder: %d io->len: %d nsector: %d "
> + "sector_num: %" PRId64 "\n",
> io->remainder_len, io->len, s->nsector, sector_num);
> if (io->remainder_len && io->len) {
> /* guest wants the rest of its previous transfer */
> int remainder_len = MIN(io->remainder_len, io->len);
> uint8_t *p = &io->remainder[0x200 - remainder_len];
>
> - MACIO_DPRINTF("copying remainder %d bytes at %#lx\n",
> + MACIO_DPRINTF("copying remainder %d bytes at %#" HWADDR_PRIx "\n",
> remainder_len, io->addr);
>
> switch (s->dma_cmd) {
> @@ -261,7 +262,7 @@ static void pmac_ide_transfer_cb(void *opaque, int ret)
> if (unaligned) {
> int nsector = io->len >> 9;
>
> - MACIO_DPRINTF("precopying unaligned %d bytes to %#lx\n",
> + MACIO_DPRINTF("precopying unaligned %d bytes to %#" HWADDR_PRIx "\n",
> unaligned, io->addr + io->len - unaligned);
>
> switch (s->dma_cmd) {
Ping?
- [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Stefan Weil, 2013/07/12
- Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression),
Stefan Weil <=
- Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Stefan Weil, 2013/07/18
- Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Alexander Graf, 2013/07/18
- Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Stefan Weil, 2013/07/19
- Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Andreas Färber, 2013/07/20
- Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Stefan Weil, 2013/07/21
- Re: [Qemu-devel] [Qemu-trivial] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression), Michael Tokarev, 2013/07/23