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." > 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. -Scott -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
