On 16/03/2021 08:01, Michael Kao wrote:
> From: "michael.kao" <[email protected]>
> 
> Add thermal zone node to Mediatek MT8183 dts file.
> 
> Evaluate the thermal zone every 500ms while not cooling
> and every 100ms when passive cooling is performed.
> 
> Signed-off-by: Matthias Kaehlcke <[email protected]>
> Signed-off-by: Michael Kao <[email protected]>
> ---
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi | 85 ++++++++++++++++++++++++
>  1 file changed, 85 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi 
> b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 5b782a4769e7..d3550af06408 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -657,6 +657,87 @@
>                       status = "disabled";
>               };
>  
> +             thermal: thermal@1100b000 {
> +                     #thermal-sensor-cells = <1>;
> +                     compatible = "mediatek,mt8183-thermal";
> +                     reg = <0 0x1100b000 0 0x1000>;
> +                     clocks = <&infracfg CLK_INFRA_THERM>,
> +                              <&infracfg CLK_INFRA_AUXADC>;
> +                     clock-names = "therm", "auxadc";
> +                     resets = <&infracfg  MT8183_INFRACFG_AO_THERM_SW_RST>;
> +                     interrupts = <0 76 IRQ_TYPE_LEVEL_LOW>;
> +                     mediatek,auxadc = <&auxadc>;
> +                     mediatek,apmixedsys = <&apmixedsys>;
> +                     nvmem-cells = <&thermal_calibration>;
> +                     nvmem-cell-names = "calibration-data";
> +             };
> +
> +             thermal-zones {
> +                     cpu_thermal: cpu_thermal {
> +                             polling-delay-passive = <100>;
> +                             polling-delay = <500>;
> +                             thermal-sensors = <&thermal 0>;
> +                             sustainable-power = <5000>;
> +                     };
> +
> +                     /* The tzts1 ~ tzts6 don't need to polling */
> +                     /* The tzts1 ~ tzts6 don't need to thermal throttle */
> +
> +                     tzts1: tzts1 {
> +                             polling-delay-passive = <0>;
> +                             polling-delay = <0>;
> +                             thermal-sensors = <&thermal 1>;
> +                             sustainable-power = <5000>;
> +                             trips {};
> +                             cooling-maps {};
> +                     };

What is the point of defining the sustainable power with no cooling
device associated ?



-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

Reply via email to