Hello, On 15-04-14 09:06:27, Peter Chen wrote: > > > > > > > > > > > > > > > > > > Hi! > > > > > > > > > > > > > > > > Stefano, maybe you can pick 1,2,3,4,6 right away for next? > > > > > > > > This way, we can solve 5/6 separately, since it might need > > > > > > > > some further dicussion. I don't want that patch to block the > > > > > > > > rest of the series though. > > > > > > > > > > > > > > The sixth patch has some minor dependencies on the fifth patch > > > > > > > like some #defines and configuration we enable for Colibri. In > > > > > > > case the fifth patch will require some more discussion and > > > > > > > time, then I guess I will need to rework the sixth patch and > > > > > > > then everything related to USB can go in a later patch. The > > > > > > > rework will be minor. So the v3 will be without USB support > > > > > > > > > > > > Well, if you want to rework the 6th patch, that'd be very nice > > > > > > of you :) > > > > > > > > > > Yes sure. Will put the USB support in the end or not, the first > > > > > patches can go in atleast since there seem to be no complaints on > > > > > them so far. > > > > > > > > > > > > and USB will go in later once this gets applied. Perhaps we > > > > > > > will send the USB along with some more other featureset > > > > > > > addition like gpio support for Vybrid which my colleague did > > > > > > > recently along with some driver model/dt stuff. > > > > > > > > > > > > My impression is that the USB is just a matter of figuring out > > > > > > where the PHY registers go -- whether in > > > > > > arch/arm/include/imx-common or into include/usb/ . I'd be more > > > > > > inclined for the later, since the PHY regs looks like chipidea IP > > > > > > specific > > thing, but I might be wrong. > > > > > > > > > > I would say it looks like chipidea specific but I have not looked > > > > > at IPs other than Freescale controllers to be sure. > > > > > > > > I think it's CI-specific indeed, thus include/usb/ . +CC Peter, who > > > > might chime in whether the PHY is CI-specific or IMX-specific. > > > > > > Which PHY? Vybrid? In Linux kernel, it belongs to mxs PHY. > > > The same controller can integrate different PHYs. > > > > Yeah. Is the PHY MXS/IMX/VF specific or can that PHY be found even on other > > SoCs not manufactured by Freescale ? > > > > This mxs PHY is from SigmaTel, I only see this PHY is used at FSL SoCs, but > not sure > if others use it.
So Chipidea IP with the SigmaTel phy. It might be better to have the regs-usbphy.h viz. register phy definitions in imx-common then since it is not clear if SoC's other than Freescale's use the same phy. Marek, what do you think? - Sanchayan. > > Peter _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
