All of lore.kernel.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: Making ARM multiplatform kernels DT-only?
Date: Thu, 3 May 2012 16:27:44 +0000	[thread overview]
Message-ID: <201205031627.44889.arnd@arndb.de> (raw)
In-Reply-To: <20120503141853.GC897@n2100.arm.linux.org.uk>

On Thursday 03 May 2012, Russell King - ARM Linux wrote:
> On Thu, May 03, 2012 at 01:50:35PM +0000, Arnd Bergmann wrote:
> > My feeling is that we should just mandate DT booting for multiplatform
> > kernels, because it significantly reduces the combinatorial space
> > at compile time, avoids a lot of legacy board files that we cannot
> > test anyway, reduces the total kernel size and gives an incentive
> > for people to move forward to DT with their existing boards.
> 
> On this point, I strongly object, especially as I'm one who uses the
> existing non-DT multiplatform support extensively.  It's really not
> a problem for what you're trying to achieve.

Just to clarify the terminology, when I said "multiplatform", I did
not mean enabling more than one board file inside a given mach-*
directory but instead enabling multiple mach-* directories that
are currently mutually exclusive, i.e. the future stuff you replied
to in the other mail, not what everyone is doing today, and this
would not stop anything from working that works today.

> I think what you're proposing is a totally artificial restriction.
> There's no problem with a kernel supporting DT and non-DT together.
> We've proven that many many times.  I prove it every night that my
> build and boot system runs - the OMAP LDP boots a multiplatform kernel
> just fine without DT.

Of course it's an artificial restriction, if it was a technical necessity,
I would not have needed to ask ;-) IMHO however it's a helpful restriction.
My current count of board files is 393 and if you consider that we won't
build v6+ and v4/v5 together and that some of them will probably not
be multiplatform capable for a long time, we probably end up with about
half of them in a given kernel, which is still a lot and I would not
expect distributors to make a good decision about which ones of these
are important to enable and which ones are not. If we restrict the
Kconfig space to just the ones that are DT-enabled, we can be reasonably
sure that these have been recently tested on actual hardware by someone
who cares about them, and we get only a fraction of the user visible
options, roughly one per soc generation.

One counterargument that just occurred to me is build coverage, and it
would be nice to have "make allyesconfig" actually build everything
together as far as possible. We could get a bit closer to that if
we allow those platforms that have no DT support to just provide a
Kconfig option for multiplatform kernels that enables everything, e.g.
when you build an ARMv4/ARMv5 kernel, you can enable all sa1100
based boards together using one option, but when you build an sa1100
kernel, you keep picking them individually.

	Arnd

WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: "Russell King - ARM Linux" <linux@arm.linux.org.uk>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linaro-dev@lists.linaro.org,
	"Jean-Christophe PLAGNIOL-VILLARD" <plagnioj@jcrosoft.com>,
	Deepak Saxena <dsaxena@linaro.org>,
	Tony Lindgren <tony@atomide.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	shawn.guo@linaro.org, Sascha Hauer <s.hauer@pengutronix.de>,
	Magnus Damm <magnus.damm@gmail.com>,
	Kukjin Kim <kgene.kim@samsung.com>,
	Olof Johansson <olof@lixom.net>,
	David Brown <davidb@codeaurora.org>,
	Nicolas Pitre <nico@fluxnic.net>,
	Haojian Zhuang <haojian.zhuang@gmail.com>,
	Jason Cooper <jason@lakedaemon.net>,
	Nicolas Ferre <nicolas.ferre@atmel.com>
Subject: Re: Making ARM multiplatform kernels DT-only?
Date: Thu, 3 May 2012 16:27:44 +0000	[thread overview]
Message-ID: <201205031627.44889.arnd@arndb.de> (raw)
In-Reply-To: <20120503141853.GC897@n2100.arm.linux.org.uk>

On Thursday 03 May 2012, Russell King - ARM Linux wrote:
> On Thu, May 03, 2012 at 01:50:35PM +0000, Arnd Bergmann wrote:
> > My feeling is that we should just mandate DT booting for multiplatform
> > kernels, because it significantly reduces the combinatorial space
> > at compile time, avoids a lot of legacy board files that we cannot
> > test anyway, reduces the total kernel size and gives an incentive
> > for people to move forward to DT with their existing boards.
> 
> On this point, I strongly object, especially as I'm one who uses the
> existing non-DT multiplatform support extensively.  It's really not
> a problem for what you're trying to achieve.

Just to clarify the terminology, when I said "multiplatform", I did
not mean enabling more than one board file inside a given mach-*
directory but instead enabling multiple mach-* directories that
are currently mutually exclusive, i.e. the future stuff you replied
to in the other mail, not what everyone is doing today, and this
would not stop anything from working that works today.

> I think what you're proposing is a totally artificial restriction.
> There's no problem with a kernel supporting DT and non-DT together.
> We've proven that many many times.  I prove it every night that my
> build and boot system runs - the OMAP LDP boots a multiplatform kernel
> just fine without DT.

