[PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor

Alexandre Bailon posted 2 patches 4 years, 5 months ago
.../thermal/virtual,thermal-sensor.yaml       |  86 +++++++
drivers/thermal/Kconfig                       |   8 +
drivers/thermal/Makefile                      |   1 +
drivers/thermal/virtual_thermal_sensor.c      | 218 ++++++++++++++++++
include/dt-bindings/thermal/virtual-sensor.h  |  15 ++
5 files changed, 328 insertions(+)
create mode 100644 Documentation/devicetree/bindings/thermal/virtual,thermal-sensor.yaml
create mode 100644 drivers/thermal/virtual_thermal_sensor.c
create mode 100644 include/dt-bindings/thermal/virtual-sensor.h
[PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor
Posted by Alexandre Bailon 4 years, 5 months ago
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

Alexandre Bailon (2):
  dt-bindings: Add DT bindings for the DT-based virtual sensor driver
  thermal: add a virtual sensor to aggregate temperatures

Alexandre Bailon (2):
  dt-bindings: Add DT bindings for the DT-based virtual sensor driver
  thermal: add a virtual sensor to aggregate temperatures

 .../thermal/virtual,thermal-sensor.yaml       |  86 +++++++
 drivers/thermal/Kconfig                       |   8 +
 drivers/thermal/Makefile                      |   1 +
 drivers/thermal/virtual_thermal_sensor.c      | 218 ++++++++++++++++++
 include/dt-bindings/thermal/virtual-sensor.h  |  15 ++
 5 files changed, 328 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/thermal/virtual,thermal-sensor.yaml
 create mode 100644 drivers/thermal/virtual_thermal_sensor.c
 create mode 100644 include/dt-bindings/thermal/virtual-sensor.h

-- 
2.25.1

Re: [PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor
Posted by Kevin Hilman 4 years, 5 months ago
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
Re: [PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor
Posted by Rafael J. Wysocki 4 years, 5 months ago
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.
Re: [PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor
Posted by Kevin Hilman 4 years, 5 months ago
"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
Re: [PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor
Posted by Rafael J. Wysocki 4 years, 5 months ago
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/
Re: [PATCH v4 RESEND 0/2] Add a generic virtual thermal sensor
Posted by Kevin Hilman 4 years, 4 months ago
"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.