Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759410Ab3JOOqk (ORCPT ); Tue, 15 Oct 2013 10:46:40 -0400 Received: from iolanthe.rowland.org ([192.131.102.54]:58526 "HELO iolanthe.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1759171Ab3JOOqi (ORCPT ); Tue, 15 Oct 2013 10:46:38 -0400 Date: Tue, 15 Oct 2013 10:46:37 -0400 (EDT) From: Alan Stern X-X-Sender: stern@iolanthe.rowland.org To: Hartley Sweeten cc: Ryan Mallon , "linux-usb@vger.kernel.org" , ARM Kernel , Linux Kernel , "gregkh@linuxfoundation.org" Subject: RE: [PATCH] usb: ohci: remove ep93xx bus glue platform driver In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1915 Lines: 51 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-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/