[U-Boot] [PATCH v1] Refactor linker-generated arrays

Andreas Bießmann
2013-02-18T10:39:15Z

Hi Albert,

On 02/16/2013 07:20 PM, Albert ARIBAUD wrote:
> Hi Andreas,
>
> On Mon, 04 Feb 2013 13:41:09 +0100, "Andreas Bie?mann"
> <andreas.devel@googlemail.com> wrote:
>
>> Hi Albert,
>>
>> On 02.02.2013 18:02, Albert ARIBAUD wrote:

<snip strict aliasing error on gcc-4.4>

> I have dug into it and found a way to avoid GCC 4.4 or below to warn
> about aliasing, by replacing 'struct {}' with 'char[0]' as the
> 0-byte-size type.
>
> I still have some warnings through, regarding some regions not being
> declared:
>
> avr32-ld:built in linker script:15: warning: memory region `FLASH' not
> declared
> avr32-ld:built in linker script:69: warning: memory region
> `CPUSRAM' not declared

I assume you use Mike Frysingers precompiled avr32 toolchain. I know
about that warnings and beware, these toolchain produce defective
binaries! The u-boot does not relocate itself properly with these newlib
toolchains (also the atmel provided one).

> It appears 'normal' in that without my patch, the same error occurs;
> but I'd prefer that you confirm whether you have the same warnings on
> your side.

It's ok so far, the arm-linux toolchain I have do not produce these
warnings. Kan you provide the patch so I will do a runtime test.

Best regards

Andreas Bie?mann