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: Tuesday, December 23, 2014 4:08 PM
> To: Xie Shaohui-B21989
> Cc: Medve Emilian-EMMEDVE1; linuxppc-dev@xxxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; Liberman Igal-B31950
> Subject: Re: [PATCH] [v2] power/fsl: add MDIO dt binding for FMan
> 
> On Tue, 2014-12-23 at 01:35 -0600, Xie Shaohui-B21989 wrote:
> > +- bus-frequency
> > +		Usage: optional
> > +		Value type: <u32>
> > +		Definition: Specifies external MDIO bus clock speed which is
> > +		different from MDIO standard 2.5MHz. Should be defined for
> SoCs
> > +		on which the standard one cannot work.
> >
> > What should I rephrase it? Replace the last sentence with "Should be
> > defined For SoCs on which a lower frequency than the standard is
> required."?
> 
> Neither of these work with Emil's scenario of a system that allows a
> faster-than-standard speed.
> 
> How about: "Definition: Specifies the external MDIO bus clock speed to be
> used, if different from the standard 2.5 MHz.  This may be due to the
> standard speed being unsupported (e.g. due to a hardware problem), or to
> advertise that all relevant components in the system support a faster
> speed."
[S.H] OK. I'll use this in V3.

> 
> > How about the value used in example?
> > Should 2.5MHz be used or a lower one?
> 
> If you don't have a realistic example to use, don't put it in the example
> at all.  2.5MHz is the worst example to use because that's the default
> and there'd be no reason to use the property at all.
[S.H] OK. The V3 will not have this in example.

Thank you all for reviewing!
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