linux-kernel - Re: [PATCH_V3 0/6] thermal: stm32: driver improvements

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Mon, 27 Jan 2020 15:39:00 +0100
From:   Daniel Lezcano <daniel.lezcano@...aro.org>
To:     Pascal PAILLET-LME <p.paillet@...com>,
        "rui.zhang@...el.com" <rui.zhang@...el.com>,
        "edubezval@...il.com" <edubezval@...il.com>,
        "amit.kucheria@...durent.com" <amit.kucheria@...durent.com>,
        "mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
        Alexandre TORGUE <alexandre.torgue@...com>,
        David HERNANDEZ SANCHEZ <david.hernandezsanchez@...com>,
        "horms+renesas@...ge.net.au" <horms+renesas@...ge.net.au>,
        "wsa+renesas@...g-engineering.com" <wsa+renesas@...g-engineering.com>,
        "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
        "linux-stm32@...md-mailman.stormreply.com" 
        <linux-stm32@...md-mailman.stormreply.com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH_V3 0/6] thermal: stm32: driver improvements

On 27/01/2020 14:17, Pascal PAILLET-LME wrote:
> The goal of this patchset is to improve and simplify the stm32 thermal
>> driver:
>> * let the framework handle the trip points: it very is hard to split this patch
>> into smaller ones and keep each individual patch functional.
>> * fix interrupt management to avoid receiving hundreds of
>> interrupts when the temperature is close to the low threshold.
>> * improve temperature reading resolution
>> * the dirver is backawrd compatible.
>>
>> Pascal Paillet (6):
>> changes in v2:
>> * Split "handle multiple trip points" patch to make ones:
>> *   rework sensor mode management
>> *   disable interrupts at probe
>> * "remove hardware irq handler" is squashed in "handle multiple trip points"
> kind reminder for review
>>    thermal: stm32: fix icifr register name
>>    thermal: stm32: rework sensor mode management
>>    thermal: stm32: disable interrupts at probe
>>    thermal: stm32: handle multiple trip points
>>    thermal: stm32: improve temperature computing
>>    thermal: stm32: fix low threshold interrupt flood
>>
>>   drivers/thermal/st/stm_thermal.c | 388 ++++++++++---------------------
>>   1 file changed, 120 insertions(+), 268 deletions(-)

The series looks good to me. I've applied it for kernelci, if it is good
it will go to the next branch and be part of the next pull request.

In the future, do not join the PATCH keyword with the version. That
puzzles patchwork.

Thanks

  -- Daniel


-- 
 <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