From: Daniel Walker <dwalker@codeaurora.org>
To: Russell King <rmk@arm.linux.org.uk>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Kevin Hilman <khilman@deeprootsystems.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
linux-arm-msm@vger.kernel.org
Subject: Re: ARM defconfig files
Date: Thu, 03 Jun 2010 13:06:48 -0700 [thread overview]
Message-ID: <1275595608.23384.58.camel@c-dwalke-linux.qualcomm.com> (raw)
In-Reply-To: <20100603195703.GG25779@flint.arm.linux.org.uk>
On Thu, 2010-06-03 at 20:57 +0100, Russell King wrote:
> On Thu, Jun 03, 2010 at 12:49:58PM -0700, Daniel Walker wrote:
> > On Thu, 2010-06-03 at 20:45 +0100, Russell King wrote:
> > > On Thu, Jun 03, 2010 at 12:35:42PM -0700, Daniel Walker wrote:
> > > > On Thu, 2010-06-03 at 11:56 -0700, Linus Torvalds wrote:
> > > > >
> > > > > On Thu, 3 Jun 2010, Russell King wrote:
> > > > > >
> > > > > > No amount of reorganising the Kconfig files into a heirarchial manner
> > > > > > (which they already are) helps. Not one bit. Because they already are.
> > > > > > That's not where the problem is.
> > > > >
> > > > > I don't think you read the whole thread.
> > > > >
> > > > > Earlier on, I explained exactly what I wanted: just add some "select"
> > > > > statements to pickt he things you need per the particular target
> > > > > configuration. You seem to have missed that part.
> > > > >
> > > > > In other words, you _can_ encode the information that is in the
> > > > > xyz_defconfig files by doing it in Kconfig.xyz files instead. But you do
> > > > > it in a human-readable manner. And the hierarchical thing is absolutely
> > > > > required for that - otherwise you'd end up with just another form of the
> > > > > current xyz_defconfig.
> > > > >
> > > > > See?
> > > > >
> > > > > In other words, you should be able to basically use "make allnoconfig"
> > > > > together with a Kconfig.xyz file input to select _exactly_ the pieces you
> > > > > need, and nothing else.
> > > >
> > > > If you did this for drivers, what about disabling a driver? If we used
> > > > "select" wouldn't that force all the drivers on without allowing it to
> > > > be unselected?
> > >
> > > I already covered that in my (ignored) email where I brought up a
> > > "STD_CONFIG" config symbol, which could be disabled to turn off all
> > > these additional "select"s.
> >
> > I didn't ignore it, I guess I just didn't fully understand it ..
> >
> > So your saying it would drop all the selects, but keep the selected
> > options in tact? Or it would just turn off all the selected options?
>
> config MACH_HALIBUT
> bool "Halibut Board (QCT SURF7201A)"
> select I2C if STD_CONFIG
> select I2C_WHATEVER if STD_CONFIG
> ...
>
> That means if you enable STD_CONFIG, you'll get everything that's required
> selected. If you then disable STD_CONFIG, I believe Kconfig leaves
> everything that was selected as still being selected.
I just did a little test, and it doesn't. Kconfig would un-select all
the drivers, at least from my test.
> So, what you _could_ do is start off with a blank configuration, then
> configure a kernel with STD_CONFIG enabled and you end up with everything
> that's required. If you then want to disable something that's selected,
> turn off STD_CONFIG first, and you'll be able to turn off individual
> options.
If my test was correct we would need a new "select" variant that would
leave the options turned on in order to get something equal to the
current defconfigs.
Daniel
next prev parent reply other threads:[~2010-06-03 20:06 UTC|newest]
Thread overview: 86+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20100603074548.GA12104@flint.arm.linux.org.uk>
2010-06-03 14:48 ` ARM defconfig files Linus Torvalds
2010-06-03 16:46 ` Tony Lindgren
2010-06-03 18:13 ` Russell King
2010-06-03 21:33 ` Tony Lindgren
2010-06-03 22:45 ` Nicolas Pitre
2010-06-04 4:59 ` Tony Lindgren
2010-06-04 0:23 ` Kevin Hilman
2010-06-04 4:53 ` Tony Lindgren
2010-06-04 1:02 ` Benjamin Herrenschmidt
2010-06-04 5:29 ` Tony Lindgren
2010-06-04 6:30 ` Geert Uytterhoeven
2010-06-04 6:53 ` Geert Uytterhoeven
2010-06-04 8:52 ` Benjamin Herrenschmidt
2010-06-03 16:53 ` Daniel Walker
2010-06-08 15:30 ` Catalin Marinas
2010-06-08 16:37 ` Daniel Walker
2010-06-03 18:10 ` Russell King
2010-06-03 18:18 ` Linus Torvalds
2010-06-03 18:53 ` Russell King
2010-06-03 18:56 ` Linus Torvalds
2010-06-03 19:20 ` Russell King
2010-06-03 19:35 ` Daniel Walker
2010-06-03 19:45 ` Russell King
2010-06-03 19:49 ` Daniel Walker
2010-06-03 19:57 ` Russell King
2010-06-03 20:06 ` Daniel Walker [this message]
2010-06-03 20:18 ` Russell King
2010-06-03 20:20 ` Nicolas Pitre
2010-06-04 1:06 ` Benjamin Herrenschmidt
2010-06-03 20:09 ` Linus Torvalds
2010-06-03 20:31 ` Linus Torvalds
2010-06-03 21:17 ` Tony Lindgren
2010-06-03 22:15 ` Grant Likely
2010-06-04 5:18 ` Felipe Balbi
2010-06-04 11:31 ` Catalin Marinas
2010-06-03 22:24 ` Daniel Walker
2010-06-05 14:12 ` Felipe Contreras
2010-06-05 14:39 ` Linus Torvalds
2010-06-05 16:39 ` Felipe Contreras
2010-06-03 21:48 ` Daniel Walker
2010-06-04 0:36 ` Paul Mackerras
2010-06-04 12:39 ` Grant Likely
2010-06-05 13:47 ` Felipe Contreras
2010-06-03 20:34 ` Nicolas Pitre
2010-06-03 20:05 ` Linus Torvalds
2010-06-06 3:28 ` david
2010-06-03 18:20 ` Daniel Walker
2010-06-03 18:21 ` Linus Torvalds
2010-06-03 18:30 ` Al Viro
2010-06-03 19:26 ` Paul Mundt
2010-06-14 8:32 ` Uwe Kleine-König
2010-06-30 10:40 ` Uwe Kleine-König
2010-07-12 15:55 ` Uwe Kleine-König
2010-07-12 16:51 ` Linus Torvalds
2010-07-12 17:32 ` Russell King - ARM Linux
2010-07-12 17:40 ` Linus Torvalds
2010-07-12 18:50 ` Uwe Kleine-König
2010-07-12 19:04 ` Linus Torvalds
2010-07-12 19:17 ` Nicolas Pitre
2010-07-12 19:34 ` Linus Torvalds
2010-07-12 19:50 ` Grant Likely
2010-07-13 7:07 ` Uwe Kleine-König
2010-07-13 8:07 ` optimized script [Was: ARM defconfig files] Uwe Kleine-König
2010-07-13 18:04 ` Olof Johansson
2010-07-13 23:39 ` Nicolas Pitre
2010-07-13 18:32 ` ARM defconfig files Grant Likely
2010-07-12 19:59 ` Uwe Kleine-König
2010-07-12 20:14 ` Nicolas Pitre
2010-07-12 19:09 ` Nicolas Pitre
2010-07-12 20:31 ` Arnd Bergmann
2010-07-12 20:50 ` Nicolas Pitre
2010-07-12 23:05 ` David Brown
2010-07-12 23:18 ` Linus Torvalds
2010-07-12 23:34 ` David Brown
2010-07-13 0:55 ` Nicolas Pitre
2010-07-14 9:13 ` Felipe Contreras
2010-07-14 13:20 ` Uwe Kleine-König
2010-07-14 17:37 ` Tony Luck
2010-07-13 18:32 ` Rob Landley
2010-07-12 20:06 ` Russell King - ARM Linux
2010-07-12 20:29 ` Nicolas Pitre
2010-07-12 21:54 ` Linus Torvalds
2010-07-14 9:21 ` Felipe Contreras
2010-06-03 18:41 ` Russell King
2010-06-03 18:53 ` Linus Torvalds
2010-06-06 3:53 ` david
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=1275595608.23384.58.camel@c-dwalke-linux.qualcomm.com \
--to=dwalker@codeaurora.org \
--cc=khilman@deeprootsystems.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rmk@arm.linux.org.uk \
--cc=torvalds@linux-foundation.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).