RE: [PATCH] [v2] power/fsl: add MDIO dt binding for FMan

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Friday, December 19, 2014 6:01 AM
> To: Xie Shaohui-B21989
> Cc: linuxppc-dev@xxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; Medve Emilian-
> EMMEDVE1; Liberman Igal-B31950
> Subject: Re: [PATCH] [v2] power/fsl: add MDIO dt binding for FMan
> 
> On Thu, 2014-12-18 at 06:53 -0600, Xie Shaohui-B21989 wrote:
> > Ping.
> >
> > Best Regards,
> > Shaohui Xie
> 
> I can't put patches in my -next until the merge window closes.
> 
> > > > > +EXAMPLE
> > > > > +
> > > > > +Example for FMan v2 external MDIO:
> > > > > +
> > > > > +mdio@f1000 {
> > > > > +	compatible = "fsl,fman-xmdio";
> > > > > +	reg = <0xf1000 0x1000>;
> > > > > +	bus-frequency = <20000>;
> > > > > +};
> > > >
> > > > So the bus frequency is only 20 KHz?  Or is the unit supposed to
> > > > be something other than Hz?
> > > [S.H] it's only an example, it could be different on real SoCs, but
> > > they always lower than the standard one, The standard one is 2.5MHz, I have
> to use Hz for it.
> 
> Is there any SoC for which 20 kHz is the right frequency?  I just want to make
> sure the example is realistic.
[S.H] the clock divider has a limitation that the MAX value it can get on Fman v2 is 255 (0xff, 8 bits),
On Fman v3 is 511(0x1ff, 9 bits).

So the lowest frequency on Fman v2 is: Fman_clock / (2 * 255),
On Fman v3 is: Fman_clock / ((2 * 511) + 1).

Take default Fman frequency setting from SDK1.7 as example, the lowest clock used for Fman v2 is 581MHz,
The lowest clock for Fman v3 is 600MHz.

Then the lowest bus frequency can get is:
Fman v2: ~1140KHz
Fman v3: ~587KHz

20KHz is not practice, we don't have a suggested value in errata document.
For this example, should I post a new version with a value like 1200KHz?

Thanks!
Shaohui
��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux