Sorry for late comment after the commit.
PRIx64 shouldn't be used instead of cast?
On Fri, Apr 01, 2011 at 10:08:23PM +0200, Aurelien Jarno wrote:
> On Mon, Feb 28, 2011 at 10:22:33AM +0800, Wen Congyang wrote:
> > I enable acpi_piix4 debug, and got the following build errors:
> > # make
> > CC libhw64/acpi_piix4.o
> > cc1: warnings being treated as errors
> > /home/wency/source/qemu/hw/acpi_piix4.c: In function ‘pm_ioport_write’:
> > /home/wency/source/qemu/hw/acpi_piix4.c:193: error: format ‘%04x’ expects
> > type ‘unsigned int’, but argument 2 has type ‘uint64_t’
> > /home/wency/source/qemu/hw/acpi_piix4.c:193: error: format ‘%04x’ expects
> > type ‘unsigned int’, but argument 3 has type ‘uint64_t’
> > /home/wency/source/qemu/hw/acpi_piix4.c: In function ‘pm_ioport_read’:
> > /home/wency/source/qemu/hw/acpi_piix4.c:219: error: format ‘%04x’ expects
> > type ‘unsigned int’, but argument 2 has type ‘uint64_t’
> > make[1]: *** [acpi_piix4.o] Error 1
> > make: *** [subdir-libhw64] Error 2
> >
> > Signed-off-by: Wen Congyang <we...@cn.fujitsu.com>
> >
> > ---
> > hw/acpi_piix4.c | 5 +++--
> > 1 files changed, 3 insertions(+), 2 deletions(-)
>
> Thanks, applied.
>
> > diff --git a/hw/acpi_piix4.c b/hw/acpi_piix4.c
> > index 5bbc2b5..b5a2762 100644
> > --- a/hw/acpi_piix4.c
> > +++ b/hw/acpi_piix4.c
> > @@ -190,7 +190,8 @@ static void pm_ioport_write(IORange *ioport, uint64_t
> > addr, unsigned width,
> > default:
> > break;
> > }
> > - PIIX4_DPRINTF("PM writew port=0x%04x val=0x%04x\n", addr, val);
> > + PIIX4_DPRINTF("PM writew port=0x%04x val=0x%04x\n", (unsigned int)addr,
> > + (unsigned int)val);
> > }
> >
> > static void pm_ioport_read(IORange *ioport, uint64_t addr, unsigned width,
> > @@ -216,7 +217,7 @@ static void pm_ioport_read(IORange *ioport, uint64_t
> > addr, unsigned width,
> > val = 0;
> > break;
> > }
> > - PIIX4_DPRINTF("PM readw port=0x%04x val=0x%04x\n", addr, val);
> > + PIIX4_DPRINTF("PM readw port=0x%04x val=0x%04x\n", (unsigned int)addr,
> > val);
> > *data = val;
> > }
> >
> > --
> > 1.7.1
> >
> >
>
> --
> Aurelien Jarno GPG: 1024D/F1BCDB73
> aurel...@aurel32.net http://www.aurel32.net
>
--
yamahata
