On 02/18/2013 04:30 AM, Wei Ni wrote: > Enable thermal driver in the dts file. > Set sensor as lm90 remote sensor, and set throttle data. > diff --git a/Documentation/devicetree/bindings/thermal/tegra3-thermal.txt b/Documentation/devicetree/bindings/thermal/tegra3-thermal.txt The DT documentation should be either a separate patch before the one which implements it in the driver, or part of the driver patch. It shouldn't be part of the patch which instantiates the binding in the board DT file. > +* Nvidia Tegra30 Thermal NVIDIA is all capitals. It seems like a word is missing: Thermal Management? Control/Zone? > +** Thermal node properties: > + > +- compatible : "nvidia,tegra30-thermal"; > +- sensors: the sensor device node which we want to use in the thermal zone, > + the arguments is the index of the sensor in sensor device node; The arguments should be defined by the DT binding documentation for that referenced device, not by the referencing device. DT properties that are custom defined for this binding should include the vendor prefix in their name. So, that'd be "nvidia,sensors". This may apply less if a "sensors" becomes a more generic standard, but it surely applies to all the other properties below. > +- passive-delay: passive delay; What does this mean? > +- num-passive-trips : number of passive trip points, this is required, set > + it 0 if none, if greater than 0, the following properties must be defined; > +- passive-trips : temperature of passive trip points; Presumably "num-passive-trips" can be calculated simply by counting the number of entries in "passive-trips"? Please describe what passive and active trips are; someone who just reads this binding document (perhaps in conjunction with some HW documentation and/or other binding documentation) should be able to fill in this binding without other knowledge. > +- num-active-trips: number of active trip points, this is required, set > + it 0 if none, if greater than 0, the following properties must be defined; > +- active-trips: temperature of active trip points; Both comments above apply here too. > +- throt-tab-size: size of the throttle table, it's the max cooling state. > +- throt-tab: throttle table. the cooling state will be defined according to > + this table. Both comments above apply here too. Also, what units are used for all these properties? Judging by the example below, this property is a list of tuples. The meaning of each field in the tuple needs to be explained. What happens when the CPU/SoC needs to be throttled? Must some clock or voltage be lowered/limited? If so, you need properties that indicate which clock/voltage/... needs to be acted upon. > +Usually these properties are separated in board related dts files. That's not really relevant. _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
