Thermal Framework, Thermal Driver, Thermal Aggregation

Balsam CHIHI bchihi at baylibre.com
Fri Sep 2 08:36:19 PDT 2022


Hello everybody,

I'm currently waiting for reviews of the LVTS Driver v9 series that I
recently submitted
"https://patchwork.kernel.org/project/linux-mediatek/cover/20220817080757.352021-1-bchihi@baylibre.com/",
then I will send the v10 with additional mt8195 specific code.
The upcoming v10 of the LVTS series will support Daniel's new Thermal
Framework implementation
"https://patchwork.kernel.org/project/linux-pm/cover/20220804224349.1926752-1-daniel.lezcano@linexp.org/",
plus new changes that may be requested from reviews of the v9.

After that, I will submit another series about Thermal Aggregation.
Basically, I need to create an additional thermal_zone with all
sensors registered to it, this will allow us to return MAX, AVG, or
MIN temperature based on all sensor values within this new
thermal_zone (could we call it Virtual?).
A series for the same purpose are sent by Alexandre Bailon
"https://patchwork.kernel.org/project/linux-pm/cover/20220218084604.1669091-1-abailon@baylibre.com/",
the implementation is done in "thermal_of.c" (I'm continuing on
Alexander's work). According to comments, other platforms not based on
Device Tree should benefit from this feature. So, according to Daniel
Lezcano, it should be implemented in "thermal_core.c" instead.

So, I would like to discuss it with you, and ask for recommendations
and help on how to implement this feature, because it is not easy for
me alone to figure out how to do it.
Should we update the thermal zone to support aggregation? or add a way
to register a thermal zone that aggregates other thermal zones?
what about applying some configuration to a multiple sensor thermal
zone? does it override the trip already defined in mono sensor thermal
zones?
And many other questions that should be asked to implement this new feature...
In my honest opinion, we should clarify together how it must be done.

Thank you in advance.

Best regards,
Balsam CHIHI



More information about the Linux-mediatek mailing list