On Fri, Sep 27, 2013 at 8:20 AM, Eduardo Valentin <eduardo.valentin@xxxxxx> wrote: > On 27-09-2013 08:24, Nishanth Menon wrote: >> On 23:13-20130926, Eduardo Valentin wrote: >>> 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" <bcousson@xxxxxxxxxxxx> >>> Cc: Tony Lindgren <tony@xxxxxxxxxxx> >>> Cc: Russell King <linux@xxxxxxxxxxxxxxxx> >>> Cc: linux-omap@xxxxxxxxxxxxxxx >>> Cc: devicetree-discuss@xxxxxxxxxxxxxxxx >>> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx >>> Cc: linux-kernel@xxxxxxxxxxxxxxx >>> Signed-off-by: Eduardo Valentin <eduardo.valentin@xxxxxx> >>> --- >>> 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 */ >>> @@ -25,9 +25,15 @@ >>> }; >>> }; >>> >>> - bandgap { >>> + thermal-zones{ >>> + #include "omap4-cpu-thermal.dtsi" >>> + }; >>> + >>> + bandgap: bandgap { >>> reg = <0x4a002260 0x4 >>> 0x4a00232C 0x4>; >>> compatible = "ti,omap4430-bandgap"; >>> + >>> + #thermal-sensor-cells = <0>; >> 2 cents: >> we might want to move bandgap under ocp? >> > > Yes, agreed. But I don't think this change is related to this patch or > to this series. Correct - this comment is not related to the current patch, hence my 2 cents ;) - will be nice to have it cleaned up though.. Regards, Nishanth Menon -- 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