[U-Boot] [PATCH V3 3/3] mkimage: adding support for Davinci AIS image

Dirk Behme
2011-10-30T08:23:05Z

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
>