Messages in this thread | | | Subject | Re: [PATCH v2 0/2] Add a generic virtual thermal sensor | From | Daniel Lezcano <> | Date | Fri, 17 Sep 2021 16:03:35 +0200 |
| |
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
| |