Thanks, applied.
On Thu, May 27, 2010 at 5:37 AM, Isaku Yamahata <yamah...@valinux.co.jp> wrote:
> This patch fixes the following compilation errors in multiboot.c
> when DEBUG_MULTIBOOT is defined.
> Use TARGET_FMT_plx instead of %x for target_phys_addr_t.
>
> CC i386-softmmu/multiboot.o
> cc1: warnings being treated as errors
> qemu/hw/multiboot.c: In function 'mb_add_mod':
> qemu/hw/multiboot.c:121: error: format '%08x' expects type 'unsigned int',
> but argument 4 has type 'target_phys_addr_t'
> qemu/hw/multiboot.c:121: error: format '%08x' expects type 'unsigned int',
> but argument 5 has type 'target_phys_addr_t'
> qemu/hw/multiboot.c: In function 'load_multiboot':
> qemu/hw/multiboot.c:279: error: format '%#x' expects type 'unsigned int', but
> argument 5 has type 'target_phys_addr_t'
> qemu/hw/multiboot.c:307: error: format '%x' expects type 'unsigned int', but
> argument 3 has type 'target_phys_addr_t'
> qemu/hw/multiboot.c:308: error: format '%x' expects type 'unsigned int', but
> argument 3 has type 'target_phys_addr_t'
> make[1]: *** [multiboot.o] Error 1
>
> Signed-off-by: Isaku Yamahata <yamah...@valinux.co.jp>
> ---
> hw/multiboot.c | 9 +++++----
> 1 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/hw/multiboot.c b/hw/multiboot.c
> index a1b665c..dc980e6 100644
> --- a/hw/multiboot.c
> +++ b/hw/multiboot.c
> @@ -118,7 +118,8 @@ static void mb_add_mod(MultibootState *s,
> stl_p(p + MB_MOD_END, end);
> stl_p(p + MB_MOD_CMDLINE, cmdline_phys);
>
> - mb_debug("mod%02d: %08x - %08x\n", s->mb_mods_count, start, end);
> + mb_debug("mod%02d: "TARGET_FMT_plx" - "TARGET_FMT_plx"\n",
> + s->mb_mods_count, start, end);
>
> s->mb_mods_count++;
> }
> @@ -276,7 +277,7 @@ int load_multiboot(void *fw_cfg,
> mb_add_mod(&mbs, mbs.mb_buf_phys + offs,
> mbs.mb_buf_phys + offs + mb_mod_length, c);
>
> - mb_debug("mod_start: %p\nmod_end: %p\n cmdline: %#x\n",
> + mb_debug("mod_start: %p\nmod_end: %p\n cmdline:
> "TARGET_FMT_plx"\n",
> (char *)mbs.mb_buf + offs,
> (char *)mbs.mb_buf + offs + mb_mod_length, c);
> initrd_filename = next_initrd+1;
> @@ -304,8 +305,8 @@ int load_multiboot(void *fw_cfg,
> stl_p(bootinfo + MBI_MMAP_ADDR, ADDR_E820_MAP);
>
> mb_debug("multiboot: mh_entry_addr = %#x\n", mh_entry_addr);
> - mb_debug(" mb_buf_phys = %x\n", mbs.mb_buf_phys);
> - mb_debug(" mod_start = %x\n", mbs.mb_buf_phys +
> mbs.offset_mods);
> + mb_debug(" mb_buf_phys = "TARGET_FMT_plx"\n",
> mbs.mb_buf_phys);
> + mb_debug(" mod_start = "TARGET_FMT_plx"\n",
> mbs.mb_buf_phys + mbs.offset_mods);
> mb_debug(" mb_mods_count = %d\n", mbs.mb_mods_count);
>
> /* save bootinfo off the stack */
> --
> 1.6.6.1
>
>
>
> --
> yamahata
>
>
