linux-kernel - Re: [RFC] USB Kconfig: Declutter USB Kconfig Menu

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <200712221649.55831.a1426z@gawab.com>
Date:	Sat, 22 Dec 2007 16:49:55 +0300
From:	Al Boldi <a1426z@...ab.com>
To:	David Brownell <david-b@...bell.net>, jengelh@...putergmbh.de
Cc:	linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
	greg@...ah.com
Subject: Re: [RFC] USB Kconfig: Declutter USB Kconfig Menu

David Brownell wrote:
> > > Which is why my suggestion was to have them both move up a level, with
> > > host and peripheral side menus nested normally:
> > >
> > > 	Device Drivers:
> > > 		...
> > > 		[ ] HID devices
> > > 		< > Host side USB
> > > 		< > Peripheral side USB
> > > 		< > MMC/SD/SDIO card support
> > > 		[ ] LED support
> > > 		...
> > >
> > > That way both host and peripheral side support would have their own
> > > menu, and that pointless nesting would be reduced.
> >
> > You mean:
> >  		[ ] Host side USB
> >  		[ ] Peripheral side USB
> >
> > Right?
>
> Both of those subsystems are already set up with the main choice
> as tristate "< >/<M>/<*>".  If we're simplifying the navigation,
> I can't see a point to requiring *two* menu actions not one.

There is a point, but it's a weak one: you want it to unfold in the same menu 
as you select the option.  So, if you unfold the sub-options in a separate 
sub-menu, you better use [ ] (or just blank), as this will allow you to 
enter the sub-menu, and see the unfold take place inside the same menu.

IMHO, I prefer < >, but newbies probably prefer [ ], as it is less abstract.

Provided there is agreement, and we go with < >, it would mean a slight 
change to Host side USB, like this (note new USB Core Config sub-menu):

      --- Support for Host-side USB                                      
      [ ]   USB Core Config  --->                                       
      [ ]   USB Host Controller Drivers  --->                        
      ---   NOTE: USB_STORAGE needs SCSI, and 'SCSI disk support' may
      ---   also be needed; see USB_STORAGE Help for more information
      < >   USB Mass Storage support  --->                          
      [ ]   The shared table of common (or usual) storage devices    
      [ ]   USB Monitor                                               
      < >   USS720 parport driver                               
      < >   USB Serial Converter support  --->                        
      < >   USB DSL modem support  --->                                
      [ ]   USB Device Class drivers  --->                            
      [ ]   USB Imaging devices  --->                                 
      [ ]   USB Miscellaneous drivers  --->                                         


Thanks!

--
Al

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/