Hi Eduardo,
Thank you for your comment.
On Mon, 29 May 2017 09:30:09 -0700 Eduardo Valentin <edubez...@gmail.com> wrote:
> On Mon, May 29, 2017 at 06:15:44PM +0900, Kunihiko Hayashi wrote:
> > Add nodes of thermal monitor and thermal zone for UniPhier PXs2 SoC.
> > The thermal monitor is included in sysctrl.
> >
> > Signed-off-by: Kunihiko Hayashi <hayashi.kunih...@socionext.com>
> > ---
> > arch/arm/boot/dts/uniphier-pxs2-gentil.dts | 21 +++++++++++++++++++++
> > arch/arm/boot/dts/uniphier-pxs2-vodka.dts | 21 +++++++++++++++++++++
> > arch/arm/boot/dts/uniphier-pxs2.dtsi | 6 ++++++
> > 3 files changed, 48 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
> > b/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
> > index 373818a..89ccb88 100644
> > --- a/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
> > +++ b/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
> > @@ -70,6 +70,27 @@
> > i2c5 = &i2c5;
> > i2c6 = &i2c6;
> > };
> > +
> > + thermal-zones {
> > + cpu_thermal {
> > + polling-delay-passive = <250>; /* 250ms */
> > + polling-delay = <1000>; /* 1000ms */
> > + thermal-sensors = <&pvtctl>;
> > +
> > + trips {
> > + cpu_crit: cpu_crit {
> > + temperature = <95000>; /* 95C */
> > + hysteresis = <2000>;
> > + type = "critical";
> > + };
> > + cpu_alert: cpu_alert {
> > + temperature = <85000>; /* 85C */
> > + hysteresis = <2000>;
> > + type = "passive";
> > + };
> > + };
> These are missing cooling-maps section.
I see. I'll add the section including another nodes.
> > + };
> > + };
> > };
> >
> > &serial2 {
> > diff --git a/arch/arm/boot/dts/uniphier-pxs2-vodka.dts
> > b/arch/arm/boot/dts/uniphier-pxs2-vodka.dts
> > index 51a3eac..0201584 100644
> > --- a/arch/arm/boot/dts/uniphier-pxs2-vodka.dts
> > +++ b/arch/arm/boot/dts/uniphier-pxs2-vodka.dts
> > @@ -68,6 +68,27 @@
> > i2c5 = &i2c5;
> > i2c6 = &i2c6;
> > };
> > +
> > + thermal-zones {
> > + cpu_thermal {
> > + polling-delay-passive = <250>; /* 250ms */
> > + polling-delay = <1000>; /* 1000ms */
> > + thermal-sensors = <&pvtctl>;
> > +
> > + trips {
> > + cpu_crit: cpu_crit {
> > + temperature = <95000>; /* 95C */
> > + hysteresis = <2000>;
> > + type = "critical";
> > + };
> > + cpu_alert: cpu_alert {
> > + temperature = <85000>; /* 85C */
> > + hysteresis = <2000>;
> > + type = "passive";
> > + };
> > + };
>
> same
>
> > + };
> > + };
> > };
> >
> > &serial2 {
> > diff --git a/arch/arm/boot/dts/uniphier-pxs2.dtsi
> > b/arch/arm/boot/dts/uniphier-pxs2.dtsi
> > index e9e031d..072e3b4 100644
> > --- a/arch/arm/boot/dts/uniphier-pxs2.dtsi
> > +++ b/arch/arm/boot/dts/uniphier-pxs2.dtsi
> > @@ -387,6 +387,12 @@
> > compatible = "socionext,uniphier-pxs2-reset";
> > #reset-cells = <1>;
> > };
> > +
> > + pvtctl: pvtctl {
> > + compatible = "socionext,uniphier-pxs2-thermal";
> > + interrupts = <0 3 1>;
> > + #thermal-sensor-cells = <0>;
> > + };
> > };
> > };
> > };
> > --
> > 2.7.4
> >
Best Regards,
Kunihiko Hayashi
