| Message ID | 20220111103346.2660639-1-abailon@baylibre.com |
|---|---|
| Headers | show |
| Series | Add a generic virtual thermal sensor | expand |
Daniel, Amit, Alexandre Bailon <abailon@baylibre.com> writes: > 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. > > Changes in v2: > - Fix some warnings / errors reported by kernel test robot > - rename some struct and functions with a more accurate name > - update the dt bindings: rename type attribute to aggregation-function > - factorize a little bit the aggregation functions > Changes in v3: > - Aggregate thermal zone instead of thermal sensors > - Use try_get_module / put_module to prevent thermal providers to be removed > - Update the bindings, to be more accurate > Changes in v4: > - Fix two warnings reported by kernel test robot Any more feedback on this series? Thanks, Kevin
On Mon, Jan 24, 2022 at 5:58 PM Kevin Hilman <khilman@baylibre.com> wrote: > > Daniel, Amit, > > Alexandre Bailon <abailon@baylibre.com> writes: > > > 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. > > > > Changes in v2: > > - Fix some warnings / errors reported by kernel test robot > > - rename some struct and functions with a more accurate name > > - update the dt bindings: rename type attribute to aggregation-function > > - factorize a little bit the aggregation functions > > Changes in v3: > > - Aggregate thermal zone instead of thermal sensors > > - Use try_get_module / put_module to prevent thermal providers to be removed > > - Update the bindings, to be more accurate > > Changes in v4: > > - Fix two warnings reported by kernel test robot > > Any more feedback on this series? Hopefully, I'll get to it this week and I'll let you know.
"Rafael J. Wysocki" <rafael@kernel.org> writes: > On Mon, Jan 24, 2022 at 5:58 PM Kevin Hilman <khilman@baylibre.com> wrote: >> >> Daniel, Amit, >> >> Alexandre Bailon <abailon@baylibre.com> writes: >> >> > 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. >> > >> > Changes in v2: >> > - Fix some warnings / errors reported by kernel test robot >> > - rename some struct and functions with a more accurate name >> > - update the dt bindings: rename type attribute to aggregation-function >> > - factorize a little bit the aggregation functions >> > Changes in v3: >> > - Aggregate thermal zone instead of thermal sensors >> > - Use try_get_module / put_module to prevent thermal providers to be removed >> > - Update the bindings, to be more accurate >> > Changes in v4: >> > - Fix two warnings reported by kernel test robot >> >> Any more feedback on this series? > > Hopefully, I'll get to it this week and I'll let you know. Thanks Rafael, Kevin
On Mon, Jan 24, 2022 at 10:23 PM Kevin Hilman <khilman@baylibre.com> wrote: > > "Rafael J. Wysocki" <rafael@kernel.org> writes: > > > On Mon, Jan 24, 2022 at 5:58 PM Kevin Hilman <khilman@baylibre.com> wrote: > >> > >> Daniel, Amit, > >> > >> Alexandre Bailon <abailon@baylibre.com> writes: > >> > >> > 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. > >> > > >> > Changes in v2: > >> > - Fix some warnings / errors reported by kernel test robot > >> > - rename some struct and functions with a more accurate name > >> > - update the dt bindings: rename type attribute to aggregation-function > >> > - factorize a little bit the aggregation functions > >> > Changes in v3: > >> > - Aggregate thermal zone instead of thermal sensors > >> > - Use try_get_module / put_module to prevent thermal providers to be removed > >> > - Update the bindings, to be more accurate > >> > Changes in v4: > >> > - Fix two warnings reported by kernel test robot > >> > >> Any more feedback on this series? > > > > Hopefully, I'll get to it this week and I'll let you know. > > Thanks Rafael, Done, please see https://lore.kernel.org/linux-pm/CAJZ5v0jejFG76OfewYg3kmKv4xwLdRBpC+zRpJ9Jom+tqo7qyg@mail.gmail.com/
"Rafael J. Wysocki" <rafael@kernel.org> writes: > On Mon, Jan 24, 2022 at 10:23 PM Kevin Hilman <khilman@baylibre.com> wrote: >> >> "Rafael J. Wysocki" <rafael@kernel.org> writes: >> >> > On Mon, Jan 24, 2022 at 5:58 PM Kevin Hilman <khilman@baylibre.com> wrote: >> >> >> >> Daniel, Amit, >> >> >> >> Alexandre Bailon <abailon@baylibre.com> writes: >> >> >> >> > 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. >> >> > >> >> > Changes in v2: >> >> > - Fix some warnings / errors reported by kernel test robot >> >> > - rename some struct and functions with a more accurate name >> >> > - update the dt bindings: rename type attribute to aggregation-function >> >> > - factorize a little bit the aggregation functions >> >> > Changes in v3: >> >> > - Aggregate thermal zone instead of thermal sensors >> >> > - Use try_get_module / put_module to prevent thermal providers to be removed >> >> > - Update the bindings, to be more accurate >> >> > Changes in v4: >> >> > - Fix two warnings reported by kernel test robot >> >> >> >> Any more feedback on this series? >> > >> > Hopefully, I'll get to it this week and I'll let you know. >> >> Thanks Rafael, > > Done, please see > https://lore.kernel.org/linux-pm/CAJZ5v0jejFG76OfewYg3kmKv4xwLdRBpC+zRpJ9Jom+tqo7qyg@mail.gmail.com/ Thank you.