m68k/m5484FireEngine and COBRA5475 strict alias warnings

Hi There appears to be something in the way the register access macros are defined for the genmcf548x BSP which results in a fairly large number of type-punned breaking strict alias warnings. There are 60 unique warnings reported in this BSP which is 3.5% of the total number of unique warnings

[PATCH 02/52] libcpu/powerpc/mpc8xx/clock: Fix warnings and clean up

--- c/src/lib/libbsp/powerpc/mbx8xx/clock/p_clock.c | 42 -- c/src/lib/libcpu/powerpc/mpc8xx/clock/clock.c | 58 - 2 files changed, 55 insertions(+), 45 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/mbx8xx/clock/p_clock.c

[PATCH 01/52] sparc64/shared/clock/ckinit.c: Clean up formatting

--- c/src/lib/libbsp/sparc64/shared/clock/ckinit.c | 31 +- 1 file changed, 16 insertions(+), 15 deletions(-) diff --git a/c/src/lib/libbsp/sparc64/shared/clock/ckinit.c b/c/src/lib/libbsp/sparc64/shared/clock/ckinit.c index d601a93..11779c4 100644 ---

[PATCH 05/52] libbsp/shared/clockdrv_shell.h: Fix warning

--- c/src/lib/libbsp/shared/clockdrv_shell.h | 1 + 1 file changed, 1 insertion(+) diff --git a/c/src/lib/libbsp/shared/clockdrv_shell.h b/c/src/lib/libbsp/shared/clockdrv_shell.h index 5af8c66..324e1e5 100644 --- a/c/src/lib/libbsp/shared/clockdrv_shell.h +++

[PATCH 08/52] nios2-iic-irq.c: Include rtems/score/interr.h so it builds

--- cpukit/score/cpu/nios2/nios2-iic-irq.c | 1 + 1 file changed, 1 insertion(+) diff --git a/cpukit/score/cpu/nios2/nios2-iic-irq.c b/cpukit/score/cpu/nios2/nios2-iic-irq.c index f51bc2d..aca25c7 100644 --- a/cpukit/score/cpu/nios2/nios2-iic-irq.c +++ b/cpukit/score/cpu/nios2/nios2-iic-irq.c

[PATCH 04/52] gdbv850sim/console/console-io.c: Fix warnings

--- c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c b/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c index c178197..a66c800 100644 ---

Upcoming Series of Warning Elimination Patches

Hi I am about to submit a series of 52 which are generally very small. They eliminate at least 100 unique warnings. Most do one of a handful of things: + Add include of bsp/bootcard.h + Add static or prototype of method + Add (void) VAR; to eliminate set but not used warning. + Some also split

[PATCH 07/52] libcpu/powerpc/mpc6xx/clock/c_clock.c: Fix warning and clean up

--- c/src/lib/libcpu/powerpc/mpc6xx/clock/c_clock.c | 89 +++-- 1 file changed, 39 insertions(+), 50 deletions(-) diff --git a/c/src/lib/libcpu/powerpc/mpc6xx/clock/c_clock.c b/c/src/lib/libcpu/powerpc/mpc6xx/clock/c_clock.c index ba6d269..75ef38b 100644 ---

[PATCH 17/52] arm/lpc24xx/misc/system-clocks.c: Fix warning

--- c/src/lib/libbsp/arm/lpc24xx/misc/system-clocks.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/arm/lpc24xx/misc/system-clocks.c b/c/src/lib/libbsp/arm/lpc24xx/misc/system-clocks.c index 6ef027d..f64fca1 100644 ---

[PATCH 20/52] arm/gba/startup/bspgetworkarea.c: Fix warning

--- c/src/lib/libbsp/arm/gba/startup/bspgetworkarea.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/c/src/lib/libbsp/arm/gba/startup/bspgetworkarea.c b/c/src/lib/libbsp/arm/gba/startup/bspgetworkarea.c index 4cd85b6..ca48afe 100644 ---

[PATCH 11/52] arm/gba/clock/clockdrv.c: Fix warnings and clean up

--- c/src/lib/libbsp/arm/gba/clock/clockdrv.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/c/src/lib/libbsp/arm/gba/clock/clockdrv.c b/c/src/lib/libbsp/arm/gba/clock/clockdrv.c index 8ba9c6d..72d0b81 100644 --- a/c/src/lib/libbsp/arm/gba/clock/clockdrv.c +++

[PATCH 15/52] arm/gdbarmsim/irq/irq.c: Fix warnings

--- c/src/lib/libbsp/arm/gdbarmsim/irq/irq.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/c/src/lib/libbsp/arm/gdbarmsim/irq/irq.c b/c/src/lib/libbsp/arm/gdbarmsim/irq/irq.c index 1d373c8..79665b4 100644 --- a/c/src/lib/libbsp/arm/gdbarmsim/irq/irq.c +++

