[U-Boot] [PATCH] drivers: hierarchize drivers Kconfig menu

[Paul Kocialkowski <contact@paulk.fr>]

Hi, 

Le lundi 27 juillet 2015 ? 14:33 +0200, Paul Kocialkowski a ?crit :
> Le dimanche 26 juillet 2015 ? 02:46 +0900, Masahiro Yamada a ?crit :
> > The menuconfig for drivers are getting more and more cluttered
> > and unreadable because too many entries are displayed in a single
> > flat menu.  Use hierarchic menu for each category.
> 
> That looks like a valuable addition to me!

This patch was submitted a week ago and I need it for merging another
series of mine, that was already accepted.

Is there anything preventing this patch from being merged at this point?

Thanks!

> I had started dealing with the USB side of this in a series currently
> under review. I'll drop the USB bits from my series and base the next
> version on top of this patch.
> 
> Thanks!
> 
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> > ---
> > 
> >  drivers/core/Kconfig   | 4 ++++
> >  drivers/crypto/Kconfig | 4 ++++
> >  drivers/demo/Kconfig   | 4 ++++
> >  drivers/gpio/Kconfig   | 8 ++++++++
> >  drivers/i2c/Kconfig    | 8 ++++++++
> >  drivers/led/Kconfig    | 4 ++++
> >  drivers/misc/Kconfig   | 8 ++++++++
> >  drivers/rtc/Kconfig    | 8 ++++++++
> >  drivers/serial/Kconfig | 8 ++++++++
> >  drivers/sound/Kconfig  | 4 ++++
> >  drivers/usb/Kconfig    | 8 ++++++++
> >  drivers/video/Kconfig  | 8 ++++++++
> >  12 files changed, 76 insertions(+)
> > 
> > diff --git a/drivers/core/Kconfig b/drivers/core/Kconfig
> > index e40372d..9cc888d 100644
> > --- a/drivers/core/Kconfig
> > +++ b/drivers/core/Kconfig
> > @@ -1,3 +1,5 @@
> > +menu "Generic Driver Options"
> > +
> >  config DM
> >  	bool "Enable Driver Model"
> >  	help
> > @@ -59,3 +61,5 @@ config DM_SEQ_ALIAS
> >  	  Most boards will have a '/aliases' node containing the path to
> >  	  numbered devices (e.g. serial0 = &serial0). This feature can be
> >  	  disabled if it is not required, to save code space in SPL.
> > +
> > +endmenu
> > diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
> > index bd26a2b..1ea116b 100644
> > --- a/drivers/crypto/Kconfig
> > +++ b/drivers/crypto/Kconfig
> > @@ -1 +1,5 @@
> > +menu "Hardware crypto devices"
> > +
> >  source drivers/crypto/fsl/Kconfig
> > +
> > +endmenu
> > diff --git a/drivers/demo/Kconfig b/drivers/demo/Kconfig
> > index 7a8ce18..98bb633 100644
> > --- a/drivers/demo/Kconfig
> > +++ b/drivers/demo/Kconfig
> > @@ -1,3 +1,5 @@
> > +menu "Demo for driver model"
> > +
> >  config DM_DEMO
> >  	bool "Enable demo uclass support"
> >  	depends on DM
> > @@ -24,3 +26,5 @@ config DM_DEMO_SHAPE
> >  	  a shape when the 'demo hello' command is executed which targets
> >  	  this device. It can be used to help understand how driver model
> >  	  works.
> > +
> > +endmenu
> > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
> > index 0c43777..5934597 100644
> > --- a/drivers/gpio/Kconfig
> > +++ b/drivers/gpio/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# GPIO infrastructure and drivers
> > +#
> > +
> > +menu "GPIO Support"
> > +
> >  config DM_GPIO
> >  	bool "Enable Driver Model for GPIO drivers"
> >  	depends on DM
> > @@ -42,3 +48,5 @@ config VYBRID_GPIO
> >  	default n
> >  	help
> >  	  Say yes here to support Vybrid vf610 GPIOs.
> > +
> > +endmenu
> > diff --git a/drivers/i2c/Kconfig b/drivers/i2c/Kconfig
> > index 86fb36b..b8b2250 100644
> > --- a/drivers/i2c/Kconfig
> > +++ b/drivers/i2c/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# I2C subsystem configuration
> > +#
> > +
> > +menu "I2C support"
> > +
> >  config DM_I2C
> >  	bool "Enable Driver Model for I2C drivers"
> >  	depends on DM
> > @@ -73,3 +79,5 @@ config SYS_I2C_UNIPHIER_F
> >  	help
> >  	  Support for UniPhier FIFO-builtin I2C controller driver.
> >  	  This I2C controller is used on PH1-Pro4 or newer UniPhier SoCs.
> > +
> > +endmenu
> > diff --git a/drivers/led/Kconfig b/drivers/led/Kconfig
> > index de5feea..b21bc94 100644
> > --- a/drivers/led/Kconfig
> > +++ b/drivers/led/Kconfig
> > @@ -1,3 +1,5 @@
> > +menu "LED Support"
> > +
> >  config LED
> >  	bool "Enable LED support"
> >  	depends on DM
> > @@ -24,3 +26,5 @@ config LED_GPIO
> >  	  GPIOs may be on the SoC or some other device which provides GPIOs.
> >  	  The GPIO driver must used driver model. LEDs are configured using
> >  	  the device tree.
> > +
> > +endmenu
> > diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> > index 3b7f76a..f29a169 100644
> > --- a/drivers/misc/Kconfig
> > +++ b/drivers/misc/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# Multifunction miscellaneous devices
> > +#
> > +
> > +menu "Multifunction device drivers"
> > +
> >  config CMD_CROS_EC
> >  	bool "Enable crosec command"
> >  	depends on CROS_EC
> > @@ -82,3 +88,5 @@ config RESET
> >  	  Each driver can provide a reset method which will be called to
> >  	  effect a reset. The uclass will try all available drivers when
> >  	  reset_walk() is called.
> > +
> > +endmenu
> > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> > index bd63621..b5d9048 100644
> > --- a/drivers/rtc/Kconfig
> > +++ b/drivers/rtc/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# RTC drivers configuration
> > +#
> > +
> > +menu "Real Time Clock"
> > +
> >  config DM_RTC
> >  	bool "Enable Driver Model for RTC drivers"
> >  	depends on DM
> > @@ -6,3 +12,5 @@ config DM_RTC
> >  	  then provides the rtc_get()/rtc_set() interface, delegating to
> >  	  drivers to perform the actual functions. See rtc.h for a
> >  	  description of the API.
> > +
> > +endmenu
> > diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig
> > index 4829284..b87eb01 100644
> > --- a/drivers/serial/Kconfig
> > +++ b/drivers/serial/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# Serial device configuration
> > +#
> > +
> > +menu "Serial drivers"
> > +
> >  config DM_SERIAL
> >  	bool "Enable Driver Model for serial drivers"
> >  	depends on DM
> > @@ -102,3 +108,5 @@ config UNIPHIER_SERIAL
> >  	help
> >  	  If you have a UniPhier based board and want to use the on-chip
> >  	  serial ports, say Y to this option. If unsure, say N.
> > +
> > +endmenu
> > diff --git a/drivers/sound/Kconfig b/drivers/sound/Kconfig
> > index 3b96e84..5de86c0 100644
> > --- a/drivers/sound/Kconfig
> > +++ b/drivers/sound/Kconfig
> > @@ -1,3 +1,5 @@
> > +menu "Sound support"
> > +
> >  config SOUND
> >  	bool "Enable sound support"
> >  	help
> > @@ -53,3 +55,5 @@ config SOUND_WM8994
> >  	  Enable the wm8994 audio codec. This is connected via I2S for
> >  	  audio data and I2C for codec control. At present it only works
> >  	  with the Samsung I2S driver.
> > +
> > +endmenu
> > diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig
> > index 3fa5b2e..19b1b3d 100644
> > --- a/drivers/usb/Kconfig
> > +++ b/drivers/usb/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# USB device configuration
> > +#
> > +
> > +menu "USB support"
> > +
> >  config USB_ARCH_HAS_HCD
> >  	def_bool y
> >  
> > @@ -60,3 +66,5 @@ config USB_STORAGE
> >  	  board's USB port.
> >  
> >  endif
> > +
> > +endmenu
> > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> > index 9ae23e8..892781d 100644
> > --- a/drivers/video/Kconfig
> > +++ b/drivers/video/Kconfig
> > @@ -1,3 +1,9 @@
> > +#
> > +# Video configuration
> > +#
> > +
> > +menu "Graphics support"
> > +
> >  config VIDEO_VESA
> >  	bool "Enable VESA video driver support"
> >  	default n
> > @@ -240,3 +246,5 @@ config VIDEO_TEGRA124
> >  	   HDMI. At present only eDP is supported by U-Boot. This option
> >  	   enables this support which can be used on devices which
> >  	   have an eDP display connected.
> > +
> > +endmenu
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150801/694ee1bc/attachment.sig>