Re: [Qemu-stable] [Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Michael Roth]

Quoting Stefan Weil (2013-07-26 00:12:59)
> Am 26.07.2013 04:03, schrieb jean-christophe manciot:
> > Public bug reported:
> >
> > Environment: Ubuntu 13.04
> >
> > "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
> > hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long 
> > unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
> > hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long 
> > int’, but argument 5 has type ‘int64_t’ [-Werror=format]
> > hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long 
> > unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > 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"
> >
> > I got the source files with a "git clone git://git.qemu-
> > project.org/qemu.git" + a recent "git fetch"
> >
> > ** Affects: qemu
> >      Importance: Undecided
> >          Status: New
> >
> 
> 
> 
> This patch should fix it: http://patchwork.ozlabs.org/patch/258774/.
> 
> It's also still missing in git master, but was already applied to
> qemu-trivial.

This doesn't seem to be vanilla 1.5.2, where 04dd1259 isn't applicable
(no MACIO_DPRINTF statements), but rather a newer release or past
version with this patch on top:

  commit 80fc95d8bdaf3392106b131a97ca701fd374489a
  Author: Alexander Graf <address@hidden>
  Date:   Fri Jun 28 13:30:01 2013 +0200
  
      PPC: dbdma: Support unaligned DMA access

I'd pull them both in if Alex wants to send a backported version for
1.5.2, but otherwise this doesn't seem to be an issue with stable.

> 
> Stefan