On Tue, Aug 27, 2013 at 05:48:36PM +0100, Yongbok Kim wrote: > From: James Hogan <[email protected]> > > In get_physical_address() is a qemu_log() call inside an #if 0 block. > When enabled the following build error is hit: > > target-mips/helper.c In function ‘get_physical_address’: > target-mips/helper.c:220:13: error: format ‘%x’ expects argument of type > ‘unsigned int’, but argument 5 has type ‘hwaddr’ [-Werror=format] > > Fix the *physical (hwaddr) formatting by using "%"HWADDR_PRIx instead of > TARGET_FMT_lx. > > Signed-off-by: James Hogan <[email protected]> > Cc: Aurelien Jarno <[email protected]> > Signed-off-by: Yongbok Kim <[email protected]> > --- > target-mips/helper.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/target-mips/helper.c b/target-mips/helper.c > index 6feef7b..33e0e88 100644 > --- a/target-mips/helper.c > +++ b/target-mips/helper.c > @@ -193,7 +193,7 @@ static int get_physical_address (CPUMIPSState *env, > hwaddr *physical, > } > } > #if 0 > - qemu_log(TARGET_FMT_lx " %d %d => " TARGET_FMT_lx " %d (%d)\n", > + qemu_log(TARGET_FMT_lx " %d %d => %" HWADDR_PRIx " %d (%d)\n", > address, rw, access_type, *physical, *prot, ret); > #endif
Thanks, applied. -- Aurelien Jarno GPG: 1024D/F1BCDB73 [email protected] http://www.aurel32.net