[PATCH 24/52] m68k/idp/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/m68k/idp/startup/bspstart.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/c/src/lib/libbsp/m68k/idp/startup/bspstart.c b/c/src/lib/libbsp/m68k/idp/startup/bspstart.c index 5c3ed75..1d8d103 100644 ---

[PATCH 10/52] gen83xx/console/console-config.c: Fix warnings

--- c/src/lib/libbsp/powerpc/gen83xx/console/console-config.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/gen83xx/console/console-config.c b/c/src/lib/libbsp/powerpc/gen83xx/console/console-config.c index b3ccb9c..e3e6098 100644 ---

[PATCH 09/52] nios2-iic-irq.c: Fix warnings

--- cpukit/score/cpu/nios2/nios2-iic-irq.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/cpukit/score/cpu/nios2/nios2-iic-irq.c b/cpukit/score/cpu/nios2/nios2-iic-irq.c index aca25c7..75d533d 100644 --- a/cpukit/score/cpu/nios2/nios2-iic-irq.c +++

[PATCH 22/52] bfin/eZKit533/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/bfin/eZKit533/startup/bspstart.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/c/src/lib/libbsp/bfin/eZKit533/startup/bspstart.c b/c/src/lib/libbsp/bfin/eZKit533/startup/bspstart.c index 63c2d66..503fb6e 100644 ---

[PATCH 18/52] arm/shared/abort/abort.c: Fix warnings and clean up

--- c/src/lib/libbsp/arm/shared/abort/abort.c | 44 +++ 1 file changed, 27 insertions(+), 17 deletions(-) diff --git a/c/src/lib/libbsp/arm/shared/abort/abort.c b/c/src/lib/libbsp/arm/shared/abort/abort.c index a686a62..05432f8 100644 ---

[PATCH 23/52] m68k/av5282/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/m68k/av5282/startup/bspstart.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/c/src/lib/libbsp/m68k/av5282/startup/bspstart.c b/c/src/lib/libbsp/m68k/av5282/startup/bspstart.c index 9794ddb..e85ea4c 100644 ---

[PATCH 19/52] bfin/TLL6527M/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/bfin/TLL6527M/startup/bspstart.c | 16 +--- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/c/src/lib/libbsp/bfin/TLL6527M/startup/bspstart.c b/c/src/lib/libbsp/bfin/TLL6527M/startup/bspstart.c index c900b76..38c2df4 100644 ---

[PATCH 13/52] arm/gdbarmsim/console/console-io.c: Fix warnings

--- c/src/lib/libbsp/arm/gdbarmsim/console/console-io.c | 8 1 file changed, 8 insertions(+) diff --git a/c/src/lib/libbsp/arm/gdbarmsim/console/console-io.c b/c/src/lib/libbsp/arm/gdbarmsim/console/console-io.c index d5ac5e8..46db250 100644 ---

[PATCH 33/52] powerpc/beatnik/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/beatnik/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/beatnik/startup/bspstart.c b/c/src/lib/libbsp/powerpc/beatnik/startup/bspstart.c index d2501b5..e4b19ad 100644 ---

[PATCH 31/52] m68k/mvme162/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/m68k/mvme162/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/m68k/mvme162/startup/bspstart.c b/c/src/lib/libbsp/m68k/mvme162/startup/bspstart.c index 8ffee61..2068049 100644 ---

[PATCH 29/52] m68k/mvme147/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/m68k/mvme147/startup/bspstart.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/c/src/lib/libbsp/m68k/mvme147/startup/bspstart.c b/c/src/lib/libbsp/m68k/mvme147/startup/bspstart.c index 1188291..cb5f84a 100644 ---

[PATCH 35/52] powerpc/haleakala/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/haleakala/startup/bspstart.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/haleakala/startup/bspstart.c b/c/src/lib/libbsp/powerpc/haleakala/startup/bspstart.c index 3dfecb6..ef015fc 100644 ---

[PATCH 34/52] powerpc/ep1a/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/ep1a/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/ep1a/startup/bspstart.c b/c/src/lib/libbsp/powerpc/ep1a/startup/bspstart.c index 92b466c..158d0dc 100644 ---

[PATCH 42/52] powerpc/ss555/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/ss555/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/ss555/startup/bspstart.c b/c/src/lib/libbsp/powerpc/ss555/startup/bspstart.c index 59aaf29..ca9b1d3 100644 ---

[PATCH 41/52] powerpc/shared/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/shared/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/shared/startup/bspstart.c b/c/src/lib/libbsp/powerpc/shared/startup/bspstart.c index c2c6949..7fff1b9 100644 ---

[PATCH 36/52] powerpc/mbx8xx/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/mbx8xx/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/mbx8xx/startup/bspstart.c b/c/src/lib/libbsp/powerpc/mbx8xx/startup/bspstart.c index bb735fd..8ba87c4 100644 ---

