All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] The cache flush using coprocessor must be in	lib_arm/cache-cp15.c
Date: Thu, 28 May 2009 21:28:36 +0200	[thread overview]
Message-ID: <20090528192836.GA1802@game.jcrosoft.org> (raw)
In-Reply-To: <b64afca20905280151p4ed19e2bq90b7b804356fc0ed@mail.gmail.com>

On 17:51 Thu 28 May     , Kim, Heung Jun wrote:
> Hi? I'm about to suggest one more thing related to "change cpu.c under
> cpu/arm_cortexa8 dir to common code.".
> 
> asm ("mcr p15, 0, %0, c7, c5, 0": :"r" (0));
> 
> It's the common feature all over the arm core, not only arm cortex A8.
> The common cache function is defined in the lib_arm/cache-cp15.c as you
> know. So, It's seems the better method that cache_flush() is moved to
> lib_arm/cache-cp15.c
NACK

to boot linux we are suppose to clean both cache I and DCache

the I/DCache flush is armv and arch dependant (example ARM7TDMI, armv4wt, armv3,
armv7, etc...)

so for now please let it in cpu/$(ARCH)/cpu.c

Best Regards,
J.

  parent reply	other threads:[~2009-05-28 19:28 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-28  8:51 [U-Boot] [PATCH] The cache flush using coprocessor must be in lib_arm/cache-cp15.c Kim, Heung Jun
2009-05-28 15:01 ` Dirk Behme
2009-05-28 19:28 ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2009-05-29  6:15   ` Dirk Behme
2009-05-29  6:44     ` Kim, Heung Jun
2009-05-29  7:07       ` Dirk Behme
2009-05-29  7:15         ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-29  7:38           ` Dirk Behme
2009-05-29  9:24             ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-29  9:48               ` Dirk Behme

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=20090528192836.GA1802@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.