Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-12-08 Thread Lukasz Majewski
Hi Albert, > Hello Lukasz, > > Thank you and Guillaume for the indications. So gcc 4.7.2 and 4.8.1 > should exhibit the problem. > > Apparently, 4.7.4 and 4.8.3 don't. :( > > /me goes and fetches a 4.8.1 somewhere. > > Amicalement, Is there any progress with preparing fix for this warning? -

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-24 Thread Albert ARIBAUD
Hello Lukasz, Thank you and Guillaume for the indications. So gcc 4.7.2 and 4.8.1 should exhibit the problem. Apparently, 4.7.4 and 4.8.3 don't. :( /me goes and fetches a 4.8.1 somewhere. Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.d

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-24 Thread Guillaume Gardet
Hi, Le 23/11/2014 18:38, Lukasz Majewski a écrit : Hi Albert, Sorry for a late reply. Hello Lukasz, On Sat, 22 Nov 2014 07:56:35 +0100, Lukasz Majewski wrote: Agreed in general, but not for this one, since "fixing" is the carpet, I assume that you are presenting below an answer to a "gen

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-24 Thread Lukasz Majewski
Hi Albert, > Hi Albert, > > Sorry for a late reply. > > > Hello Lukasz, > > > > On Sat, 22 Nov 2014 07:56:35 +0100, Lukasz Majewski > > wrote: > > > > > > Agreed in general, but not for this one, since "fixing" is the > > > > carpet, > > > > > > I assume that you are presenting below an ans

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-23 Thread Lukasz Majewski
Hi Albert, Sorry for a late reply. > Hello Lukasz, > > On Sat, 22 Nov 2014 07:56:35 +0100, Lukasz Majewski > wrote: > > > > Agreed in general, but not for this one, since "fixing" is the > > > carpet, > > > > I assume that you are presenting below an answer to a "general" > > case. > > > >

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-22 Thread Albert ARIBAUD
Hello Lukasz, On Sat, 22 Nov 2014 07:56:35 +0100, Lukasz Majewski wrote: > > Agreed in general, but not for this one, since "fixing" is the > > carpet, > > I assume that you are presenting below an answer to a "general" case. > > However, as Thomas pointed out earlier, this "fix" is perfectly

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-22 Thread Lukasz Majewski
Hi Jeroen > Hello Albert, > > On 21-11-14 16:30, Albert ARIBAUD wrote: > > On Fri, 21 Nov 2014 13:34:41 +0100, Jeroen Hofstee > > wrote: > > > But oh well, if it fixes a warning :-) > >>> I didn't claim that there is a bug in the code :-). > >>> > >>> I just get annoying when on my continu

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Albert ARIBAUD
Hello Lukasz, On Fri, 21 Nov 2014 09:22:43 +0100, Lukasz Majewski wrote: > When building with my toolchain (4.8.2): > CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.18-binutils-2.24-kernel-3.12-sanitized/bin/arm-v7a-linux-g

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Jeroen Hofstee
Hello Albert, On 21-11-14 16:30, Albert ARIBAUD wrote: On Fri, 21 Nov 2014 13:34:41 +0100, Jeroen Hofstee wrote: But oh well, if it fixes a warning :-) I didn't claim that there is a bug in the code :-). I just get annoying when on my continuous integration script I see the same warning for

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Albert ARIBAUD
Hello Jeroen, On Fri, 21 Nov 2014 13:34:41 +0100, Jeroen Hofstee wrote: > >> But oh well, if it fixes a warning :-) > > I didn't claim that there is a bug in the code :-). > > > > I just get annoying when on my continuous integration script I see the > > same warning for all cross compiled board

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Jeroen Hofstee
Hi, On 21-11-14 10:20, Lukasz Majewski wrote: Hi Thomas, Dear Lukasz Majewski, Thanks for your patch. On Fri, 21 Nov 2014 09:22:43 +0100, Lukasz Majewski wrote: When building with my toolchain (4.8.2): CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Heiko Schocher
Hello Lukasz, Am 21.11.2014 09:22, schrieb Lukasz Majewski: When building with my toolchain (4.8.2): CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.18-binutils-2.24-kernel-3.12-sanitized/bin/arm-v7a-linux-gnueabi- I see f

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Stefan Roese
On 21.11.2014 09:22, Lukasz Majewski wrote: When building with my toolchain (4.8.2): CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.18-binutils-2.24-kernel-3.12-sanitized/bin/arm-v7a-linux-gnueabi- I see following WARNING:

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Lukasz Majewski
Hi Thomas, > Dear Lukasz Majewski, > > Thanks for your patch. > > On Fri, 21 Nov 2014 09:22:43 +0100, Lukasz Majewski wrote: > > When building with my toolchain (4.8.2): > > CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.1

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Thomas Petazzoni
Dear Lukasz Majewski, Thanks for your patch. On Fri, 21 Nov 2014 09:22:43 +0100, Lukasz Majewski wrote: > When building with my toolchain (4.8.2): > CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.18-binutils-2.24-kernel-3.1

[U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Lukasz Majewski
When building with my toolchain (4.8.2): CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-2013.12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.18-binutils-2.24-kernel-3.12-sanitized/bin/arm-v7a-linux-gnueabi- I see following WARNING: tools/kwbimage.c: In function "kwbimage_set_h