Messages in this thread | | | Date | Sat, 1 Jul 2023 16:20:10 +0200 | Subject | Re: [PATCH 1/1] thermal: sysfs: avoid actual readings from sysfs | From | Daniel Lezcano <> |
| |
Hi Eduardo,
On 01/07/2023 03:38, Eduardo Valentin wrote: > Hey Daniel, > > On Fri, Jun 30, 2023 at 12:11:25PM +0200, Daniel Lezcano wrote: >> >> >> >> Hi Rafael, >> >> On 30/06/2023 10:16, Rafael J. Wysocki wrote: >>> On Wed, Jun 28, 2023 at 11:10 PM Eduardo Valentin <evalenti@kernel.org> wrote: >> >> [ ... ] >> >>> So what about adding a new zone attribute that can be used to specify >>> the preferred caching time for the temperature? >>> >>> That is, if the time interval between two consecutive updates of the >>> cached temperature value is less than the value of the new attribute, >>> the cached temperature value will be returned by "temp". Otherwise, >>> it will cause the sensor to be read and the value obtained from it >>> will be returned to user space and cached. >>> >>> If the value of the new attribute is 0, everything will work as it >>> does now (which will also need to be the default behavior). >> >> I'm still not convinced about the feature. >> >> Eduardo provided some numbers but they seem based on the characteristics >> of the I2C, not to a real use case. Eduardo? > > Why I2C is not a real use case?
What I meant is "I2C is slow, ok. But what is the setup where the problem arises?"
-- <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
| |