[PATCH v7 3/9] add Freescale SerDes PHY support
Liu Dave
DaveLiu at freescale.com
Wed Nov 21 14:48:42 EST 2007
> On Oct 19, 2007, at 10:35 AM, Grant Likely wrote:
>
> > On 10/19/07, Li Yang <leoli at freescale.com> wrote:
> >> The SerDes(serializer/deserializer) PHY block is a new SoC
> block used
> >> in Freescale chips to support multiple serial interfaces,
> such as PCI
> >> Express, SGMII, SATA.
> >
> > This looks like board setup behaviour. Shouldn't setting this up be
> > the responsibility firmware? And failing that, I think it should be
> > done directly by the platform setup function (in other
> words; make it
> > a helper function and call it at board setup time).
> Besides, you want
> > to provide guarantees that the board is setup correctly before the
> > device driver that uses it gets probed.
> >
> > Cheers,
> > g.
>
> Upon further review of all this I don't think this belongs in the
> kernel at all. This is one time setup and should be done in firmware.
The latest u-boot supports the serdes initialization for 837x.
So, it can be dropped from kernel for now.
Dave
More information about the Linuxppc-dev
mailing list