All of lore.kernel.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: nommu: re-enable use of vexpress without ARCH_MULTIPLATFORM
Date: Wed, 9 Jan 2013 18:43:48 +0000	[thread overview]
Message-ID: <201301091843.48942.arnd@arndb.de> (raw)
In-Reply-To: <1357755328-17075-1-git-send-email-jonathan.austin@arm.com>

On Wednesday 09 January 2013, Jonathan Austin wrote:
> From: Will Deacon <will.deacon@arm.com>
> 
> Since 617276307cd4c ("ARM: vexpress: convert to multi-platform") it has
> been impossible to select ARCH_VEXPRESS without ARCH_MULTIPLATFORM.
> 
> ARCH_MULTIPLATFORM doesn't make sense for NOMMU targets, not least
> because of the need to hard-code the memory map. However, it should
> still be possible to run NOMMU kernels on top of the Versatile Express
> by selecting it as the only platform.
> 
> This patch creates a shim ARCH_VEXPRESS_NOMMU config option in the 'choice'
> for "ARM system type" to make this possible again.
> 
> Signed-off-by: Will Deacon <will.deacon@arm.com>
> Signed-off-by: Jonathan Austin <jonathan.austin@arm.com>
> CC: Nicolas Pitre <nico@fluxnic.net>
> CC: Arnd Bergmann <arnd@arndb.de>
> ---
> 
> Arnd and Nicolas: I've removed your acks as I've changed the location
> of the new block (as Nicolas suggested) and ever so slightly tweaked
> the name of the new option.

The patch is still good. Generally you can leave an Ack when doing
small changes but leaving the patch conceptually the same.

Acked-by: Arnd Bergmann <arnd@arndb.de>

On a related topic, I still think we should fix ARCH_MULTI_V7 not
to select ARCH_VEXPRESS unconditionally and come up with a better
way to avoid having an empty platform list to make 'allnoconfig'
still work.

	Arnd

  parent reply	other threads:[~2013-01-09 18:43 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-09 18:15 [PATCH] ARM: nommu: re-enable use of vexpress without ARCH_MULTIPLATFORM Jonathan Austin
2013-01-09 18:30 ` Nicolas Pitre
2013-01-09 18:41   ` Will Deacon
2013-01-09 19:17     ` [PATCH v2] " Jonathan Austin
2013-01-09 18:43 ` Arnd Bergmann [this message]
2013-01-09 18:54   ` [PATCH] " Nicolas Pitre
2013-01-09 20:22     ` Arnd Bergmann
2013-01-09 20:39       ` Rob Herring
2013-01-09 20:48         ` Russell King - ARM Linux
2013-01-10  3:51           ` Rob Herring
2013-01-10 10:16             ` Arnd Bergmann
2013-01-10 13:20               ` Christopher Covington
2013-01-10 13:45                 ` Marc Zyngier
2013-01-09 21:09         ` Nicolas Pitre
2013-01-09 21:00       ` Nicolas Pitre
2013-01-09 21:15         ` Arnd Bergmann
2013-01-09 21:37           ` Nicolas Pitre
2013-01-09 22:14             ` Arnd Bergmann

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=201301091843.48942.arnd@arndb.de \
    --to=arnd@arndb.de \
    --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.