linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dave Martin <dave.martin@linaro.org>
To: Rob Herring <robherring2@gmail.com>
Cc: linux-arm-kernel@lists.infradead.org, patches@linaro.org,
	Anton Vorontsov <avorontsov@mvista.com>,
	Barry Song <baohua.song@csr.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Colin Cross <ccross@google.com>,
	Haojian Zhuang <haojian.zhuang@marvell.com>,
	John Linn <john.linn@xilinx.com>,
	Kukjin Kim <kgene.kim@samsung.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	linux-omap@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
	linux-sh@vger.kernel.org, linux-tegra@vger.kernel.org,
	Magnus Damm <magnus.damm@gmail.com>,
	Paul Mundt <lethal@linux-sh.org>, Pawel Moll <pawel.moll@arm.com>,
	Sascha Hauer <kernel@pengutronix.de>,
	Shawn Guo <shawn.guo@linaro.org>,
	Tony Lindgren <tony@atomide.com>,
	Will Deacon <will.deacon@arm.com>
Subject: Re: [PATCH v4 REPOST 4/5] highbank: Unconditionally require l2x0 L2 cache controller support
Date: Wed, 14 Dec 2011 13:55:13 +0000	[thread overview]
Message-ID: <20111214135513.GC2568@linaro.org> (raw)
In-Reply-To: <4EE8A69C.40304@gmail.com>

On Wed, Dec 14, 2011 at 07:37:32AM -0600, Rob Herring wrote:
> 
> On 12/14/2011 05:39 AM, Dave Martin wrote:
> > If running in the Normal World on a TrustZone-enabled SoC, Linux
> > does not have complete control over the L2 cache controller
> > configuration.  The kernel cannot work reliably on such platforms
> > without the l2x0 cache support code built in.
> > 
> > This patch unconditionally enables l2x0 support for the Highbank
> > SoC.
> > 
> > Thanks to Rob Herring for this suggestion. [1]
> > 
> > Signed-off-by: Dave Martin <dave.martin@linaro.org>
> > 
> > [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2011-November/074495.html
> 
> Doesn't this need to be above the SOB? Otherwise:

You may be right ... certainly I see no reason _not_ to change it.
So I'll change it.

> 
> Acked-by: Rob Herring <rob.herring@calxeda.com>

Thanks

---Dave

  reply	other threads:[~2011-12-14 13:55 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-14 11:39 [PATCH v4 REPOST 0/5] Refactor common Kconfigs for easier maintenance Dave Martin
2011-12-14 11:39 ` [PATCH v4 REPOST 1/5] ARM: l2x0/pl310: Refactor Kconfig to be more maintainable Dave Martin
2011-12-14 12:03   ` Anton Vorontsov
2011-12-14 18:15     ` Tony Lindgren
2011-12-14 11:39 ` [PATCH v4 REPOST 2/5] ARM: SMP: " Dave Martin
     [not found]   ` <1323862781-3465-3-git-send-email-dave.martin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-12-14 18:16     ` Tony Lindgren
2011-12-15 18:14     ` David Brown
2011-12-16  0:31     ` David Brown
2011-12-14 11:39 ` [PATCH v4 REPOST 3/5] omap4: Unconditionally require l2x0 L2 cache controller support Dave Martin
2011-12-14 18:14   ` Tony Lindgren
     [not found]     ` <20111214181425.GB32251-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2011-12-14 18:30       ` Dave Martin
2011-12-14 18:39         ` Tony Lindgren
     [not found]           ` <20111214183952.GG32251-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2011-12-14 21:01             ` Rob Herring
2011-12-14 21:48               ` Tony Lindgren
2011-12-14 11:39 ` [PATCH v4 REPOST 4/5] highbank: " Dave Martin
2011-12-14 13:37   ` Rob Herring
2011-12-14 13:55     ` Dave Martin [this message]
     [not found] ` <1323862781-3465-1-git-send-email-dave.martin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-12-14 11:39   ` [PATCH v4 REPOST 5/5] imx6q: Remove unconditional dependency on l2x0 L2 cache support Dave Martin
2011-12-14 13:26     ` Shawn Guo
2011-12-14 14:05       ` Richard Zhao
2011-12-14 15:01         ` Dave Martin
     [not found]           ` <20111214150119.GD2568-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-12-15  1:02             ` Richard Zhao
2011-12-15  1:46               ` Shawn Guo
2011-12-15  1:54                 ` Richard Zhao
     [not found]                   ` <20111215015414.GQ28768-iWYTGMXpHj9ITqJhDdzsOjpauB2SiJktrE5yTffgRl4@public.gmane.org>
2011-12-15 15:14                     ` Dave Martin

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=20111214135513.GC2568@linaro.org \
    --to=dave.martin@linaro.org \
    --cc=avorontsov@mvista.com \
    --cc=baohua.song@csr.com \
    --cc=catalin.marinas@arm.com \
    --cc=ccross@google.com \
    --cc=haojian.zhuang@marvell.com \
    --cc=john.linn@xilinx.com \
    --cc=kernel@pengutronix.de \
    --cc=kgene.kim@samsung.com \
    --cc=lethal@linux-sh.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=patches@linaro.org \
    --cc=pawel.moll@arm.com \
    --cc=robherring2@gmail.com \
    --cc=shawn.guo@linaro.org \
    --cc=tony@atomide.com \
    --cc=will.deacon@arm.com \
    /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;
as well as URLs for NNTP newsgroup(s).