From: gores@marvell.com (Siddarth Gore)
To: linux-arm-kernel@lists.infradead.org
Subject: L2 cache support for pxa16x
Date: Fri, 21 May 2010 15:14:06 +0530 [thread overview]
Message-ID: <1274435046.6926.69.camel@pe-dt434> (raw)
In-Reply-To: <AANLkTil839_xcOATFEFyQH5YVgWPwJcf31Wo1ZqmJ7Z6@mail.gmail.com>
On Thu, 2010-05-20 at 07:06 -0700, Eric Miao wrote:
> On Thu, May 20, 2010 at 6:17 PM, Siddarth Gore <gores@marvell.com> wrote:
> > On Wed, 2010-05-12 at 03:11 -0700, Eric Miao wrote:
> >> On Wed, May 12, 2010 at 11:56 AM, Siddarth Gore <gores@marvell.com> wrote:
> >> > Hi Eric/Haojian,
> >> >
> >> > Can the Tauros2 support be used for pxa168 as well? The one difference I
> >> > can see is that L2 Enable is in control register instead of extra
> >> > feature register. But rest of the things look very similar to me.
> >> >
> >
> > I tried doing this. It works when I enable L2 before turning the MMU on,
> > i.e. in __mohawk_setup
> >
> > But when I do the following in tauros2_init(), the kernel crashes.
> > 1. flush and disable dcache
> > 2. invalidate and disable icache
> > 3. drain write buffer
> > 4. invalidate TLB
> > 5. invalidate L2
> > 6. enable L2
> > 7. enable icache
> > 8. enable dcache
> >
> > I think the right place to enable L2 is in tauros2_init, so any idea
> > what I am doing wrong here?
> >
>
> I believe that was the reason why I didn't put tauros2 support to pxa168
> at the first place. And enabling L2 after MMU is enabled is supposed to
> be unsafe, which is mentioned in xscale3 manual, though not sure if that's
> the case for pxa168 as well.
>
I did not find it mentioned anywhere in the mohawk (PJ1) datasheet. Also
any idea why is this unsafe? I clean and disable the L1 cache first so
all the page tables, etc. will be backed up in main memory before
turning L2 on. plus invalidate the entire L2, so all the fetches will
first go to main memory.
other CPUs using tauros2 (also feroceon) turn L2 on after MMU, and they
work fine.
-siddarth
> > -siddarth
> >
> >
next prev parent reply other threads:[~2010-05-21 9:44 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-12 9:56 L2 cache support for pxa16x Siddarth Gore
2010-05-12 10:11 ` Eric Miao
2010-05-20 10:17 ` Siddarth Gore
2010-05-20 14:06 ` Eric Miao
2010-05-21 9:44 ` Siddarth Gore [this message]
2010-05-21 10:27 ` Eric Miao
2010-05-21 10:37 ` Siddarth Gore
2010-05-21 11:07 ` Haojian Zhuang
2010-05-21 11:30 ` Eric Miao
2010-05-21 11:05 ` Haojian Zhuang
2010-05-21 11:37 ` Eric Miao
2010-06-11 10:20 ` Siddarth Gore
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=1274435046.6926.69.camel@pe-dt434 \
--to=gores@marvell.com \
--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