On Wed, Apr 18, 2012 at 09:18:23AM +0000, Chen Peter-B29397 wrote: > > > > > > > > > > This driver will handle i.MX233/i.MX28 and I hope soon i.MX6Q. I > > tried to > > > > keep > > > > this separate from the MXC EHCI to avoid further polution of the MXC > > EHCI, > > > > though eventually these two might be merged. > > > > > > > As mxc and mxs platforms use the same USB controller (chipidea's), I > > wonder if > > > you can use ehci-mxc, or change name from ehci-mxc to ehci-imx? > > > Sascha, what's your opinion? > > > > Maybe better ehci-imx. I think it's a good idea to not use the ehci-mxc > > glue code but instead write a mxs specific glue code with having in > > mind that later we want to drop ehci-mxc and use ehci-imx for all imx > > including mxs. This assumes the USB folks are ok with this, I don't > > know. > > > How about create a patch changing name from ehci-mxc to ehci-imx first, then add > mx28/mx23/mx6q host ehci driver. I can do it. ehci-imx won't need clock handling and it won't need phy handling. Also the setup of the portsc register won't be in the new driver. The clk_get(dev, "usb_phy1") part is horribly broken anyway. It's more than just a rename. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- 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
