linux-kernel - Re: [PATCH v2 0/2] Add a generic virtual thermal sensor

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-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 17 Sep 2021 16:03:35 +0200
From:   Daniel Lezcano <daniel.lezcano@...aro.org>
To:     Alexandre Bailon <abailon@...libre.com>, rui.zhang@...el.com,
        amitk@...nel.org
Cc:     linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org,
        ben.tseng@...iatek.com, khilman@...libre.com, mka@...omium.org
Subject: Re: [PATCH v2 0/2] Add a generic virtual thermal sensor

On 17/09/2021 15:33, Alexandre Bailon wrote:
> Hi Daniel,
> 
> On 9/17/21 2:41 PM, Daniel Lezcano wrote:
>> On 17/09/2021 09:27, Alexandre Bailon wrote:
>>> This series add a virtual thermal sensor.
>>> It could be used to get a temperature using some thermal sensors.
>>> Currently, the supported operations are max, min and avg.
>>> The virtual sensor could be easily extended to support others
>>> operations.
>>>
>>> Note:
>>> Currently, thermal drivers must explicitly register their sensors to
>>> make them
>>> available to the virtual sensor.
>>> This doesn't seem a good solution to me and I think it would be
>>> preferable to
>>> update the framework to register the list of each available sensors.
>> Why must the drivers do that ?
> Because there are no central place where thermal sensor are registered.
> The only other way I found was to update thermal_of.c,
> to register the thermal sensors and make them available later to the
> virtual thermal sensor.
> 
> To work, the virtual thermal need to get the sensor_data the ops from
> the thermal sensor.
> And as far I know, this is only registered in thermal_of.c, in the
> thermal zone data
> but I can't access it directly from the virtual thermal sensor.
> 
> How would you do it ?

Via the phandles when registering the virtual sensor ?

Or is the problem you are mentioning related to the sensor module
loading after the virtual sensor ?

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