On 20-11-2013 08:32, Pavel Machek wrote: > HI! > >> This patch changes the dtsi entry on omap4430 to contain >> the thermal data. This data will enable the passive >> cooling with CPUfreq cooling device at 100C and the >> system will do a thermal shutdown at 125C. >> >> Cc: "Benoît Cousson" <[email protected]> >> Cc: Tony Lindgren <[email protected]> >> Cc: Russell King <[email protected]> >> Cc: [email protected] >> Cc: [email protected] >> Cc: [email protected] >> Cc: [email protected] >> Signed-off-by: Eduardo Valentin <[email protected]> >> --- >> arch/arm/boot/dts/omap443x.dtsi | 10 ++++++++-- >> 1 file changed, 8 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm/boot/dts/omap443x.dtsi >> b/arch/arm/boot/dts/omap443x.dtsi >> index bcf455e..e9c97d6 100644 >> --- a/arch/arm/boot/dts/omap443x.dtsi >> +++ b/arch/arm/boot/dts/omap443x.dtsi >> @@ -12,7 +12,7 @@ >> >> / { >> cpus { >> - cpu@0 { >> + cpu0: cpu@0 { >> /* OMAP443x variants OPP50-OPPNT */ >> operating-points = < >> /* kHz uV */ > > I see you also add labels to various other entries...
I think I didn't quite get your point here. Can you please elaborate?
This label has been added because it will be used in thermal zone while
referencing the cooling device.
>
>> @@ -25,9 +25,15 @@
>> };
>> };
>>
>> - bandgap {
>> + thermal-zones{
>
> You may want to include space here.
OK.
>
>> + #include "omap4-cpu-thermal.dtsi"
>> + };
>> +
>> + bandgap: bandgap {
>> reg = <0x4a002260 0x4
>> 0x4a00232C 0x4>;
>> compatible = "ti,omap4430-bandgap";
>> +
>> + #thermal-sensor-cells = <0>;
>> };
>> };
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
