[U-Boot] [PATCH 07/12] usb: USB_KEYBOARD move to usb Kconfig

Hans de Goede hdegoede at redhat.com
Wed Jul 22 11:22:50 CEST 2015


Hi,

On 22-07-15 10:45, Paul Kocialkowski wrote:
> The USB_KEYBOARD Kconfig is not specific to sunxi, so we can move it to the
> common USB Kconfig.
>
> Signed-off-by: Paul Kocialkowski <contact at paulk.fr>
> ---
>   board/sunxi/Kconfig | 5 -----
>   drivers/usb/Kconfig | 6 ++++++
>   2 files changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/board/sunxi/Kconfig b/board/sunxi/Kconfig
> index 9c25d49..66d2c4e 100644
> --- a/board/sunxi/Kconfig
> +++ b/board/sunxi/Kconfig
> @@ -531,13 +531,8 @@ config VIDEO_LCD_TL059WV5C0
>
>   endchoice
>
> -
>   config USB_KEYBOARD
> -	boolean "Enable USB keyboard support"
>   	default y
> -	---help---
> -	Say Y here to add support for using a USB keyboard (typically used
> -	in combination with a graphical console).
>
>   config GMAC_TX_DELAY
>   	int "GMAC Transmit Clock Delay Chain"

This leaves with just:

config USB_KEYBOARD
    	default y

And we've just gone to a whole bunch of removals of Kconfig
code like this because it breaks "make savedefconfig" when
used with non sunxi boards (in this case).

> diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig
> index d602e44..ab2a6a1 100644
> --- a/drivers/usb/Kconfig
> +++ b/drivers/usb/Kconfig
> @@ -66,4 +66,10 @@ config USB_STORAGE
>   	  Say Y here if you want to connect USB mass storage devices to your
>   	  board's USB port.
>
> +config USB_KEYBOARD
> +	boolean "Enable USB keyboard support"
> +	---help---
> +	  Say Y here to add support for using a USB keyboard (typically used
> +	  in combination with a graphical console).
> +
>   endif

So we need to set the default here, how about we add a
CONFIG_USB_HAS_HCD hidden bool option, and then select that from
CONFIG_USB_EHCI, CONFIG_USB_MUSB_HOST, etc, and then here do:

	default y if CONFIG_USB_HAS_HCD

?

Regards,

Hans


More information about the U-Boot mailing list