diff mbox series

drivers/thermal/ti-soc-thermal: Add hwmon support

Message ID 20220216172206.7448-1-romain.naour@smile.fr (mailing list archive)
State New, archived
Headers show
Series drivers/thermal/ti-soc-thermal: Add hwmon support | expand

Commit Message

Romain Naour Feb. 16, 2022, 5:22 p.m. UTC
Expose ti-soc-thermal thermal sensors as HWMON devices.

  # sensors
  cpu_thermal-virtual-0
  Adapter: Virtual device
  temp1:        +54.2 C  (crit = +105.0 C)

  dspeve_thermal-virtual-0
  Adapter: Virtual device
  temp1:        +51.4 C  (crit = +105.0 C)

  gpu_thermal-virtual-0
  Adapter: Virtual device
  temp1:        +54.2 C  (crit = +105.0 C)

  iva_thermal-virtual-0
  Adapter: Virtual device
  temp1:        +54.6 C  (crit = +105.0 C)

  core_thermal-virtual-0
  Adapter: Virtual device
  temp1:        +52.6 C  (crit = +105.0 C)

Similar to imx_sc_thermal d2bc4dd91da6095a769fdc9bc519d3be7ad5f97a.

No need to take care of thermal_remove_hwmon_sysfs() since
devm_thermal_add_hwmon_sysfs() (a wrapper around devres) is
used. See c7fc403e40b0ea18976a59e968c23439a80809e8.

Signed-off-by: Romain Naour <romain.naour@smile.fr>
---
 drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Tony Lindgren Feb. 18, 2022, 8:28 a.m. UTC | #1
* Romain Naour <romain.naour@smile.fr> [220216 17:21]:
> Expose ti-soc-thermal thermal sensors as HWMON devices.

Can you please send this to the ti-soc-thermal maintainer(s)
and related subsystem mailing list with linux-omap in Cc?
See the MAINTAINERS file for info and scripts/get-maintainer.pl
script.

Then the subsystem maintainer(s) will pick it up, or will
ack it and I can pick it up.

Regards,

Tony
Romain Naour Feb. 18, 2022, 10:50 a.m. UTC | #2
Le 18/02/2022 à 09:28, Tony Lindgren a écrit :
> * Romain Naour <romain.naour@smile.fr> [220216 17:21]:
>> Expose ti-soc-thermal thermal sensors as HWMON devices.
> 
> Can you please send this to the ti-soc-thermal maintainer(s)
> and related subsystem mailing list with linux-omap in Cc?
> See the MAINTAINERS file for info and scripts/get-maintainer.pl
> script.

Ok, I was not sure if adding two mailing list was mandatory.

> 
> Then the subsystem maintainer(s) will pick it up, or will
> ack it and I can pick it up.

Thanks, sorry for the noise.

Best regards,
Romain


> 
> Regards,
> 
> Tony
diff mbox series

Patch

diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
index f84375865c97..0959632b2170 100644
--- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
+++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
@@ -21,6 +21,7 @@ 
 
 #include "ti-thermal.h"
 #include "ti-bandgap.h"
+#include "../thermal_hwmon.h"
 
 /* common data structures */
 struct ti_thermal_data {
@@ -189,6 +190,9 @@  int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int id,
 	ti_bandgap_set_sensor_data(bgp, id, data);
 	ti_bandgap_write_update_interval(bgp, data->sensor_id, interval);
 
+	if (devm_thermal_add_hwmon_sysfs(data->ti_thermal))
+		dev_warn(bgp->dev, "failed to add hwmon sysfs attributes\n");
+
 	return 0;
 }