LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Alexandre Belloni <alexandre.belloni@free-electrons.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
	Nicolas Ferre <nicolas.ferre@atmel.com>,
	linux-kernel@vger.kernel.org,
	Brian Norris <computersforpeace@gmail.com>,
	Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
	Andrew Victor <linux@maxim.org.za>
Subject: Re: [PATCH] ARM: at91: force CPU selection
Date: Fri, 13 Mar 2015 19:32:00 +0100	[thread overview]
Message-ID: <20150313183200.GK4329@piout.net> (raw)
In-Reply-To: <2279482.Ohk2PcyWHV@wuerfel>

On 10/03/2015 at 14:42:24 +0100, Arnd Bergmann wrote :
> On Tuesday 10 March 2015 10:42:09 Alexandre Belloni wrote:
> > > 
> > > I actually added a bunch of these in other places, but have stopped
> > > doing so because Russell didn't like them, and I tend to follow
> > > his argument now that it's actually pretty confusing.
> > > 
> > > In case of at91, we can do better now, once we have multiplatform
> > > enabled, and we can simply remove the SOC_SAM_V7 and SOC_SAM_V4_V5
> > > symbols entirely (if Alexandre's patch doesn't already do that).
> > > 
> > 
> > I didn't remove it to keep the sub menus SOC_SAM_V7 -> SOC_SAMA5D4 and
> > SOC_SAMA5D3, SOC_SAM_V4_V5 -> SOC_AT91SAM9 and SOC_AT91RM9200. But I can
> > also expose SOC_AT91RM9200, SOC_AT91SAM9, SOC_SAMA5D3 and SOC_SAMA5D4
> > instead of SOC_SAM_V7 and SOC_SAM_V4_V5 and remove them.
> 
> Exposing multiple levels won't help here since you can never enable
> both v4/v5 and v7 at the same time in multiplatform.
> 
> Just use one level for a 'menuconfig ARCH_AT91' entry and put the
> four options below that.
> 

Actually, I had a closer look today and there is already just one level
in my current version of patches.

I can put everything under menuconfig ARCH_AT91 but I can't remove
SOC_SAM_V7 and SOC_SAM_V4_V5 because that is what is used to select
which dtb should be compiled.

This would also break sama5_defconfig. This is a simple fix but that
means that it makes it difficult to migrate an existing .config.

I'd say let's keep it that way as this is not broken, unless you really
want to see SOC_SAM_V7 and SOC_SAM_V4_V5 being removed.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2015-03-13 18:32 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-10  6:59 Brian Norris
2015-03-10  9:02 ` Arnd Bergmann
2015-03-10  9:42   ` Alexandre Belloni
2015-03-10 13:42     ` Arnd Bergmann
2015-03-13 18:32       ` Alexandre Belloni [this message]
2015-03-13 19:50         ` Arnd Bergmann
2015-03-13 20:30           ` Alexandre Belloni
2015-03-10 20:02   ` Brian Norris
2015-03-10 21:13     ` Alexandre Belloni

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=20150313183200.GK4329@piout.net \
    --to=alexandre.belloni@free-electrons.com \
    --cc=arnd@arndb.de \
    --cc=computersforpeace@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@maxim.org.za \
    --cc=nicolas.ferre@atmel.com \
    --cc=plagnioj@jcrosoft.com \
    --subject='Re: [PATCH] ARM: at91: force CPU selection' \
    /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

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).