Hi Balsam,
On 17/08/2022 10:07, bchihi@xxxxxxxxxxxx wrote:
From: Michael Kao <michael.kao@xxxxxxxxxxxx>
Add LVTS v4 (Low Voltage Thermal Sensor) driver to report junction
temperatures in MediaTek SoC mt8192 and register the maximum temperature
of sensors and each sensor as a thermal zone.
Thanks for your work
First of all, the patch is way too big.
The organization of the data is hard to understand.
Could you give a description of the sensors, how they are organized ?
I can see the there are 'tc' and each have a group of sensing points? Is
that correct? Do have the 'tc's a shared clock? etc ...
I have another email with the comments inline but without more insights
on the hardware it is difficult to review accurately. This driver looks
more complex than the other ones I've reviewed. At least that is what
looks like with the different macros names found.
--
<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
