On Wed, Sep 03, 2014 at 02:53:17AM +0200, Benoît Thébaudeau wrote: > Hi Helmut, all, > > On Tue, Aug 19, 2014 at 10:55 PM, Benoît Thébaudeau > <benoit.thebaudeau....@gmail.com> wrote: > > On Fri, Aug 15, 2014 at 7:45 PM, Magnus Lilja <lilja.mag...@gmail.com> > > wrote: > >> On 13 August 2014 14:01, Helmut Raiger <helmut.rai...@hale.at> wrote: > >>> On 08/05/2014 02:32 PM, Magnus Lilja wrote: > >>>> I would expect Helmut to create a formal patch then I can test that > >>>> and add a Tested-by. > >>>> > >>> The problem is it does not work with only the 'b reset' change on my > >>> platform. > >>> Should I provide a patch with the nops and the question marks around them? > >>> It still could be a toolchain difference, mine is pretty old: > >>> > >>> $ arm-angstrom-linux-gnueabi-gcc --version > >>> arm-angstrom-linux-gnueabi-gcc (GCC) 4.7.2 > >>> Copyright (C) 2012 Free Software Foundation, Inc. > >>> > >>> When I objdump the elf file I can see the very same code in > >>> cpu_init_crit() > >>> as in start.S, > >>> whatever that might mean (objdump is from the same toolchain). > >> > >> I use an even older gcc so I don't think that's the problem. I use: > >> arm-none-linux-gnueabi-gcc (Sourcery CodeBench Lite 2011.09-70) 4.6.1 > >> > >> Not sure how you should proceed with the path. > > > > IMHO, the 'b reset' and the 'nop nop nop' are two different issues, so > > Helmut should create a formal patch for the 'b reset' issue right now, > > which will fix mx31pdk (and maybe other boards) for the release. Then, > > once the 'nop nop nop' issue has been resolved for TT-01 (cache issue > > or something else), another formal patch should be created for this > > issue, unless it is purely out of tree. > > v2014.10 is getting closer with the release of -rc2. It would be much > better to get mx31pdk fixed for this release. Helmut, can you send a > patch for the 'b reset' issue? If not, do you agree that someone else > (maybe the board maintainer: Magnus?) sends it with a 'Reported-by: > Helmut Raiger <helmut.rai...@hale.at>'?
Yes, please, I'd like to see this fixed for the release proper. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot