Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 08:33:58 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201109261937.01263.marek.va...@gmail.com> you wrote: > > > What exactly are the advantages, and what the disadvantages? > > > > > > code size / memory footprint? > > > > I assume converting those t

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109261937.01263.marek.va...@gmail.com> you wrote: > > > What exactly are the advantages, and what the disadvantages? > > > > code size / memory footprint? > > I assume converting those to inline function will yield no memory usage > growth > at all. Please don

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 01:30:41 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201109261132.43616.marek.va...@gmail.com> you wrote: > > > Would it be better to change the __arch_putb macro into an extern > > > inline function instead which would catch these and future cases? >

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109261132.43616.marek.va...@gmail.com> you wrote: > > > Would it be better to change the __arch_putb macro into an extern inline > > function instead which would catch these and future cases? > > Yes, but you'll need to do that on a much larger scale. Is anyone up

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 11:44:22 AM Nick Thompson wrote: > On 26/09/11 10:32, Marek Vasut wrote: > > On Monday, September 26, 2011 11:26:51 AM Nick Thompson wrote: > >> On 26/09/11 03:06, Marek Vasut wrote: > >>> Signed-off-by: Marek Vasut > >>> --- > >>> > >>> arch/arm/include/asm/io.h |

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Nick Thompson
On 26/09/11 10:32, Marek Vasut wrote: > On Monday, September 26, 2011 11:26:51 AM Nick Thompson wrote: >> On 26/09/11 03:06, Marek Vasut wrote: >>> Signed-off-by: Marek Vasut >>> --- >>> >>> arch/arm/include/asm/io.h | 30 ++ >>> 1 files changed, 18 insertions(+), 12

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 11:26:51 AM Nick Thompson wrote: > On 26/09/11 03:06, Marek Vasut wrote: > > Signed-off-by: Marek Vasut > > --- > > > > arch/arm/include/asm/io.h | 30 ++ > > 1 files changed, 18 insertions(+), 12 deletions(-) > > > > diff --git a/arch

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Nick Thompson
On 26/09/11 03:06, Marek Vasut wrote: > Signed-off-by: Marek Vasut > --- > arch/arm/include/asm/io.h | 30 ++ > 1 files changed, 18 insertions(+), 12 deletions(-) > > diff --git a/arch/arm/include/asm/io.h b/arch/arm/include/asm/io.h > index 1fbc531..61f4987 100644 >

[U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-25 Thread Marek Vasut
Signed-off-by: Marek Vasut --- arch/arm/include/asm/io.h | 30 ++ 1 files changed, 18 insertions(+), 12 deletions(-) diff --git a/arch/arm/include/asm/io.h b/arch/arm/include/asm/io.h index 1fbc531..61f4987 100644 --- a/arch/arm/include/asm/io.h +++ b/arch/arm/inclu