Comment 4 for bug 1205156

Quoting Michael Roth (2013-08-12 20:05:32)
> 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 <email 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.

Forgot to cc Alex.

>
> >
> > Stefan