Messages in this thread | | | Date | Wed, 11 Apr 2007 00:21:04 +0200 | From | Stefan Richter <> | Subject | Re: [PATCH 13/30] Use menuconfig objects - IEEE1394 |
| |
Jan Engelhardt wrote: > Use menuconfigs instead of menus, so the whole menu can be disabled at > once instead of going through all options.
If you excuse the nitpicking: The patch description is not very precise. You don't have to go through all options in the IEEE 1394 menu if you don't need them. Once the first entry in the menu is disabled, all other options vanish. (Of course with your patch you don't have to enter the submenu in the first place if you switch it off.) I suppose it's the same with some more of the menus you edited.
But even though, keeping such groups of options in a menu _without_ the possibility to switch the menu off helps to focus on these options. Less clutter in the display one is working with in a moment. IMO, kernel configuration is not a sport where speed matters but a task where we want to present the user a very cleaned-up, calm display.
In general though, I'm impartial on this kind of change.
> Signed-off-by: Jan Engelhardt <jengelh@gmx.de> > > Index: linux-2.6.21-rc5/drivers/ieee1394/Kconfig > =================================================================== > --- linux-2.6.21-rc5.orig/drivers/ieee1394/Kconfig > +++ linux-2.6.21-rc5/drivers/ieee1394/Kconfig > @@ -1,8 +1,6 @@ > # -*- shell-script -*- > > -menu "IEEE 1394 (FireWire) support" > - > -config IEEE1394 > +menuconfig IEEE1394 > tristate "IEEE 1394 (FireWire) support" > depends on PCI || BROKEN > select NET > @@ -19,12 +17,12 @@ config IEEE1394
[...]
For the readers of linux1394-devel:
Jan Engelhardt wrote at lkml: Subject: [PATCH 0/30] Use menuconfig objects > Hello list, > > > the following patch series turns some menus into menuconfigs, so they > can be disabled whilst "walking" thorugh the parent menu (check the > videos [1], [2] to see what I mean), enabling for disabling lots of > options _quickly_. > > I'll send the patches (as a reply to this mail) piece by piece out > when I figure out the maintainers to Cc for each. > > [1] 6.04 MB(1000s) 70 sec http://jengelh.hopto.org/mc/without.ogg > [2] 1.96 MB(1000s) 44 sec http://jengelh.hopto.org/mc/with.ogg > (Ogg container, Theora Video, no audio) > > I have not poked on all subsystems (it is quite a lot), but I can > already give a glimpse (`ls -1`) of who gets one and who does not. > And, I am not completely finished changing some subsystems - but > the patches are already big enough for a first wave. I want to get > some feedback first. Thanks! >
-- Stefan Richter -=====-=-=== -=-- -=-== http://arcgraph.de/sr/ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |