On Thu, Feb 10, 2011 at 01:48:54PM +0200, Felipe Balbi wrote: > Hi, > > On Thu, Feb 10, 2011 at 05:07:06PM +0530, Premi, Sanjeev wrote: > > > diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig > > > index 5a7c8f1..ac137d1 100644 > > > --- a/drivers/usb/Kconfig > > > +++ b/drivers/usb/Kconfig > > > @@ -10,8 +10,6 @@ menuconfig USB_SUPPORT > > > This option adds core support for Universal Serial Bus (USB). > > > You will also need drivers from the following menu to make use > > > of it. > > > > > > -if USB_SUPPORT > > > - > > > # Host-side USB depends on having a host controller > > > # NOTE: dummy_hcd is always an option, but it's ignored here ... > > > # NOTE: SL-811 option should be board-specific ... > > > @@ -59,16 +57,9 @@ config USB_ARCH_HAS_OHCI > > > # some non-PCI hcds implement EHCI > > > config USB_ARCH_HAS_EHCI > > > boolean > > > - default y if PPC_83xx > > > - default y if PPC_MPC512x > > > - default y if SOC_AU1200 > > > - default y if ARCH_IXP4XX > > > - default y if ARCH_W90X900 > > > - default y if ARCH_AT91SAM9G45 > > > - default y if ARCH_MXC > > > - default y if ARCH_OMAP3 > > > - default y if ARCH_CNS3XXX > > > - default PCI > > > + default n > > > + > > > +if USB_SUPPORT > > > > > > # ARM SA1111 chips have a non-PCI based "OHCI-compatible" USB host > > > interface. > > > config USB > > > > > > and expect ARCH Kconfigs to select the features they have. > > > > > > Greg, what do you think ? > > > > > > > I noticed same problem when building kernel from latest master today. > > > > Felipe, Greg: > > Do you have agreement on what would be good fix for this warning? > > Yeah, I never got a reply about the proposal above. Greg, what do you > think ? I don't know, I can't remember what the outcome of the "arches need to select the proper Kconfig options" discussion ended up being. Anyone remember? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
