[U-Boot] [PATCH 06/12] usb: USB_MUSB_SUNXI move to musb-new Kconfig

[Paul Kocialkowski <contact@paulk.fr>]

Le mercredi 22 juillet 2015 ? 11:17 +0200, Hans de Goede a ?crit :
> Hi,
> 
> On 22-07-15 10:45, Paul Kocialkowski wrote:
> > Now that the musb-new driver has a Kconfig, we can move Kconfig options to
> > enable controllers to it, so that it's easier in e.g. menuconfig.
> >
> > Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> > ---
> >   board/sunxi/Kconfig          |  9 ---------
> >   drivers/usb/musb-new/Kconfig | 14 ++++++++++++++
> >   2 files changed, 14 insertions(+), 9 deletions(-)
> >
> > diff --git a/board/sunxi/Kconfig b/board/sunxi/Kconfig
> > index 2a1cd3c..9c25d49 100644
> > --- a/board/sunxi/Kconfig
> > +++ b/board/sunxi/Kconfig
> > @@ -532,15 +532,6 @@ config VIDEO_LCD_TL059WV5C0
> >   endchoice
> >
> >
> > -config USB_MUSB_SUNXI
> > -	bool "Enable sunxi OTG / DRC USB controller in host mode"
> > -	default n
> > -	---help---
> > -	Say y here to enable support for the sunxi OTG / DRC USB controller
> > -	used on almost all sunxi boards. Note currently u-boot can only have
> > -	one usb host controller enabled at a time, so enabling this on boards
> > -	which also use the ehci host controller will result in build errors.
> > -
> >   config USB_KEYBOARD
> >   	boolean "Enable USB keyboard support"
> >   	default y
> > diff --git a/drivers/usb/musb-new/Kconfig b/drivers/usb/musb-new/Kconfig
> > index 34a8474..5be072a 100644
> > --- a/drivers/usb/musb-new/Kconfig
> > +++ b/drivers/usb/musb-new/Kconfig
> > @@ -12,3 +12,17 @@ config USB_MUSB_GADGET
> >   	bool "MUSB gadget mode support"
> >   	help
> >   	  Enables the MUSB USB dual-role controller in gadget mode.
> > +
> > +if USB_MUSB_HOST || USB_MUSB_GADGET
> > +
> > +config USB_MUSB_SUNXI
> > +	bool "Enable sunxi OTG / DRC USB controller"
> > +	depends on ARCH_SUNXI
> > +	default n
> > +	---help---
> > +	Say y here to enable support for the sunxi OTG / DRC USB controller
> > +	used on almost all sunxi boards. Note currently u-boot can only have
> > +	one usb host controller enabled at a time, so enabling this on boards
> > +	which also use the ehci host controller will result in build errors.
> > +
> > +endif
> >
> 
> AFAICT this will break MUSB host usage on current sunxi boards, since their
> defconfig-s contain CONFIG_USB_MUSB_SUNXI=y but not CONFIG_USB_MUSB_HOST=y
> please update all sunxi defconfigs which have CONFIG_USB_MUSB_SUNXI=y to also
> set CONFIG_USB_MUSB_HOST=y in this patch to avoid this breakage.

Ah, I suspected you wouldn't like it! This is fixed by the next page, as
you probably saw, but I don't object to having every patch leave the
tree in an unsable state.

I'll submit v2 with that in mind.

> Note all earlier patches in this series look good to me and are:
> 
> Acked-by: Hans de Goede <hdegoede@redhat.com>
> 
> 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: <http://lists.denx.de/pipermail/u-boot/attachments/20150724/def58005/attachment.sig>