From mboxrd@z Thu Jan  1 00:00:00 1970
From: =?UTF-8?B?QW5kcmVhcyBCaWXDn21hbm4=?= <andreas.devel@googlemail.com>
Date: Mon, 18 Feb 2013 11:39:15 +0100
Subject: [U-Boot] [PATCH v1] Refactor linker-generated arrays
In-Reply-To: <20130216192000.7434cbdc@lilith>
References: <1359824574-32627-1-git-send-email-albert.u.boot@aribaud.net>
	<1359824574-32627-2-git-send-email-albert.u.boot@aribaud.net>
	<510FAC65.7090301@gmail.com> <20130216192000.7434cbdc@lilith>
Message-ID: <512204D3.2020205@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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