[U-Boot] [PATCH v2 0/6] Add support for Colibri Vybrid Modules

[maitysanchayan at gmail.com]

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