Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-07 Thread Peter Korsgaard
> "Kim" == Kim Phillips writes: Hi, >> Sure - But if we're adding _EARLY to all the configs, shouldn't we just >> make _EARLY default (or simply get rid of the late variant?) Kim> probably the latter. Thanks! Great - I'll send an updated patch just doing that then instead. >> Also fo

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-07 Thread Peter Korsgaard
> "Kim" == Kim Phillips writes: Hi, Kim> ok let's leave it as _EARLY then. wrt boards in current u-boot, can Kim> you add the _EARLY define in their default config files? Sure - But if we're adding _EARLY to all the configs, shouldn't we just make _EARLY default (or simply get rid of the

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-07 Thread Kim Phillips
On Mon, 7 Dec 2009 22:10:49 +0100 Peter Korsgaard wrote: > > "Kim" == Kim Phillips writes: > Kim> ok let's leave it as _EARLY then. wrt boards in current u-boot, can > Kim> you add the _EARLY define in their default config files? > > Sure - But if we're adding _EARLY to all the configs,

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-07 Thread Kim Phillips
On Sat, 5 Dec 2009 21:27:30 +0100 Peter Korsgaard wrote: > > "Kim" == Kim Phillips writes: > > Kim> On Fri, 20 Nov 2009 12:42:43 +0100 > Kim> Peter Korsgaard wrote: > > >> E.G. on a 8347 board a bootup time increase of ~600ms has been observed: > > Kim> heh, even more on an 8313! Th

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-05 Thread Peter Korsgaard
> "Kim" == Kim Phillips writes: Kim> On Fri, 20 Nov 2009 12:42:43 +0100 Kim> Peter Korsgaard wrote: >> E.G. on a 8347 board a bootup time increase of ~600ms has been observed: Kim> heh, even more on an 8313! Thanks for this - I hadn't realized the Kim> difference was so large (or neg

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-04 Thread Kim Phillips
On Fri, 20 Nov 2009 12:42:43 +0100 Peter Korsgaard wrote: > E.G. on a 8347 board a bootup time increase of ~600ms has been observed: heh, even more on an 8313! Thanks for this - I hadn't realized the difference was so large (or neglected it since the move to init_r was done at the last moment).

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-04 Thread Wolfgang Denk
Dear Kim, In message <20091204185150.1e2939e8.kim.phill...@freescale.com> you wrote: > > > It seems Kim is on vacation or something like that. > > I was, but I'm back now. I hope you had a good time. Welcome back. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfga

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-04 Thread Kim Phillips
On Sat, 5 Dec 2009 01:29:25 +0100 Wolfgang Denk wrote: > Dear Peter Korsgaard, > > In message <87pr7472vw@macbook.be.48ers.dk> you wrote: > > > "Peter" == Peter Korsgaard writes: > > > > Peter> Commit c7190f02 (retain POR values of non-configured ACR, SPCR, > > SCCR, > > Peter> and

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-12-04 Thread Wolfgang Denk
Dear Peter Korsgaard, In message <87pr7472vw@macbook.be.48ers.dk> you wrote: > > "Peter" == Peter Korsgaard writes: > > Peter> Commit c7190f02 (retain POR values of non-configured ACR, SPCR, SCCR, > Peter> and LCRR bitfields) moved the LCRR assignment to after relocation > Peter> to R

Re: [U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-11-27 Thread Peter Korsgaard
> "Peter" == Peter Korsgaard writes: Peter> Commit c7190f02 (retain POR values of non-configured ACR, SPCR, SCCR, Peter> and LCRR bitfields) moved the LCRR assignment to after relocation Peter> to RAM because of the potential problem with changing the local bus Peter> clock while executin

[U-Boot] [PATCH, REGRESSION] mpc83xx: provide option to set LCRR early

2009-11-20 Thread Peter Korsgaard
Commit c7190f02 (retain POR values of non-configured ACR, SPCR, SCCR, and LCRR bitfields) moved the LCRR assignment to after relocation to RAM because of the potential problem with changing the local bus clock while executing from flash. For some boards this isn't needed (E.G. when running from a