[PATCH 47/52] m68k/rtems/score/cpu.h: _CPU_Context_Restart_self needed RTEMS_COMPILER_NO_RETURN_ATTRIBUTE

--- cpukit/score/cpu/m68k/rtems/score/cpu.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cpukit/score/cpu/m68k/rtems/score/cpu.h b/cpukit/score/cpu/m68k/rtems/score/cpu.h index 7d164f3..c36b2f1 100644 --- a/cpukit/score/cpu/m68k/rtems/score/cpu.h +++

[PATCH 44/52] sparc64/shared/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/sparc64/shared/startup/bspstart.c | 21 ++--- 1 file changed, 6 insertions(+), 15 deletions(-) diff --git a/c/src/lib/libbsp/sparc64/shared/startup/bspstart.c b/c/src/lib/libbsp/sparc64/shared/startup/bspstart.c index 25d4276..f6c190b 100644 ---

[PATCH 40/52] powerpc/score603e/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/score603e/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/score603e/startup/bspstart.c b/c/src/lib/libbsp/powerpc/score603e/startup/bspstart.c index 63569ff..e5c860f 100644 ---

[PATCH 38/52] powerpc/mvme3100/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

--- c/src/lib/libbsp/powerpc/mvme3100/startup/bspstart.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/mvme3100/startup/bspstart.c b/c/src/lib/libbsp/powerpc/mvme3100/startup/bspstart.c index 4fa9951..b38995f 100644 ---

[PATCH 48/52] samples/unlimited: Fix printf() warning and clean up

--- testsuites/samples/unlimited/init.c | 23 ++- testsuites/samples/unlimited/system.h | 2 +- testsuites/samples/unlimited/test1.c | 2 +- testsuites/samples/unlimited/test2.c | 28 testsuites/samples/unlimited/test3.c | 2 +- 5 files

Re: [PATCH 04/52] gdbv850sim/console/console-io.c: Fix warnings

On Wed, Oct 8, 2014 at 3:15 PM, Joel Sherrill joel.sherr...@oarcorp.com wrote: --- c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c

Re: [PATCH 18/52] arm/shared/abort/abort.c: Fix warnings and clean up

On Wed, Oct 8, 2014 at 3:16 PM, Joel Sherrill joel.sherr...@oarcorp.com wrote: --- c/src/lib/libbsp/arm/shared/abort/abort.c | 44 +++ 1 file changed, 27 insertions(+), 17 deletions(-) diff --git a/c/src/lib/libbsp/arm/shared/abort/abort.c

Re: [PATCH 04/52] gdbv850sim/console/console-io.c: Fix warnings

On 10/8/2014 3:10 PM, Gedare Bloom wrote: On Wed, Oct 8, 2014 at 3:15 PM, Joel Sherrill joel.sherr...@oarcorp.com wrote: --- c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 48/52] samples/unlimited: Fix printf() warning and clean up

On Wed, Oct 8, 2014 at 3:16 PM, Joel Sherrill joel.sherr...@oarcorp.com wrote: --- testsuites/samples/unlimited/init.c | 23 ++- testsuites/samples/unlimited/system.h | 2 +- testsuites/samples/unlimited/test1.c | 2 +- testsuites/samples/unlimited/test2.c | 28

Re: [PATCH 35/52] powerpc/haleakala/startup/bspstart.c: Add include of bsp/bootcard.h to fix warning and clean up

Fixed. Keep smacking me. On 10/8/2014 3:17 PM, Gedare Bloom wrote: On Wed, Oct 8, 2014 at 3:16 PM, Joel Sherrill joel.sherr...@oarcorp.com wrote: --- c/src/lib/libbsp/powerpc/haleakala/startup/bspstart.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git

Re: [PATCH 48/52] samples/unlimited: Fix printf() warning and clean up

On 10/8/2014 3:20 PM, Gedare Bloom wrote: On Wed, Oct 8, 2014 at 3:16 PM, Joel Sherrill joel.sherr...@oarcorp.com wrote: --- testsuites/samples/unlimited/init.c | 23 ++- testsuites/samples/unlimited/system.h | 2 +- testsuites/samples/unlimited/test1.c | 2 +-

Re: [PATCH 50/52] cpukit/score/src/ts64*: Return a value from non-void function

Adding Hesham On 10/8/2014 3:22 PM, Gedare Bloom wrote: How did this stuff ever work, is it tested anywhere? This only showed up on the or1k build. I can only hope this explained some of his failures. Or.. the return value was just laying around in a register. :) -Gedare On Wed, Oct 8, 2014

Re: GCC 4.9.1 changes.

[ Add AG for the moxie issue ] On 9/10/2014 8:31 am, Joel Sherrill wrote: On 10/5/2014 1:23 AM, Chris Johns wrote: Hi, There are a few issues with 4.9.1 which need to be looked at. I have not attempted a Windows build yet. My question is: Do we move the architectures that *do* build to