All of lore.kernel.org
 help / color / mirror / Atom feed
From: jonathan.austin@arm.com (Jonathan Austin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/3] ARM: allow MULTIPLATFORM on no-MMU machines
Date: Tue, 01 Oct 2013 11:29:54 +0100	[thread overview]
Message-ID: <524AA422.9040106@arm.com> (raw)
In-Reply-To: <1380534577-15076-3-git-send-email-u.kleine-koenig@pengutronix.de>

On 30/09/13 10:49, Uwe Kleine-K?nig wrote:
> There is no technical reason to not use the benefits of MULTIPLATFORM on a
> no-MMU machine and my not-yet-mainline efm32 port does work just fine with
> it. So drop the dependency.
>
> Signed-off-by: Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>
> ---
>   arch/arm/Kconfig | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 0f63a70..d873a34 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -296,8 +296,7 @@ choice
>
>   config ARCH_MULTIPLATFORM
>   	bool "Allow multiple platforms to be selected"
> -	depends on MMU
> -	select ARM_PATCH_PHYS_VIRT
> +	select ARM_PATCH_PHYS_VIRT if MMU && !XIP_KERNEL
>   	select AUTO_ZRELADDR
>   	select COMMON_CLK
>   	select MULTI_IRQ_HANDLER
>


A patch much like this has been submitted in the past by Arnd and me, 
and Russell wasn't too happy with the idea because ARCH_MULTIPLATFORM 
doesn't really make sense for !MMU.

However, back then we were talking R/A class. For M-class the memory map 
is more clearly mandated, the idea of V7M platforms that are 
ARCH_MULTIPLATFORM compatible is much more feasible. One of the big 
issues for !MMU/ARCH_MULTIPLATFORM in the past was the need to select 
the vector configuration at build time - not something we need to worry 
about with V7M (where we have a configurable vector base), so from that 
point of view, this change makes sense.

But I'm not sure about the XIP_KERNEL part. Once you're using XIP_KERNEL 
again, multi-platform doesn't make much sense. Uwe, can you correct me 
on that?

Jonny

  reply	other threads:[~2013-10-01 10:29 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-30  9:49 [PATCH 0/3] v7-M support using MULTIPLATFORM Uwe Kleine-König
2013-09-30  9:49 ` [PATCH 1/3] ARM: drop !ARCH_MULTIPLATFORM dependency of XIP_KERNEL Uwe Kleine-König
2013-09-30  9:49 ` [PATCH 2/3] ARM: allow MULTIPLATFORM on no-MMU machines Uwe Kleine-König
2013-10-01 10:29   ` Jonathan Austin [this message]
2013-09-30  9:49 ` [PATCH 3/3] [RFC] ARM: prepare ARMv7-M for MULTIPLATFORM use Uwe Kleine-König
2013-09-30 10:34   ` Arnd Bergmann
2013-09-30 16:02     ` Uwe Kleine-König
2013-10-03 20:04     ` Uwe Kleine-König
2013-10-03 22:00 ` [PATCH 0/3] v7-M support using MULTIPLATFORM Russell King - ARM Linux
2013-10-03 22:28   ` Uwe Kleine-König
2013-10-03 23:05     ` Russell King - ARM Linux
2013-10-04  1:46       ` Nicolas Pitre
2013-11-04  8:25         ` Uwe Kleine-König
2013-11-05 11:40           ` Arnd Bergmann
2013-10-04  7:31       ` Uwe Kleine-König

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=524AA422.9040106@arm.com \
    --to=jonathan.austin@arm.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 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.