[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.44L0.1301231018390.2045-100000@iolanthe.rowland.org>
Date: Wed, 23 Jan 2013 10:22:48 -0500 (EST)
From: Alan Stern <stern@...land.harvard.edu>
To: Sascha Hauer <s.hauer@...gutronix.de>
cc: Arnd Bergmann <arnd@...db.de>,
Manjunath Goudar <manjunath.goudar@...aro.org>, <balbi@...com>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-kernel@...r.kernel.org>, <arm@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
<linux-usb@...r.kernel.org>, Shawn Guo <shawn.guo@...aro.org>,
Jason Cooper <jason@...edaemon.net>,
Andrew Lunn <andrew@...n.ch>,
Gregory Clement <gregory.clement@...e-electrons.com>
Subject: Re: [PATCH 13/15] USB: ehci: make orion and mxc bus glues coexist
On Tue, 22 Jan 2013, Sascha Hauer wrote:
> On Tue, Jan 22, 2013 at 03:38:55PM +0000, Arnd Bergmann wrote:
> > On Tuesday 22 January 2013, Alan Stern wrote:
> > > In order to prevent this, you have to make sure that each glue driver
> > > depends on USB_ARCH_HAS_EHCI. A simple way to do this is to surround
> > > the Kconfig entries for those drivers with "if USB &&
> > > USB_ARCH_HAS_EHCI" ... "endif".
> >
> > I was actually thinking we could remove the use of USB_ARCH_HAS_EHCI
> > as well once we have inverted the logic for selecting USB_EHCI_HCD,
> > but there is another problem with that, because then we still need
> > something to select USB_ARCH_HAS_HCD, or kill that symbol as well.
>
> +1 for killing it. Such symbols get more and more meaningless anyway
> with multiarch kernels-
I tend to agree. Anyway, the symbols are named badly. _Any_
architecture with HAS_IOMEM can in theory aupport host-side USB.
Maybe some of the platforms don't, but that has nothing to do with the
CPU's architecture.
Furthermore, even platforms that don't support USB host controllers
can use things like dummy-hcd and usbip, which provide virtual host
controllers.
Anybody think these symbols should be retained?
Alan Stern
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists