From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Kocialkowski Date: Fri, 24 Jul 2015 19:18:14 +0200 Subject: [U-Boot] [PATCH 07/12] usb: USB_KEYBOARD move to usb Kconfig In-Reply-To: <55AF60EA.9010703@redhat.com> References: <1437554735-11728-1-git-send-email-contact@paulk.fr> <1437554735-11728-7-git-send-email-contact@paulk.fr> <55AF60EA.9010703@redhat.com> Message-ID: <1437758294.2690.76.camel@collins> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Le mercredi 22 juillet 2015 ? 11:22 +0200, Hans de Goede a ?crit : > 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 > > --- > > 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). Okay, I hadn't followed those developments. My mistake. > > 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 This would work. I'm not sure it is expected for Kconfig options to be "phony", but if you think it's fine, I'll go with it. > Regards, > > Hans -------------- 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: