Hi Albert,
On Thursday, April 4, 2013 11:07:23 AM, Albert ARIBAUD wrote:
Hi Benoît,
On Wed, 3 Apr 2013 08:30:12 +0200 (CEST), Benoît Thébaudeau benoit.thebaudeau@advansee.com wrote:
The resulting HEAD is identical to yours, except for board/freescale/mx31ads/u-boot.lds in which you had removed the following unrelated line for 30/30: __bss_end = .;
Regarding this line, it is also present in a few other .lds, as well as the following ones: KEEP(*(.__bss_start)); KEEP(*(__bss_end));
However, the end section is named .__bss_end in arch/arm/lib/bss.c, so there is perhaps something wrong here, unless you did that on purpose because of the __bss_end test at the end of the linker script. I had noticed that, but I let you handle it. If something needs to be changed here, please do it after my series if possible in order to avoid a v11 because of newer rebase conflicts. ;)
I have a patch ready and currently undergoing build and run tests. As per http://www.denx.de/wiki/view/U-Boot/Patches#Review_Process_Git_Tags, I must ask your permission to add your 'Reported-By:' in my patch. Can I?
Sure: Reported-by: Benoît Thébaudeau benoit.thebaudeau@advansee.com
Best regards, Benoît