Hi Benoît,

On Thu, 4 Apr 2013 16:05:27 +0200 (CEST), Benoît Thébaudeau
<benoit.thebaud...@advansee.com> wrote:

> 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.thebaud...@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.thebaud...@advansee.com>

Actually, I think you've just stated to our patchwork that your patch v9
18/30 fixes a bug reported by you... But never mind, v9 is superseded
now anyway, and I'll put your Reported-By manually in my BSS fix patch.

> Best regards,
> Benoît

Amicalement,
-- 
Albert.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to