mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Al Boldi <a1426z@gawab.com>
To: Jan Engelhardt <jengelh@linux01.gwdg.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/30] Use menuconfig objects
Date: Wed, 11 Apr 2007 07:42:38 +0300	[thread overview]
Message-ID: <200704110742.38727.a1426z@gawab.com> (raw)
In-Reply-To: <Pine.LNX.4.61.0704110026250.32056@yvahk01.tjqt.qr>

Jan Engelhardt wrote:
> On Apr 11 2007 01:12, Al Boldi wrote:
> >Jan Engelhardt wrote:
> >> the following patch series turns some menus into menuconfigs, so they
> >> can be disabled whilst "walking" thorugh the parent menu
> >
> >Great, but instead of making it a simple on/off, make it tri-state that
> > would default select all child-options appropriately.  (see HW_RANDOM)
>
> I do not remember having touched the Random Number Generators or Character
> Devices anywhere in this series.
>
> Don't worry, I have paid attention (or at least I hope so ;-) to
> 'm'-able options, for example the SCSI, IEEE1394, or I2C stacks.
>
> But, for example you will see
>  [*] Old CD-ROM drivers (not SCSI, not IDE)
>
> which is because this entry in itself does not generate any object file.
>
> Generally, I just made the 'menuconfig' entry have the same state
> (bool/tristate) as the original entry. If I have overseen that
> somewhere, please let me know!

Correct, no oversight here; but it may be more meaningful, if you could 
default select child-options based on the parent-state, like HW_RANDOM.

Also, I don't think it's necessary to touch any of the "depends on"; keep 
them as is, as they don't hurt staying that way, and may actually be 
necessary under certain circumstances.  (see EMBEDDED)


Thanks!

--
Al


  reply	other threads:[~2007-04-11  4:39 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-10 22:12 Al Boldi
2007-04-10 22:30 ` Jan Engelhardt
2007-04-11  4:42   ` Al Boldi [this message]
2007-04-11  7:48     ` Jan Engelhardt
2007-04-11 16:53       ` Stefan Richter
2007-04-11 17:05         ` Jan Engelhardt
2007-04-11 17:30         ` Sam Ravnborg
2007-04-11 18:32       ` Al Boldi
  -- strict thread matches above, loose matches on Subject: below --
2007-04-10 19:17 Jan Engelhardt
2007-04-10 22:04 ` Stefan Richter
2007-04-10 21:40   ` David Lang
2007-04-10 22:41     ` Stefan Richter
2007-04-10 22:09   ` Jan Engelhardt
2007-04-10 23:19     ` Adrian Bunk
2007-04-11  5:52       ` Robert P. J. Day
2007-04-12 22:50 ` Andrew Morton
2007-04-12 23:07   ` Randy Dunlap
2007-04-12 23:18     ` Jan Engelhardt
2007-04-12 23:30       ` Randy Dunlap
2007-04-12 23:16   ` Jan Engelhardt
2007-04-12 23:35     ` Andrew Morton
2007-04-13  9:50 ` Jean Delvare
2007-04-13 16:02   ` Jan Engelhardt
2007-05-10 22:38 ` Andrew Morton
2007-05-11  6:05   ` Jan Engelhardt
2007-05-11  6:18     ` Andrew Morton
2007-05-11  7:47       ` Jan Engelhardt

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=200704110742.38727.a1426z@gawab.com \
    --to=a1426z@gawab.com \
    --cc=jengelh@linux01.gwdg.de \
    --cc=linux-kernel@vger.kernel.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

all inboxes | Powered by JetHome®