| Message ID | 1380251605-3804-12-git-send-email-eduardo.valentin@ti.com |
|---|---|
| State | New |
| Headers | show |
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@baylibre.com> > Cc: Tony Lindgren <tony@atomide.com> > Cc: Russell King <linux@arm.linux.org.uk> > Cc: linux-omap@vger.kernel.org > Cc: devicetree-discuss@lists.ozlabs.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> > --- > 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? > }; > }; > -- > 1.8.2.1.342.gfa7285d > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
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@baylibre.com> >> Cc: Tony Lindgren <tony@atomide.com> >> Cc: Russell King <linux@arm.linux.org.uk> >> Cc: linux-omap@vger.kernel.org >> Cc: devicetree-discuss@lists.ozlabs.org >> Cc: linux-arm-kernel@lists.infradead.org >> Cc: linux-kernel@vger.kernel.org >> Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> >> --- >> 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. Tks Nishanth >> }; >> }; >> -- >> 1.8.2.1.342.gfa7285d >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >
On Fri, Sep 27, 2013 at 8:20 AM, Eduardo Valentin <eduardo.valentin@ti.com> 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@baylibre.com> >>> Cc: Tony Lindgren <tony@atomide.com> >>> Cc: Russell King <linux@arm.linux.org.uk> >>> Cc: linux-omap@vger.kernel.org >>> Cc: devicetree-discuss@lists.ozlabs.org >>> Cc: linux-arm-kernel@lists.infradead.org >>> Cc: linux-kernel@vger.kernel.org >>> Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> >>> --- >>> 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
On 27-09-2013 09:26, Nishanth Menon wrote: > On Fri, Sep 27, 2013 at 8:20 AM, Eduardo Valentin > <eduardo.valentin@ti.com> 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@baylibre.com> >>>> Cc: Tony Lindgren <tony@atomide.com> >>>> Cc: Russell King <linux@arm.linux.org.uk> >>>> Cc: linux-omap@vger.kernel.org >>>> Cc: devicetree-discuss@lists.ozlabs.org >>>> Cc: linux-arm-kernel@lists.infradead.org >>>> Cc: linux-kernel@vger.kernel.org >>>> Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> >>>> --- >>>> 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.. Yes, it will be cleaned. ;-) Tks. > > Regards, > Nishanth Menon > >
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>; }; };
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@baylibre.com> Cc: Tony Lindgren <tony@atomide.com> Cc: Russell King <linux@arm.linux.org.uk> Cc: linux-omap@vger.kernel.org Cc: devicetree-discuss@lists.ozlabs.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> --- arch/arm/boot/dts/omap443x.dtsi | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)