|
|
Re: [PATCH 1/2] usb: change the dependency chain for LPC32XX and ISP1301 |
On 08/23/2012 10:29 AM, Sebastian Andrzej Siewior wrote: >> For the PNX4008 removal - should I already post a patch to build the >> respective changes on top of, without PNX4008 references? > > If nobody cares about PNX4008 and nobody tested if it actually works in > the OHCI mode I don't see the point in keeping it around especially if > there are rumors to remove the arch. > > It would be nice, if you could manage to compile this thingy without > I2C. In the end one could attach something different than isp1301, no? > So I think, in the perfect world you would probe for your phy and > notice that it is not there and print a message: "Dude! Select isp1301 > for OHCI, aight?!" Currently, we really need I2C for OHCI on LPC32XX, so actually need to depend on it. Removing the I2C calls in ohci-nxp.c is waiting for the phy framework so that I don't need to introduce new API in isp1301.c at this point. And right, in the ideal world, we have no I2C dependency and can control a phy in an abstract way, even without depending on ISP1301, which could be replaced by sth. else (not sure if this will be done/soldered practically, though). For now, will post patches for the current I2C dependency and the PNX4008 removal. Roland _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
[Linux ARM (vger)] [Linux ARM MSM] [Linux Omap] [Linux Arm] [Linux Tegra] [Fedora ARM] [eCos] [Linux Fastboot] [Gcc Help] [Git] [DCCP] [IETF Announce] [Security] [PDAs] [Linux] [Linux MIPS] [Yosemite Campsites] [Photos]
![]() |
![]() |