On Mon, 14 Oct 2013, Hartley Sweeten wrote: > >> config USB_OHCI_HCD_PLATFORM > >> tristate "Generic OHCI driver for a platform device" > >> - default n > >> + default y if ARCH_EP93XX > > > > Shouldn't we select USB_OHCI_HCD_PLATFORM, e.g. something like: > > > > config ARCH_EP93XX_USB > > tristate "USB OHCI support" > > default y > > select USB_OHCI_HCD_PLATFORM > > > > In arch/arm/mach-ep93xx/Kconfig rather than polluting > > drivers/usb/host/Kconfig with arch specific stuff? > > I wasn't sure where the best place to enable > USB_OHCI_HCD_PLATFORM would be. > > Currently USB support on the EP93xx only needs USB_OHCI_HCD > enabled, which is already enabled in the ep93xx_defconfig. I'm not > sure if adding the config option above would create a problem where > the user would need to enable USB_OHCI_HCD in drivers/usb then > have to go back to the arch stuff to enable ARCH_EP93XX_USB. > > With the default y above they just have to enable USB_OHCI_HCD > like they currently do. > > I'm hoping Alan can provide some feedback. In the past this sort of thing has been done in two different ways, depending on whether or not OHCI support was previously configurable. In cases where it was, we kept the old Kconfig entry and made it select USB_OHCI_HCD_PLATFORM, but added a notice that the entry was now deprecated. For example, in drivers/usb/host/Kconfig see the entry for USB_OHCI_ATH79. In cases where support was always present (i.e., not configurable), we added an entry for USB_OHCI_HCD_PLATFORM to the platform's defconfig file. For example, see arch/arm/configs/marzen_defconfig. This is probably what you want to do. Alan Stern -- 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
