diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-02-15 11:09:50 +0100 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-02-15 11:09:50 +0100 |
commit | 73f8d938474d04013d785f5918d75b9d82c80ca3 (patch) | |
tree | c6ddedad00d35feb387e681bd2666504ea7d9a36 /c/src/lib/libcpu | |
parent | score: Fix warning (diff) | |
download | rtems-73f8d938474d04013d785f5918d75b9d82c80ca3.tar.bz2 |
bsps/powerpc: Fix warnings
Diffstat (limited to 'c/src/lib/libcpu')
-rw-r--r-- | c/src/lib/libcpu/powerpc/mpc6xx/mmu/bat.c | 20 | ||||
-rw-r--r-- | c/src/lib/libcpu/powerpc/mpc6xx/mmu/pte121.c | 4 | ||||
-rw-r--r-- | c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/ppc_exc_print.c | 92 | ||||
-rw-r--r-- | c/src/lib/libcpu/powerpc/shared/src/stack.c | 3 |
4 files changed, 61 insertions, 58 deletions
diff --git a/c/src/lib/libcpu/powerpc/mpc6xx/mmu/bat.c b/c/src/lib/libcpu/powerpc/mpc6xx/mmu/bat.c index c848f4cf56..7a9487b9ff 100644 --- a/c/src/lib/libcpu/powerpc/mpc6xx/mmu/bat.c +++ b/c/src/lib/libcpu/powerpc/mpc6xx/mmu/bat.c @@ -335,29 +335,29 @@ setbat (int typ, int bat_index, unsigned long virt, unsigned long phys, ubat bat; if (check_bat_index (bat_index)) { - printk ("Invalid BAT index\n", bat_index); + printk ("Invalid BAT index %d\n", bat_index); return -1; } if ((int) (bl = check_bat_size (size)) < 0) { - printk ("Invalid BAT size\n", size); + printk ("Invalid BAT size %u\n", size); return -1; } if (virt & (size - 1)) { - printk ("BAT effective address 0x%08x misaligned (size is 0x%08x)\n", + printk ("BAT effective address 0x%08lx misaligned (size is 0x%08x)\n", virt, size); return -1; } if (phys & (size - 1)) { - printk ("BAT physical address 0x%08x misaligned (size is 0x%08x)\n", phys, + printk ("BAT physical address 0x%08lx misaligned (size is 0x%08x)\n", phys, size); return -1; } if (virt + size - 1 < virt) { - printk ("BAT range invalid: wraps around zero 0x%08x..0x%08x\n", virt, + printk ("BAT range invalid: wraps around zero 0x%08lx..0x%08lx\n", virt, virt + size - 1); return -1; } @@ -383,7 +383,7 @@ setbat (int typ, int bat_index, unsigned long virt, unsigned long phys, err = check_overlap (typ, virt, size); if ((size >= (1 << 17)) && (err >= 0) && (err != bat_index)) { rtems_interrupt_enable (level); - printk ("BATs must not overlap; area 0x%08x..0x%08x hits %cBAT %i\n", + printk ("BATs must not overlap; area 0x%08lx..0x%08lx hits %cBAT %i\n", virt, virt + size, (TYP_I == typ ? 'I' : 'D'), err); return -1; } @@ -495,13 +495,13 @@ getbat (int typ, int idx, unsigned long *pu, unsigned long *pl) ubat b; b.words.u = u; b.words.l = l; - printk ("Raw %cBAT %i contents; UPPER: (0x%08x)", (TYP_I == typ ? 'I' : 'D'), idx, u); + printk ("Raw %cBAT %i contents; UPPER: (0x%08lx)", (TYP_I == typ ? 'I' : 'D'), idx, u); printk (" BEPI: 0x%08x", b.bat.batu.bepi); - printk (" BL: 0x%08x", (u >> 2) & ((1 << 15) - 1)); + printk (" BL: 0x%08lx", (u >> 2) & ((1 << 15) - 1)); printk (" VS: 0b%i", b.bat.batu.vs); printk (" VP: 0b%i", b.bat.batu.vp); printk ("\n"); - printk (" LOWER: (0x%08x)", l); + printk (" LOWER: (0x%08lx)", l); printk (" RPN: 0x%08x", b.bat.batl.brpn); printk (" wimg: 0b%1i%1i%1i%1i", b.bat.batl.w, b.bat.batl.i, b.bat.batl.m, b.bat.batl.g); @@ -509,7 +509,7 @@ getbat (int typ, int idx, unsigned long *pu, unsigned long *pl) printk ("\n"); printk ("Covering EA Range: "); if (bat_in_use[typ] & (1 << idx)) - printk ("0x%08x .. 0x%08x\n", bat_addrs[typ][idx].start, + printk ("0x%08lx .. 0x%08lx\n", bat_addrs[typ][idx].start, bat_addrs[typ][idx].limit); else printk ("<none> (BAT off)\n"); diff --git a/c/src/lib/libcpu/powerpc/mpc6xx/mmu/pte121.c b/c/src/lib/libcpu/powerpc/mpc6xx/mmu/pte121.c index f6e9f42da6..fb346ead11 100644 --- a/c/src/lib/libcpu/powerpc/mpc6xx/mmu/pte121.c +++ b/c/src/lib/libcpu/powerpc/mpc6xx/mmu/pte121.c @@ -182,11 +182,11 @@ seg2vsid (uint32_t ea) /* Horrible Macros */ #ifdef __rtems__ /* must not use printf until multitasking is up */ -typedef void (*PrintF) (const char *, ...); +typedef int (*PrintF) (const char *, ...); static PrintF whatPrintf (void) { - return _Thread_Executing ? (PrintF) printf : printk; + return _Thread_Executing ? printf : printk; } #define PRINTF(args...) ((void)(whatPrintf())(args)) diff --git a/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/ppc_exc_print.c b/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/ppc_exc_print.c index 27b76a1934..f745713e00 100644 --- a/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/ppc_exc_print.c +++ b/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/ppc_exc_print.c @@ -21,6 +21,8 @@ #include <rtems/score/threaddispatch.h> +#include <inttypes.h> + #ifndef __SPE__ #define GET_GPR(gpr) (gpr) #else @@ -65,7 +67,7 @@ void BSP_printStackTrace(const BSP_Exception_frame *excPtr) printk("Stack Trace: \n "); if (excPtr) { - printk("IP: 0x%08x, ", excPtr->EXC_SRR0); + printk("IP: 0x%08" PRIu32 ", ", excPtr->EXC_SRR0); sp = (LRFrame) GET_GPR(excPtr->GPR1); lr = (void *) excPtr->EXC_LR; } else { @@ -73,9 +75,9 @@ void BSP_printStackTrace(const BSP_Exception_frame *excPtr) __asm__ __volatile__("mr %0, 1":"=r"(sp)); lr = (LRFrame) ppc_link_register(); } - printk("LR: 0x%08x\n", lr); + printk("LR: 0x%08" PRIuPTR "\n", (uintptr_t) lr); for (f = (LRFrame) sp, i = 0; f->frameLink && i < STACK_CLAMP; f = f->frameLink) { - printk("--^ 0x%08x", (long) (f->frameLink->lr)); + printk("--^ 0x%08" PRIuPTR "", (uintptr_t) (f->frameLink->lr)); if (!(++i % 5)) printk("\n"); } @@ -94,67 +96,67 @@ void _CPU_Exception_frame_print(const CPU_Exception_frame *excPtr) unsigned n = excPtr->_EXC_number & 0x7fff; printk("exception vector %d (0x%x)\n", n, n); - printk(" next PC or address of fault = 0x%08x\n", excPtr->EXC_SRR0); - printk(" saved MSR = 0x%08x\n", excPtr->EXC_SRR1); + printk(" next PC or address of fault = 0x%08" PRIu32 "\n", excPtr->EXC_SRR0); + printk(" saved MSR = 0x%08" PRIu32 "\n", excPtr->EXC_SRR1); /* Try to find out more about the context where this happened */ printk( - " context = %s, ISR nest level = %u\n", + " context = %s, ISR nest level = %" PRIu32 "\n", _ISR_Nest_level == 0 ? "task" : "interrupt", _ISR_Nest_level ); printk( - " thread dispatch disable level = %u\n", + " thread dispatch disable level = %" PRIu32 "\n", _Thread_Dispatch_disable_level ); /* Dump registers */ - printk(" R0 = 0x%08x", GET_GPR(excPtr->GPR0)); + printk(" R0 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR0)); if (synch) { - printk(" R1 = 0x%08x", GET_GPR(excPtr->GPR1)); - printk(" R2 = 0x%08x", GET_GPR(excPtr->GPR2)); + printk(" R1 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR1)); + printk(" R2 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR2)); } else { printk(" "); printk(" "); } - printk(" R3 = 0x%08x\n", GET_GPR(excPtr->GPR3)); - printk(" R4 = 0x%08x", GET_GPR(excPtr->GPR4)); - printk(" R5 = 0x%08x", GET_GPR(excPtr->GPR5)); - printk(" R6 = 0x%08x", GET_GPR(excPtr->GPR6)); - printk(" R7 = 0x%08x\n", GET_GPR(excPtr->GPR7)); - printk(" R8 = 0x%08x", GET_GPR(excPtr->GPR8)); - printk(" R9 = 0x%08x", GET_GPR(excPtr->GPR9)); - printk(" R10 = 0x%08x", GET_GPR(excPtr->GPR10)); - printk(" R11 = 0x%08x\n", GET_GPR(excPtr->GPR11)); - printk(" R12 = 0x%08x", GET_GPR(excPtr->GPR12)); + printk(" R3 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR3)); + printk(" R4 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR4)); + printk(" R5 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR5)); + printk(" R6 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR6)); + printk(" R7 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR7)); + printk(" R8 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR8)); + printk(" R9 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR9)); + printk(" R10 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR10)); + printk(" R11 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR11)); + printk(" R12 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR12)); if (synch) { - printk(" R13 = 0x%08x", GET_GPR(excPtr->GPR13)); - printk(" R14 = 0x%08x", GET_GPR(excPtr->GPR14)); - printk(" R15 = 0x%08x\n", GET_GPR(excPtr->GPR15)); - printk(" R16 = 0x%08x", GET_GPR(excPtr->GPR16)); - printk(" R17 = 0x%08x", GET_GPR(excPtr->GPR17)); - printk(" R18 = 0x%08x", GET_GPR(excPtr->GPR18)); - printk(" R19 = 0x%08x\n", GET_GPR(excPtr->GPR19)); - printk(" R20 = 0x%08x", GET_GPR(excPtr->GPR20)); - printk(" R21 = 0x%08x", GET_GPR(excPtr->GPR21)); - printk(" R22 = 0x%08x", GET_GPR(excPtr->GPR22)); - printk(" R23 = 0x%08x\n", GET_GPR(excPtr->GPR23)); - printk(" R24 = 0x%08x", GET_GPR(excPtr->GPR24)); - printk(" R25 = 0x%08x", GET_GPR(excPtr->GPR25)); - printk(" R26 = 0x%08x", GET_GPR(excPtr->GPR26)); - printk(" R27 = 0x%08x\n", GET_GPR(excPtr->GPR27)); - printk(" R28 = 0x%08x", GET_GPR(excPtr->GPR28)); - printk(" R29 = 0x%08x", GET_GPR(excPtr->GPR29)); - printk(" R30 = 0x%08x", GET_GPR(excPtr->GPR30)); - printk(" R31 = 0x%08x\n", GET_GPR(excPtr->GPR31)); + printk(" R13 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR13)); + printk(" R14 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR14)); + printk(" R15 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR15)); + printk(" R16 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR16)); + printk(" R17 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR17)); + printk(" R18 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR18)); + printk(" R19 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR19)); + printk(" R20 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR20)); + printk(" R21 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR21)); + printk(" R22 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR22)); + printk(" R23 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR23)); + printk(" R24 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR24)); + printk(" R25 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR25)); + printk(" R26 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR26)); + printk(" R27 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR27)); + printk(" R28 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR28)); + printk(" R29 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR29)); + printk(" R30 = 0x%08" PRIu32 "", GET_GPR(excPtr->GPR30)); + printk(" R31 = 0x%08" PRIu32 "\n", GET_GPR(excPtr->GPR31)); } else { printk("\n"); } - printk(" CR = 0x%08x\n", excPtr->EXC_CR); - printk(" CTR = 0x%08x\n", excPtr->EXC_CTR); - printk(" XER = 0x%08x\n", excPtr->EXC_XER); - printk(" LR = 0x%08x\n", excPtr->EXC_LR); + printk(" CR = 0x%08" PRIu32 "\n", excPtr->EXC_CR); + printk(" CTR = 0x%08" PRIu32 "\n", excPtr->EXC_CTR); + printk(" XER = 0x%08" PRIu32 "\n", excPtr->EXC_XER); + printk(" LR = 0x%08" PRIu32 "\n", excPtr->EXC_LR); /* Would be great to print DAR but unfortunately, * that is not portable across different CPUs. @@ -164,7 +166,7 @@ void _CPU_Exception_frame_print(const CPU_Exception_frame *excPtr) */ if (ppc_exc_get_DAR != NULL) { char* reg = ppc_cpu_is_60x() ? " DAR" : "DEAR"; - printk(" %s = 0x%08x\n", reg, ppc_exc_get_DAR()); + printk(" %s = 0x%08" PRIu32 "\n", reg, ppc_exc_get_DAR()); } if (ppc_cpu_is_bookE()) { unsigned esr, mcsr; @@ -219,7 +221,7 @@ void _CPU_Exception_frame_print(const CPU_Exception_frame *excPtr) const char *name = (const char *) &executing->Object.name; printk( - " executing thread ID = 0x%08x, name = %c%c%c%c\n", + " executing thread ID = 0x%08" PRIx32 ", name = %c%c%c%c\n", executing->Object.id, name [0], name [1], diff --git a/c/src/lib/libcpu/powerpc/shared/src/stack.c b/c/src/lib/libcpu/powerpc/shared/src/stack.c index 657ae2fed1..f19cecd87d 100644 --- a/c/src/lib/libcpu/powerpc/shared/src/stack.c +++ b/c/src/lib/libcpu/powerpc/shared/src/stack.c @@ -1,6 +1,7 @@ #include "stackTrace.h" #include <rtems/bspIo.h> #include <libcpu/spr.h> +#include <inttypes.h> SPR_RO(PPC_LR) @@ -36,7 +37,7 @@ void CPU_print_stack(void) for (i=0; stck[i]; i++) { if (i%5) printk("--> "); else printk("\n"); - printk("0x%08x",stck[i]); + printk("0x%08" PRIuPTR, (uintptr_t)stck[i]); } printk("\n"); } |