Of course it's an artificial restriction, if it was a technical necessity,
I would not have needed to ask ;-) IMHO however it's a helpful restriction.
My current count of board files is 393 and if you consider that we won't
build v6+ and v4/v5 together and that some of them will probably not
be multiplatform capable for a long time, we probably end up with about
half of them in a given kernel, which is still a lot and I would not
expect distributors to make a good decision about which ones of these
are important to enable and which ones are not. If we restrict the
Kconfig space to just the ones that are DT-enabled, we can be reasonably
sure that these have been recently tested on actual hardware by someone
who cares about them, and we get only a fraction of the user visible
options, roughly one per soc generation.

One counterargument that just occurred to me is build coverage, and it
would be nice to have "make allyesconfig" actually build everything
together as far as possible. We could get a bit closer to that if
we allow those platforms that have no DT support to just provide a
Kconfig option for multiplatform kernels that enables everything, e.g.
when you build an ARMv4/ARMv5 kernel, you can enable all sa1100
based boards together using one option, but when you build an sa1100
kernel, you keep picking them individually.

	Arnd


  parent reply	other threads:[~2012-05-03 16:27 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-03 13:50 Making ARM multiplatform kernels DT-only? Arnd Bergmann
2012-05-03 13:50 ` Arnd Bergmann
2012-05-03 13:45 ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-03 13:45   ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-03 14:04 ` Russell King - ARM Linux
2012-05-03 14:04   ` Russell King - ARM Linux
2012-05-03 13:52   ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-03 13:52     ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-04  6:31   ` Deepak Saxena
2012-05-04  6:31     ` Deepak Saxena
2012-05-04  7:27     ` Russell King - ARM Linux
2012-05-04  7:27       ` Russell King - ARM Linux
2012-05-04 12:20   ` Arnd Bergmann
2012-05-04 12:20     ` Arnd Bergmann
2012-05-04 16:39     ` Rob Herring
2012-05-04 16:39       ` Rob Herring
2012-05-04 16:56       ` Russell King - ARM Linux
2012-05-04 16:56         ` Russell King - ARM Linux
2012-05-04 16:40         ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-04 16:40           ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-04 16:51         ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-04 16:51           ` Jean-Christophe PLAGNIOL-VILLARD
2012-05-04 18:56       ` Arnd Bergmann
2012-05-04 18:56         ` Arnd Bergmann
2012-05-03 14:18 ` Russell King - ARM Linux
2012-05-03 14:18   ` Russell King - ARM Linux
2012-05-03 14:23   ` Magnus Damm
2012-05-03 14:23     ` Magnus Damm
2012-05-03 16:27   ` Arnd Bergmann [this message]
2012-05-03 16:27     ` Arnd Bergmann
2012-05-04  9:22   ` Arnaud Patard (Rtp)
2012-05-04  9:22     ` Arnaud Patard
2012-05-04 12:34     ` Arnd Bergmann
2012-05-04 12:34       ` Arnd Bergmann
2012-05-10 10:55   ` Ben Dooks
2012-05-10 10:55     ` Ben Dooks
2012-05-10 11:02     ` Russell King - ARM Linux
2012-05-10 11:02       ` Russell King - ARM Linux
2012-05-03 14:46 ` Sascha Hauer
2012-05-03 14:46   ` Sascha Hauer
2012-05-04 16:24   ` Arnd Bergmann
2012-05-04 16:24     ` Arnd Bergmann
2012-05-05  8:09     ` Sascha Hauer
2012-05-05  8:09       ` Sascha Hauer
2012-05-05 13:17       ` Arnd Bergmann
2012-05-05 13:17         ` Arnd Bergmann
2012-05-14  8:54         ` Arnd Bergmann
2012-05-14  8:54           ` Arnd Bergmann
2012-05-04  5:38 ` Deepak Saxena
2012-05-04  5:38   ` Deepak Saxena
2012-05-04  7:39   ` Russell King - ARM Linux
2012-05-04  7:39     ` Russell King - ARM Linux
2012-05-04 14:20   ` Wookey
2012-05-04 14:20     ` Wookey
2012-05-04 14:35     ` Russell King - ARM Linux
2012-05-04 14:35       ` Russell King - ARM Linux
2012-05-04 15:17       ` Arnd Bergmann
2012-05-04 15:17         ` Arnd Bergmann
2012-05-04 16:05         ` Wookey
2012-05-04 16:05           ` Wookey
2012-05-04 18:49           ` Arnd Bergmann
2012-05-04 18:49             ` Arnd Bergmann
2012-05-04 20:03       ` Linus Walleij
2012-05-04 20:03         ` Linus Walleij
2012-05-04 20:42         ` Christian Robottom Reis
2012-05-04 20:42           ` Christian Robottom Reis
2012-05-04 21:05           ` Arnd Bergmann
2012-05-04 21:05             ` Arnd Bergmann
2012-05-04 22:43         ` Russell King - ARM Linux
2012-05-04 22:43           ` Russell King - ARM Linux

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=201205031627.44889.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.