Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-04 Thread Wolfgang Denk
Dear Benoît Thébaudeau, In message <356588499.3415554.1346682315106.javamail.r...@advansee.com> you wrote: > > > So NAK from me, too. > > OK, but do you agree with the following that Mike and me agreed on after that? > http://lists.denx.de/pipermail/u-boot/2012-August/130650.html > http://lists.

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-03 Thread Graeme Russ
Hi Wolfgang, On Tue, Sep 4, 2012 at 4:41 PM, Wolfgang Denk wrote: > Dear Graeme Russ, > > Well, it appears awkward to me to use a completely unrelated command > for such "testing". The same result should be available (much more > straightforward) with: > > # dcache on > # dcache

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-03 Thread Wolfgang Denk
Dear Graeme Russ, In message you wrote: > > > If this line is overlooked, it's the same as not having it. If it is not > > overlooked, it is useful both to detail the test conditions and as a > > reminder > > not to do stupid things. This line does not prevent users from doing any > > manual

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-03 Thread Graeme Russ
Hi Benoît, On Tue, Sep 4, 2012 at 7:14 AM, Benoît Thébaudeau wrote: > Hi Albert, > > On Monday, September 3, 2012 6:50:14 PM, Albert ARIBAUD wrote: >> Hi Benoît, >> >> On Mon, 3 Sep 2012 16:25:15 +0200 (CEST), Benoît Thébaudeau >> wrote: >> >> > Dear Wolfgang Denk, >> > >> > On Sunday, September

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-03 Thread Benoît Thébaudeau
Hi Albert, On Monday, September 3, 2012 6:50:14 PM, Albert ARIBAUD wrote: > Hi Benoît, > > On Mon, 3 Sep 2012 16:25:15 +0200 (CEST), Benoît Thébaudeau > wrote: > > > Dear Wolfgang Denk, > > > > On Sunday, September 2, 2012 6:30:23 PM, Wolfgang Denk wrote: > > > Dear Benoît Thébaudeau, > > > >

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-03 Thread Albert ARIBAUD
Hi Benoît, On Mon, 3 Sep 2012 16:25:15 +0200 (CEST), Benoît Thébaudeau wrote: > Dear Wolfgang Denk, > > On Sunday, September 2, 2012 6:30:23 PM, Wolfgang Denk wrote: > > Dear Benoît Thébaudeau, > > > > In message > > <1725235724.2300239.1344694624384.javamail.r...@advansee.com> you > > wrote:

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-03 Thread Benoît Thébaudeau
Dear Wolfgang Denk, On Sunday, September 2, 2012 6:30:23 PM, Wolfgang Denk wrote: > Dear Benoît Thébaudeau, > > In message > <1725235724.2300239.1344694624384.javamail.r...@advansee.com> you > wrote: > > > > On 08/11/2012 05:18 AM, Mike Frysinger wrote: > ... > > > if you want dcache disabled, t

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-09-02 Thread Wolfgang Denk
Dear Benoît Thébaudeau, In message <1725235724.2300239.1344694624384.javamail.r...@advansee.com> you wrote: > > On 08/11/2012 05:18 AM, Mike Frysinger wrote: ... > > if you want dcache disabled, then why don't you run `dcache off` > > first ? i > > think it's useful to be able to do both, and f

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-13 Thread Albert ARIBAUD
Hi Mike, On Sat, 11 Aug 2012 16:15:16 -0400, Mike Frysinger wrote: > On Saturday 11 August 2012 16:05:36 Benoît Thébaudeau wrote: > > On 08/11/2012 21:47, Andrew Dyer wrote: > > > I agree with Mike, use the current dcache settings. U-boot has > > > always assumed the user knew what they were doin

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Benoît Thébaudeau
On Saturday 11 August 2012 22:15:16 Mike Frysinger wrote: > On Saturday 11 August 2012 16:05:36 Benoît Thébaudeau wrote: > > On 08/11/2012 21:47, Andrew Dyer wrote: > > > I agree with Mike, use the current dcache settings. U-boot has > > > always > > > assumed the user knew what they were doing. >

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Mike Frysinger
On Saturday 11 August 2012 16:05:36 Benoît Thébaudeau wrote: > On 08/11/2012 21:47, Andrew Dyer wrote: > > I agree with Mike, use the current dcache settings. U-boot has always > > assumed the user knew what they were doing. > > If you want to print a small message with the dcache setting that > >

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Benoît Thébaudeau
On 08/11/2012 21:47, Andrew Dyer wrote: > I agree with Mike, use the current dcache settings. U-boot has always > assumed the user knew what they were doing. > If you want to print a small message with the dcache setting that > makes sense to me, but no big warning. Then, something like the follow

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Benoît Thébaudeau
Hi Mike, On 08/11/2012 19:50, Mike Frysinger wrote: > On Saturday 11 August 2012 10:17:04 Benoît Thébaudeau wrote: > > On 08/11/2012 05:18 AM, Mike Frysinger wrote: > > > On Friday 10 August 2012 15:16:15 Benoît Thébaudeau wrote: > > > > mtest is supposed to test many types of memory accesses in m

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Mike Frysinger
On Saturday 11 August 2012 10:17:04 Benoît Thébaudeau wrote: > On 08/11/2012 05:18 AM, Mike Frysinger wrote: > > On Friday 10 August 2012 15:16:15 Benoît Thébaudeau wrote: > > > mtest is supposed to test many types of memory accesses in many > > > different > > > conditions. If dcache is enabled, m

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Benoît Thébaudeau
Hi Mike, On 08/11/2012 05:18 AM, Mike Frysinger wrote: > On Friday 10 August 2012 15:16:15 Benoît Thébaudeau wrote: > > mtest is supposed to test many types of memory accesses in many > > different > > conditions. If dcache is enabled, memory accesses are likely > > bursts, and > > some memory acc

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-10 Thread Mike Frysinger
On Friday 10 August 2012 15:16:15 Benoît Thébaudeau wrote: > mtest is supposed to test many types of memory accesses in many different > conditions. If dcache is enabled, memory accesses are likely bursts, and > some memory accesses are simply skipped. Hence, dcache should be disabled > during mtes

[U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-10 Thread Benoît Thébaudeau
mtest is supposed to test many types of memory accesses in many different conditions. If dcache is enabled, memory accesses are likely bursts, and some memory accesses are simply skipped. Hence, dcache should be disabled during mtest operation so that what mtest actually tests is not masked by dcac