public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Dirk Behme <dirk.behme@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V3 3/3] mkimage: adding support for Davinci AIS image
Date: Sun, 30 Oct 2011 09:23:05 +0100	[thread overview]
Message-ID: <4EAD0969.2030209@googlemail.com> (raw)
In-Reply-To: <4EACFCCB.5030609@denx.de>

On 30.10.2011 08:29, stefano babic wrote:
> Am 29/10/2011 18:51, schrieb Dirk Behme:
>
>>
>> Hmm, using recent git head this results in
>
> Hi Remy,
>
>>
>> aisimage.c: In function ?aisimage_generate?:
>> aisimage.c:365: warning: ?tsize? may be used uninitialized in this function
>>
>> for each board I build :(
>>
>> E.g.
>>
>> Configuring for omap3_evm board...
>> aisimage.c: In function ?aisimage_generate?:
>> aisimage.c:365: warning: ?tsize? may be used uninitialized in this function
>>     text    data     bss     dec     hex filename
>>   256063    7512  259120  522695   7f9c7 ./u-boot
>
> Apart from the fact that the tsize parameter is not needed anymore and I
> will fix it, can I ask you which toolchain and which gcc have you used ?

Seems I still have the good old

gcc version 4.3.3 (Sourcery G++ Lite 2009q1-203)

in my path.

Best regards

Dirk

> I have tried with a very old gcc 4.2.2 (ELDK-4.2) and gcc 4.6.1
> (ELDK-5.1). With the last one I get a warning, but for another reason:
>
> ./MAKEALL omap3_evm
> Configuring for omap3_evm board...
> musb_hcd.c: In function 'submit_control_msg':
> musb_hcd.c:851:6: warning: variable 'csr' set but not used
> [-Wunused-but-set-variable]
>     text	   data	    bss	    dec	    hex	filename
>   250892	   7472	 259120	 517484	  7e56c	./u-boot
>
> No warning with older gcc.
>
> Stefano
>

  reply	other threads:[~2011-10-30  8:23 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-16  9:50 [U-Boot] [PATCH 1/3] mkimage: ublimage must return if the header is not verified Stefano Babic
2011-09-16  9:50 ` [U-Boot] [PATCH 2/3] mkimage: Add variable lenght header support Stefano Babic
2011-10-05 18:56   ` Wolfgang Denk
2011-09-16  9:50 ` [U-Boot] [PATCH 3/3] mkimage: adding support for Davinci AIS image Stefano Babic
2011-10-05 18:56   ` Wolfgang Denk
2011-10-05 20:21     ` Wolfgang Denk
2011-09-16 10:03 ` [U-Boot] [PATCH 1/3] mkimage: ublimage must return if the header is not verified Heiko Schocher
2011-09-22 10:25 ` [U-Boot] [PATCH V2 " Stefano Babic
2011-10-05 18:55   ` Wolfgang Denk
2011-10-06  8:20 ` [U-Boot] [PATCH v2 3/3] mkimage: adding support for Davinci AIS image Stefano Babic
2011-10-15 20:23   ` Wolfgang Denk
2011-10-17  8:21     ` Stefano Babic
2011-10-17  9:18       ` Wolfgang Denk
2011-10-17 10:07 ` [U-Boot] [PATCH V3 " Stefano Babic
2011-10-23 21:08   ` Wolfgang Denk
2011-10-29 16:51     ` Dirk Behme
2011-10-30  7:29       ` stefano babic
2011-10-30  8:23         ` Dirk Behme [this message]
2011-10-30 13:14           ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EAD0969.2030209@googlemail.com \
    --to=dirk.behme@googlemail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox