Re: inconsistent design of menus and submenus in 2.5.53

From: Roman Zippel (zippel@linux-m68k.org)
Date: Thu Jan 02 2003 - 20:30:18 EST


Hi,

On Wed, 1 Jan 2003, Robert P. J. Day wrote:

> another example of this (good) design is the recent addition
> of the 10/100 Mbit and 1000 Mbit options under Network device
> support, once again showing clearly the child option structure.
>
> contrast that with Parallel port support, where lower options
> in the list that *appear* to be sibling choices to Parallel
> port support are actually child options that vanish when you
> deselect Parallel port support. very confusing.

This is quite easy to fix, but it will require some work. Someone has to
go through the configuration and check the dependencies, as the
dependencies are used to generate the menu structure.
Below I fixed e.g. the parport config, sometimes there is still cml1
cruft, which couldn't be completely fixed during the automatic conversion,
the logic is still the same, but the menu structure is a little messed up.
So if someone wants to do this, I'd be happy to help and review the work.

bye, Roman

diff -ur -X /home/roman/nodiff linux.org/drivers/parport/Kconfig linux/drivers/parport/Kconfig
--- linux.org/drivers/parport/Kconfig 2002-12-16 03:08:23.000000000 +0100
+++ linux/drivers/parport/Kconfig 2003-01-02 18:11:58.000000000 +0100
@@ -34,9 +34,10 @@

           If unsure, say Y.

+if PARPORT
+
 config PARPORT_PC
         tristate "PC-style hardware"
- depends on PARPORT
         ---help---
           You should say Y here if you have a PC-style parallel port. All
           IBM PC compatible computers and some Alphas have PC-style
@@ -51,15 +52,9 @@

           If unsure, say Y.

-config PARPORT_PC_CML1
- tristate
- depends on PARPORT!=n && PARPORT_PC!=n
- default PARPORT_PC if SERIAL_8250=y
- default m if SERIAL_8250=m
-
 config PARPORT_SERIAL
         tristate "Multi-IO cards (parallel and serial)"
- depends on SERIAL_8250!=n && PARPORT_PC_CML1
+ depends on SERIAL_8250 && PARPORT_PC
         help
           This adds support for multi-IO PCI cards that have parallel and
           serial ports. You should say Y or M here. If you say M, the module
@@ -88,18 +83,18 @@

 config PARPORT_PC_PCMCIA
         tristate "Support for PCMCIA management for PC-style ports"
- depends on PARPORT!=n && HOTPLUG && (PCMCIA!=n && PARPORT_PC=m && PARPORT_PC || PARPORT_PC=y && PCMCIA)
+ depends on PCMCIA && PARPORT_PC
         help
           Say Y here if you need PCMCIA support for your PC-style parallel
           ports. If unsure, say N.

 config PARPORT_ARC
         tristate "Archimedes hardware"
- depends on ARM && PARPORT
+ depends on ARM

 config PARPORT_AMIGA
         tristate "Amiga builtin port"
- depends on AMIGA && PARPORT
+ depends on AMIGA
         help
           Say Y here if you need support for the parallel port hardware on
           Amiga machines. This code is also available as a module (say M),
@@ -107,7 +102,7 @@

 config PARPORT_MFC3
         tristate "Multiface III parallel port"
- depends on AMIGA && ZORRO && PARPORT
+ depends on AMIGA && ZORRO
         help
           Say Y here if you need parallel port support for the MFC3 card.
           This code is also available as a module (say M), called
@@ -115,7 +110,7 @@

 config PARPORT_ATARI
         tristate "Atari hardware"
- depends on ATARI && PARPORT
+ depends on ATARI
         help
           Say Y here if you need support for the parallel port hardware on
           Atari machines. This code is also available as a module (say M),
@@ -128,7 +123,7 @@

 config PARPORT_SUNBPP
         tristate "Sparc hardware (EXPERIMENTAL)"
- depends on SBUS && EXPERIMENTAL && PARPORT
+ depends on SBUS && EXPERIMENTAL
         help
           This driver provides support for the bidirectional parallel port
           found on many Sun machines. Note that many of the newer Ultras
@@ -138,7 +133,6 @@
 # support for loading any others. Defeat this if the user is keen.
 config PARPORT_OTHER
         bool "Support foreign hardware"
- depends on PARPORT
         help
           Say Y here if you want to be able to load driver modules to support
           other non-standard types of parallel ports. This causes a
@@ -146,7 +140,6 @@

 config PARPORT_1284
         bool "IEEE 1284 transfer modes"
- depends on PARPORT
         help
           If you have a printer that supports status readback or device ID, or
           want to use a device that uses enhanced parallel port transfer modes
@@ -154,5 +147,7 @@
           transfer modes. Also say Y if you want device ID information to
           appear in /proc/sys/dev/parport/*/autoprobe*. It is safe to say N.

+endif
+
 endmenu

-
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/



This archive was generated by hypermail 2b29 : Tue Jan 07 2003 - 22:00:20 EST