From: nico@fluxnic.net (Nicolas Pitre)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] pxa: enable L2 if present in XSC3
Date: Wed, 30 Dec 2009 21:19:19 -0500 (EST) [thread overview]
Message-ID: <alpine.LFD.2.00.0912302115010.7330@xanadu.home> (raw)
In-Reply-To: <771cded00912301757t164b2f2cx829977547122eaff@mail.gmail.com>
On Wed, 30 Dec 2009, Haojian Zhuang wrote:
> On Wed, Dec 30, 2009 at 1:06 PM, Nicolas Pitre <nico@fluxnic.net> wrote:
> > On Wed, 30 Dec 2009, Eric Miao wrote:
> >
> >> On Wed, Dec 30, 2009 at 3:28 PM, Haojian Zhuang
> >> <haojian.zhuang@gmail.com> wrote:
> >> > From 2c4a7d0e907df9a762c5c9f0ec48b3b7860e1f7c Mon Sep 17 00:00:00 2001
> >> > From: Haojian Zhuang <haojian.zhuang@marvell.com>
> >> > Date: Wed, 30 Dec 2009 10:02:57 -0500
> >> > Subject: [PATCH] [ARM] pxa: enable L2 if present in XSC3
> >> >
> >> > Check whether L2 is present or not in XSC3. If it's present, enable L2
> >> > immediately.
> >> >
> >> > Disabling L2 after L2 is enabled that would result in unpredicatable behavior
> >> > of XSC3 processor.
> >> >
> >>
> >> You may want to enclose this with #ifdef CONFIG_CACHE_XSC3L2 .. #endif
> >> at least, to allow L2 to remain disabled even if L2 is present.
> >
> > Also... might be a good idea to invalidate L2 before enabling it.
> >
>
> XSC3 supports these L2 operations in below.
> 1) invalidate L2 cache line (parameter is MVA)
> 2) clean L2 cache line (parameter is MVA)
> 3) clean L2 cache line (parameter is set/way)
> 4) clean and invalidate L2 cache line (parameter is set/way)
>
> The most important thing is we couldn't call these L2 operations while
> L2 is disabled.
Well... The idea behind invalidating L2 before enabling it is to make
sure it doesn't contain random stuff. But if this is impossible to
perform while L2 is disabled then we can only assume it is always going
to be in a coherent state when it is enabled. Furthermore there is no
way to only invalidate the entire L2.
Nicolas
prev parent reply other threads:[~2009-12-31 2:19 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-30 7:28 [PATCH 2/2] pxa: enable L2 if present in XSC3 Haojian Zhuang
2009-12-30 8:50 ` Eric Miao
2009-12-30 9:05 ` Haojian Zhuang
2010-01-01 7:56 ` Eric Miao
2009-12-30 18:06 ` Nicolas Pitre
2009-12-31 1:57 ` Haojian Zhuang
2009-12-31 2:19 ` Nicolas Pitre [this message]
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=alpine.LFD.2.00.0912302115010.7330@xanadu.home \
--to=nico@fluxnic.net \
--cc=linux-arm-kernel@lists.infradead.org \
/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