linux-kernel - Re: [v4,0/7] Add Mediatek thermal dirver and dtsi

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 21 May 2020 14:51:42 +0200
From:   Matthias Brugger <matthias.bgg@...il.com>
To:     Michael Kao <michael.kao@...iatek.com>,
        Zhang Rui <rui.zhang@...el.com>,
        Eduardo Valentin <edubezval@...il.com>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>, hsinyi@...omium.org,
        linux-pm@...r.kernel.org, srv_heupstream@...iatek.com
Cc:     devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-mediatek@...ts.infradead.org
Subject: Re: [v4,0/7] Add Mediatek thermal dirver and dtsi

Hi Michael,

On 23/03/2020 13:15, Michael Kao wrote:
> This patchset supports for MT8183 chip to mtk_thermal.c.
> Add thermal zone of all the thermal sensor in SoC for
> another get temperatrue. They don't need to thermal throttle.
> And we bind coolers for thermal zone nodes of cpu_thermal.
> 
> Rebase to kernel-5.6-rc1.
> 
> Update content:
> 
> [1/7]
>     - Squash thermal zone settings in the dtsi from [v3,5/8]
>       arm64: dts: mt8183: Increase polling frequency for CPU thermal zone
> 
>     - Remove the property of interrupts and mediatek,hw-reset-temp
> 
> [2/7]
>     - Correct commit message
> 
> [4/7]
>     - Change the target temperature to the 80C and change the commit message
> 
> [6/7]
>     - Adjust newline alignment
> 
>     - Fix the judgement on the return value of registering thermal zone
> 
> This patch series base on these patches [1].
> 
> [v7,3/3] PM / AVS: SVS: Introduce SVS engine (https://patchwork.kernel.org/patch/11439829/)
> 
> Matthias Kaehlcke (1):
>   arm64: dts: mt8183: Configure CPU cooling
> 
> Michael Kao (6):
>   arm64: dts: mt8183: add thermal zone node
>   arm64: dts: mt8183: add dynamic power coefficients
>   arm64: dts: mt8183: Add #cooling-cells to CPU nodes
>   thermal: mediatek: mt8183: fix bank number settings

Do I understand correctly that we need to fix the bank number before we can add
the device tree changes. And that the last two patches are enhancements for the
driver but needed to get a working version?

Regards,
Matthias

>   thermal: mediatek: add another get_temp ops for thermal sensors
>   thermal: mediatek: use spinlock to protect PTPCORESEL
> 
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi | 156 +++++++++++++++++++++++
>  drivers/thermal/mtk_thermal.c            |  88 +++++++++++--
>  2 files changed, 231 insertions(+), 13 deletions(-)
>