Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Message-ID: <5dc09bbb.1c69fb81.196e5.9770@mx.google.com>
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <1572610108-1363-2-git-send-email-rkambl@codeaurora.org>
References: <1572610108-1363-1-git-send-email-rkambl@codeaurora.org> <1572610108-1363-2-git-send-email-rkambl@codeaurora.org>
Cc:     linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, sivaa@codeaurora.org,
        sanm@codeaurora.org, Rajeshwari <rkambl@codeaurora.org>,
        Amit Kucheria <amit.kucheria@linaro.org>
Subject: Re: [PATCH 1/1] arm64: dts: qcom: sc7180:  Add device node support for TSENS in SC7180
From:   Stephen Boyd <swboyd@chromium.org>
To:     Andy Gross <agross@kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Rajeshwari <rkambl@codeaurora.org>,
        Rob Herring <robh+dt@kernel.org>
User-Agent: alot/0.8.1
Date:   Mon, 04 Nov 2019 13:44:27 -0800
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Quoting Rajeshwari (2019-11-01 05:08:28)
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/q=
com/sc7180.dtsi
> index 07ea393..06ded1d 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -449,6 +465,508 @@
>                 };
>         };
> =20
> +       thermal-zones {
> +               aoss-0-usr {
> +                       polling-delay-passive =3D <0>;
> +                       polling-delay =3D <0>;

Can we get real polling delays instead of 0?

> +                       thermal-governor =3D "user_space";
> +                       thermal-sensors =3D <&tsens0 0>;
> +                       wake-capable-sensor;

What is this property?

> +                       trips {
> +                               active-config0 {
> +                                       temperature =3D <125000>;
> +                                       hysteresis =3D <1000>;
> +                                       type =3D "passive";
> +                               };
> +                               reset-mon-cfg {
> +                                       temperature =3D <115000>;
> +                                       hysteresis =3D <5000>;
> +                                       type =3D "passive";
> +                               };
> +                       };
> +               };
> +
> +               cpu-0-0-usr {
> +                       polling-delay-passive =3D <0>;
> +                       polling-delay =3D <0>;
> +                       thermal-governor =3D "user_space";

What is this property?