Hi Rajeshwari, On Tue, Nov 5, 2019 at 3:14 AM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote: > > Quoting Rajeshwari (2019-11-01 05:08:28) > > diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/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 @@ > > }; > > }; > > > > + thermal-zones { > > + aoss-0-usr { > > + polling-delay-passive = <0>; > > + polling-delay = <0>; > > Can we get real polling delays instead of 0? > > > + thermal-governor = "user_space"; > > + thermal-sensors = <&tsens0 0>; > > + wake-capable-sensor; > > What is this property? Downstream property. If you need this, propose changes to the tsens bindings. > > + trips { > > + active-config0 { > > + temperature = <125000>; > > + hysteresis = <1000>; > > + type = "passive"; > > + }; > > + reset-mon-cfg { > > + temperature = <115000>; > > + hysteresis = <5000>; > > + type = "passive"; > > + }; > > + }; > > + }; > > + > > + cpu-0-0-usr { > > + polling-delay-passive = <0>; > > + polling-delay = <0>; > > + thermal-governor = "user_space"; > > What is this property? Based on a downstream property that was rejected upstream. Please get rid of it. Regards, Amit