MIME-Version: 1.0
References: <20230209105628.50294-1-bchihi@baylibre.com> <b41930f9-89d9-1b12-b024-05fb12700769@linaro.org>
In-Reply-To: <b41930f9-89d9-1b12-b024-05fb12700769@linaro.org>
From:   Balsam CHIHI <bchihi@baylibre.com>
Date:   Fri, 10 Feb 2023 10:12:47 +0100
Message-ID: <CAGuA+oo+1tSDhVkrc-2feH9e8wMpBgAa-N1VEOuGN8_c=uo1zw@mail.gmail.com>
Subject: Re: [PATCH v14 0/6] Add LVTS Thermal Architecture
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     angelogioacchino.delregno@collabora.com, rafael@kernel.org,
        amitk@kernel.org, rui.zhang@intel.com, matthias.bgg@gmail.com,
        robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        rdunlap@infradead.org, ye.xingchen@zte.com.cn,
        p.zabel@pengutronix.de, linux-pm@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org,
        khilman@baylibre.com, james.lo@mediatek.com,
        rex-bc.chen@mediatek.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Daniel,

On Fri, Feb 10, 2023 at 7:44 AM Daniel Lezcano
<daniel.lezcano@linaro.org> wrote:
>
> On 09/02/2023 11:56, bchihi@baylibre.com wrote:
> > From: Balsam CHIHI <bchihi@baylibre.com>
> >
> > The LVTS (Low Voltage Thermal Sensor) driver is capable of monitoring
> > multiple hot points. For that, it contains 7 thermal control blocks
> > dedicated to specific devices on the die. Each control block can handle
> > up to 4 sensors.
> >
> > The thermal controller supports several interrupts. One for the cold
> > trip point, the hot trip point, the return to the normal trip point,
> > and a specific programmable trip point to monitor the temperature
> > dynamically.
> >
> > The temperature measurement can be done in two ways, the immediate mode
> > where the temperature read is instantaneous and the filtered mode where
> > the controller uses, by configuration, an average of a set of values
> > removing the minimum and the maximum.
> >
> > Finally, it is composed of 2 finite-state machines responsible for
> > the state of the temperature (cold, hot, hot 2 normal, hot hot),
> > the triggering of the interrupts, and the monitoring of the temperature=
.
> >
> > As requested, the thermal driver has been reworked to reduce
> > the complexity of the code. At this time, the 4 little CPUs and
> > the 4 big CPUs are supported by the thermal driver.They are described
> > in a data structure and more devices can be added later.
> > The calibration routine has been simplified also.
> >
> > The series provide the following changes:
> >   - Move the Mediatek drivers inside a dedicated folder as their number
> >     is increasing
> >   - Add the DT bindings for the controller
> >   - Add the efuse node for the mt8195
> >   - The LVTS driver
> >   - The thermal zones description in the DT
>
> Applied patch 1,2 and 4 for v6.3

Thank you very much!

>
> Patches 5 and 6 should go through the Mediatek tree.
>
> Thanks!
>
>    -- Daniel
>
>
> --
> <http://www.linaro.org/> Linaro.org =E2=94=82 Open source software for AR=
M SoCs
>
> Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
> <http://twitter.com/#!/linaroorg> Twitter |
> <http://www.linaro.org/linaro-blog/> Blog
>

Best regards,
Balsam