public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Benoît Thébaudeau" <benoit.thebaudeau@advansee.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] mtest: Disable dcache during test
Date: Mon, 3 Sep 2012 16:25:15 +0200 (CEST)	[thread overview]
Message-ID: <356588499.3415554.1346682315106.JavaMail.root@advansee.com> (raw)
In-Reply-To: <20120902163023.6AC19203B2F@gemini.denx.de>

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.root@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 forcing it one way
> > > is
> > > wrong.
> > > 
> > > thus, NAK from me.
> > > -mike
> > 
> > Because you will very likely trust mtest and forget about running
> > `dcache off`
> > first, so you may then be happy about falsely positive mtest
> > results. Moreover,
> > I can't find any sense or usefulness in running mtest with dcache
> > enabled.
> 
> I agree with Mike.
> 
> "UNIX was not designed to stop you from doing stupid things,  because
> that would also stop you from doing clever things."       - Doug Gwyn
> 
> 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.denx.de/pipermail/u-boot/2012-August/130726.html
http://patchwork.ozlabs.org/patch/176909/

Best regards,
Beno?t

  reply	other threads:[~2012-09-03 14:25 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-10 19:16 [U-Boot] [PATCH] mtest: Disable dcache during test Benoît Thébaudeau
2012-08-11  3:18 ` Mike Frysinger
2012-08-11 14:17   ` Benoît Thébaudeau
2012-08-11 17:50     ` Mike Frysinger
2012-08-11 18:49       ` Benoît Thébaudeau
2012-09-02 16:30     ` Wolfgang Denk
2012-09-03 14:25       ` Benoît Thébaudeau [this message]
2012-09-03 16:50         ` Albert ARIBAUD
2012-09-03 21:14           ` Benoît Thébaudeau
2012-09-03 23:30             ` Graeme Russ
2012-09-04  6:41               ` Wolfgang Denk
2012-09-04  6:43                 ` Graeme Russ
2012-09-04  7:23         ` Wolfgang Denk
     [not found] <CANYL96C2=fYHoTAp0SQird-Yz1W2JPq22U=3q4ZFKgwomXxODA@mail.gmail.com>
2012-08-11 20:05 ` Benoît Thébaudeau
2012-08-11 20:15   ` Mike Frysinger
2012-08-11 20:25     ` Benoît Thébaudeau
2012-08-14  6:27     ` Albert ARIBAUD

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=356588499.3415554.1346682315106.JavaMail.root@advansee.com \
    --to=benoit.thebaudeau@advansee.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox