From: Arnd Bergmann <arnd@arndb.de>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/2] ARM: multi_v7_defconfig: Enable shmobile platforms
Date: Mon, 01 Sep 2014 08:58:19 +0000 [thread overview]
Message-ID: <7714045.KMII2MK3EF@wuerfel> (raw)
In-Reply-To: <0a6f01cfc432$507900f0$f16b02d0$@samsung.com>
On Saturday 30 August 2014 18:10:59 Kukjin Kim wrote:
> Arnd Bergmann wrote:
> > On Friday 29 August 2014 20:05:49 Bartlomiej Zolnierkiewicz wrote:
> > > I've been looking lately into making it possible to easily go
> > > from multi_v7_defconfig config to a single platform one (in my
> > > case Exynos one) and removing the need to keep the latter (i.e.
> > > exynos_defconfig) in the kernel tree in the long-term.
> > >
> Why? I don't have any idea why we should use only multi_v7_defconfig for each
> hardware platform. In my understanding, multi_v7_defconfig can be used for one
> kernel image can support all of the platforms but it doesn't mean it should be
> used for each platform. And I think, if we cannot maintain each platform's
> defconfig in mainline, it will be kept in each SoC vendor and it is not a good
> way. I mean both defconfigs has each value in mainline, exynos_defconfig should
> be updated though ;)
It's not clear how we are going to do this in the long run. We definitely
have far more defconfig files in the kernel than we want to have, it's
a burden for testers. In theory, running multi_v7_defconfig should have
no downsides other than the obvious space overhead of the unused platform
code and there should be no difference between disabling unused file systems
and PCI drivers to disabling unused platforms.
Another argument is that even an exynos_defconfig contains much more
code than you really want, and for an embedded system you would turn off
all the drivers that a particular exynos machine does not have. So there
is no fundamental difference between turning off part of exynos_defconfig
and turning off parts of multi_v7_defconfig, the difference is mainly
how many options you'd turn off.
Then again, I also don't see a reason to remove exynos_defconfig any time
soon. So far the rule is that each platform can have its own defconfig
because that's the way it has been done for a long time.
Arnd
next prev parent reply other threads:[~2014-09-01 8:58 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-28 14:00 [PATCH 1/2] ARM: multi_v7_defconfig: Enable shmobile platforms Geert Uytterhoeven
2014-08-28 14:00 ` [PATCH 2/2] [RFC] ARM: Set CONFIG_LOCALVERSION in multi v5/v7 defconfigs Geert Uytterhoeven
2014-08-28 23:23 ` [PATCH 1/2] ARM: multi_v7_defconfig: Enable shmobile platforms Simon Horman
2014-08-29 18:05 ` Bartlomiej Zolnierkiewicz
2014-08-29 18:08 ` Bartlomiej Zolnierkiewicz
2014-08-29 20:11 ` Arnd Bergmann
2014-08-30 9:10 ` Kukjin Kim
2014-09-01 8:58 ` Arnd Bergmann [this message]
2014-09-01 10:47 ` Bartlomiej Zolnierkiewicz
2014-08-30 9:32 ` Geert Uytterhoeven
2014-09-01 8:24 ` Arnd Bergmann
2014-09-01 1:54 ` Simon Horman
2014-09-01 12:42 ` Bartlomiej Zolnierkiewicz
2014-09-01 13:19 ` Simon Horman
2014-09-10 0:48 ` Simon Horman
2014-11-25 15:08 ` Geert Uytterhoeven
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=7714045.KMII2MK3EF@wuerfel \
--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 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).