From 526ccb7a26fea4b07eae4d34c8ad0599802a93ce Mon Sep 17 00:00:00 2001 From: balrog Date: Wed, 16 Jul 2008 12:13:52 +0000 Subject: [PATCH] Fix a bunch of type mismatch-related warnings (Jan Kiszka). Fix a typo in my previous comming (spotted by Laurent Desnouges). git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@4877 c046a42c-6fe2-441c-8c8c-71466251a162 --- hw/pc.c | 13 +++++++------ hw/sh7750.c | 4 ++-- linux-user/flatload.c | 6 +++--- linux-user/m68k-sim.c | 12 ++++++------ linux-user/signal.c | 6 +++--- linux-user/syscall.c | 1 + target-sh4/op.c | 2 +- 7 files changed, 23 insertions(+), 21 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index 99df09d9a5..fb3e0c9e6d 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -550,9 +550,9 @@ static void load_linux(const char *kernel_filename, #if 0 fprintf(stderr, - "qemu: real_addr = %#zx\n" - "qemu: cmdline_addr = %#zx\n" - "qemu: prot_addr = %#zx\n", + "qemu: real_addr = 0x" TARGET_FMT_plx "\n" + "qemu: cmdline_addr = 0x" TARGET_FMT_plx "\n" + "qemu: prot_addr = 0x" TARGET_FMT_plx "\n", real_addr, cmdline_addr, prot_addr); @@ -607,8 +607,8 @@ static void load_linux(const char *kernel_filename, initrd_size = get_file_size(fi); initrd_addr = (initrd_max-initrd_size) & ~4095; - fprintf(stderr, "qemu: loading initrd (%#x bytes) at %#zx\n", - initrd_size, initrd_addr); + fprintf(stderr, "qemu: loading initrd (%#x bytes) at 0x" TARGET_FMT_plx + "\n", initrd_size, initrd_addr); if (!fread_targphys_ok(initrd_addr, initrd_size, fi)) { fprintf(stderr, "qemu: read error on initial ram disk '%s'\n", @@ -778,7 +778,8 @@ static void pc_init1(ram_addr_t ram_size, int vga_ram_size, /* above 4giga memory allocation */ if (above_4g_mem_size > 0) { - cpu_register_physical_memory(0x100000000ULL, above_4g_mem_size, + cpu_register_physical_memory((target_phys_addr_t) 0x100000000ULL, + above_4g_mem_size, ram_addr + below_4g_mem_size); } diff --git a/hw/sh7750.c b/hw/sh7750.c index a60a83ba66..a76e6d4d3c 100644 --- a/hw/sh7750.c +++ b/hw/sh7750.c @@ -182,13 +182,13 @@ static void portb_changed(SH7750State * s, uint16_t prev) static void error_access(const char *kind, target_phys_addr_t addr) { - fprintf(stderr, "%s to %s (0x%08x) not supported\n", + fprintf(stderr, "%s to %s (0x" TARGET_FMT_plx ") not supported\n", kind, regname(addr), addr); } static void ignore_access(const char *kind, target_phys_addr_t addr) { - fprintf(stderr, "%s to %s (0x%08x) ignored\n", + fprintf(stderr, "%s to %s (0x" TARGET_FMT_plx ") ignored\n", kind, regname(addr), addr); } diff --git a/linux-user/flatload.c b/linux-user/flatload.c index 95d386468a..29664b1447 100644 --- a/linux-user/flatload.c +++ b/linux-user/flatload.c @@ -349,9 +349,9 @@ void old_reloc(struct lib_info *libinfo, uint32_t rl) reloc_type = rl >> 30; /* ??? How to handle this? */ #if defined(CONFIG_COLDFIRE) - ptr = (uint32_t *) (libinfo->start_code + offset); + ptr = (uint32_t *) ((unsigned long) libinfo->start_code + offset); #else - ptr = (uint32_t *) (libinfo->start_data + offset); + ptr = (uint32_t *) ((unsigned long) libinfo->start_data + offset); #endif #ifdef DEBUG @@ -670,7 +670,7 @@ static int load_flat_file(struct linux_binprm * bprm, } /* zero the BSS. */ - memset((void*)(datapos + data_len), 0, bss_len); + memset((void *)((unsigned long)datapos + data_len), 0, bss_len); return 0; } diff --git a/linux-user/m68k-sim.c b/linux-user/m68k-sim.c index 61c2468752..149399b547 100644 --- a/linux-user/m68k-sim.c +++ b/linux-user/m68k-sim.c @@ -101,19 +101,19 @@ void do_m68k_simcall(CPUM68KState *env, int nr) { uint32_t *args; - args = (uint32_t *)(env->aregs[7] + 4); + args = (uint32_t *)(unsigned long)(env->aregs[7] + 4); switch (nr) { case SYS_EXIT: exit(ARG(0)); case SYS_READ: - check_err(env, read(ARG(0), (void *)ARG(1), ARG(2))); + check_err(env, read(ARG(0), (void *)(unsigned long)ARG(1), ARG(2))); break; case SYS_WRITE: - check_err(env, write(ARG(0), (void *)ARG(1), ARG(2))); + check_err(env, write(ARG(0), (void *)(unsigned long)ARG(1), ARG(2))); break; case SYS_OPEN: - check_err(env, open((char *)ARG(0), translate_openflags(ARG(1)), - ARG(2))); + check_err(env, open((char *)(unsigned long)ARG(0), + translate_openflags(ARG(1)), ARG(2))); break; case SYS_CLOSE: { @@ -142,7 +142,7 @@ void do_m68k_simcall(CPUM68KState *env, int nr) struct m86k_sim_stat *p; rc = check_err(env, fstat(ARG(0), &s)); if (rc == 0) { - p = (struct m86k_sim_stat *)ARG(1); + p = (struct m86k_sim_stat *)(unsigned long)ARG(1); p->sim_st_dev = tswap16(s.st_dev); p->sim_st_ino = tswap16(s.st_ino); p->sim_st_mode = tswap32(s.st_mode); diff --git a/linux-user/signal.c b/linux-user/signal.c index 599b8af2c0..af40238b2b 100644 --- a/linux-user/signal.c +++ b/linux-user/signal.c @@ -2755,7 +2755,7 @@ static void setup_rt_frame(int sig, struct target_sigaction *ka, /* Create the ucontext. */ err |= __put_user(0, &frame->uc.uc_flags); err |= __put_user(0, (unsigned long *)&frame->uc.uc_link); - err |= __put_user((void *)target_sigaltstack_used.ss_sp, + err |= __put_user((unsigned long)target_sigaltstack_used.ss_sp, &frame->uc.uc_stack.ss_sp); err |= __put_user(sas_ss_flags(regs->gregs[15]), &frame->uc.uc_stack.ss_flags); @@ -2982,11 +2982,11 @@ static void setup_frame(int sig, struct target_sigaction *ka, setup_sigcontext(&frame->sc, env); /* Move the stack and setup the arguments for the handler. */ - env->regs[R_SP] = (uint32_t) frame; + env->regs[R_SP] = (uint32_t) (unsigned long) frame; env->regs[10] = sig; env->pc = (unsigned long) ka->_sa_handler; /* Link SRP so the guest returns through the trampoline. */ - env->pregs[PR_SRP] = (uint32_t) &frame->retcode[0]; + env->pregs[PR_SRP] = (uint32_t) (unsigned long) &frame->retcode[0]; unlock_user_struct(frame, frame_addr, 1); return; diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 839ac7f757..c1cfe801cb 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -72,6 +72,7 @@ #include "linux_loop.h" #include "qemu.h" +#include "qemu-common.h" #if defined(USE_NPTL) #include diff --git a/target-sh4/op.c b/target-sh4/op.c index 4dc7193e19..22d49138fb 100644 --- a/target-sh4/op.c +++ b/target-sh4/op.c @@ -594,7 +594,7 @@ void OPPROTO op_shlr16_Rn(void) void OPPROTO op_tasb_rN(void) { - cond_t((env->gregs[PARAM1] && 0xff) == 0); + cond_t((env->gregs[PARAM1] & 0xff) == 0); *(int8_t *) &env->gregs[PARAM1] |= 0x80; RETURN(); } -- 2.11.4.GIT