All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Lezcano <daniel.lezcano@linexp.org>
To: daniel.lezcano@linaro.org, rafael@kernel.org
Cc: rui.zhang@intel.com, linux-pm@vger.kernel.org,
	linux-kernel@vger.kernel.org, khilman@baylibre.com,
	abailon@baylibre.com, lukasz.luba@arm.com, broonie@kernel.org,
	damien.lemoal@opensource.wdc.com, heiko@sntech.de,
	hayashi.kunihiko@socionext.com, mhiramat@kernel.org,
	talel@amazon.com, thierry.reding@gmail.com, digetx@gmail.com,
	jonathanh@nvidia.com, anarsoul@gmail.com, tiny.windzz@gmail.com,
	baolin.wang7@gmail.com, f.fainelli@gmail.com,
	bjorn.andersson@linaro.org, mcoquelin.stm32@gmail.com,
	glaroque@baylibre.com, miquel.raynal@bootlin.com,
	shawnguo@kernel.org, niklas.soderlund@ragnatech.se,
	matthias.bgg@gmail.com, j-keerthy@ti.com,
	Guenter Roeck <linux@roeck-us.net>,
	Jean Delvare <jdelvare@suse.com>,
	linux-hwmon@vger.kernel.org (open list:HARDWARE MONITORING)
Subject: [PATCH v5 27/33] hwmon/drivers/core: Switch to new of thermal API
Date: Fri,  5 Aug 2022 00:43:43 +0200	[thread overview]
Message-ID: <20220804224349.1926752-28-daniel.lezcano@linexp.org> (raw)
In-Reply-To: <20220804224349.1926752-1-daniel.lezcano@linexp.org>

The thermal OF code has a new API allowing to migrate the OF
initialization to a simpler approach. The ops are no longer device
tree specific and are the generic ones provided by the core code.

Convert the ops to the thermal_zone_device_ops format and use the new
API to register the thermal zone with these generic ops.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linexp.org>
Acked-by: Guenter Roeck <linux@roeck-us.net>
---
 drivers/hwmon/hwmon.c      | 14 +++++++-------
 drivers/hwmon/scpi-hwmon.c | 14 +++++++-------
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/drivers/hwmon/hwmon.c b/drivers/hwmon/hwmon.c
index 2e2cd79d89eb..4218750d5a66 100644
--- a/drivers/hwmon/hwmon.c
+++ b/drivers/hwmon/hwmon.c
@@ -151,9 +151,9 @@ static DEFINE_IDA(hwmon_ida);
  * between hwmon and thermal_sys modules.
  */
 #ifdef CONFIG_THERMAL_OF
-static int hwmon_thermal_get_temp(void *data, int *temp)
+static int hwmon_thermal_get_temp(struct thermal_zone_device *tz, int *temp)
 {
-	struct hwmon_thermal_data *tdata = data;
+	struct hwmon_thermal_data *tdata = tz->devdata;
 	struct hwmon_device *hwdev = to_hwmon_device(tdata->dev);
 	int ret;
 	long t;
@@ -168,9 +168,9 @@ static int hwmon_thermal_get_temp(void *data, int *temp)
 	return 0;
 }
 
-static int hwmon_thermal_set_trips(void *data, int low, int high)
+static int hwmon_thermal_set_trips(struct thermal_zone_device *tz, int low, int high)
 {
-	struct hwmon_thermal_data *tdata = data;
+	struct hwmon_thermal_data *tdata = tz->devdata;
 	struct hwmon_device *hwdev = to_hwmon_device(tdata->dev);
 	const struct hwmon_chip_info *chip = hwdev->chip;
 	const struct hwmon_channel_info **info = chip->info;
@@ -203,7 +203,7 @@ static int hwmon_thermal_set_trips(void *data, int low, int high)
 	return 0;
 }
 
-static const struct thermal_zone_of_device_ops hwmon_thermal_ops = {
+static const struct thermal_zone_device_ops hwmon_thermal_ops = {
 	.get_temp = hwmon_thermal_get_temp,
 	.set_trips = hwmon_thermal_set_trips,
 };
@@ -227,8 +227,8 @@ static int hwmon_thermal_add_sensor(struct device *dev, int index)
 	tdata->dev = dev;
 	tdata->index = index;
 
-	tzd = devm_thermal_zone_of_sensor_register(dev, index, tdata,
-						   &hwmon_thermal_ops);
+	tzd = devm_thermal_of_zone_register(dev, index, tdata,
+					    &hwmon_thermal_ops);
 	if (IS_ERR(tzd)) {
 		if (PTR_ERR(tzd) != -ENODEV)
 			return PTR_ERR(tzd);
diff --git a/drivers/hwmon/scpi-hwmon.c b/drivers/hwmon/scpi-hwmon.c
index 5187c6dd5a4f..4d75385f7d5e 100644
--- a/drivers/hwmon/scpi-hwmon.c
+++ b/drivers/hwmon/scpi-hwmon.c
@@ -62,9 +62,9 @@ static void scpi_scale_reading(u64 *value, struct sensor_data *sensor)
 	}
 }
 
-static int scpi_read_temp(void *dev, int *temp)
+static int scpi_read_temp(struct thermal_zone_device *tz, int *temp)
 {
-	struct scpi_thermal_zone *zone = dev;
+	struct scpi_thermal_zone *zone = tz->devdata;
 	struct scpi_sensors *scpi_sensors = zone->scpi_sensors;
 	struct scpi_ops *scpi_ops = scpi_sensors->scpi_ops;
 	struct sensor_data *sensor = &scpi_sensors->data[zone->sensor_id];
@@ -121,7 +121,7 @@ scpi_show_label(struct device *dev, struct device_attribute *attr, char *buf)
 	return sprintf(buf, "%s\n", sensor->info.name);
 }
 
-static const struct thermal_zone_of_device_ops scpi_sensor_ops = {
+static const struct thermal_zone_device_ops scpi_sensor_ops = {
 	.get_temp = scpi_read_temp,
 };
 
@@ -275,10 +275,10 @@ static int scpi_hwmon_probe(struct platform_device *pdev)
 
 		zone->sensor_id = i;
 		zone->scpi_sensors = scpi_sensors;
-		z = devm_thermal_zone_of_sensor_register(dev,
-							 sensor->info.sensor_id,
-							 zone,
-							 &scpi_sensor_ops);
+		z = devm_thermal_of_zone_register(dev,
+						  sensor->info.sensor_id,
+						  zone,
+						  &scpi_sensor_ops);
 		/*
 		 * The call to thermal_zone_of_sensor_register returns
 		 * an error for sensors that are not associated with
-- 
2.25.1


  parent reply	other threads:[~2022-08-04 22:55 UTC|newest]

Thread overview: 116+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-04 22:43 [PATCH v5 00/33] New thermal OF code Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 01/33] thermal/of: Rework the thermal device tree initialization Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-10-18 10:29   ` [PATCH v5 01/33] " Ido Schimmel
2022-10-18 10:33     ` Ido Schimmel
2022-10-19 19:53     ` Daniel Lezcano
2022-10-20 10:39       ` Ido Schimmel
2022-08-04 22:43 ` [PATCH v5 02/33] thermal/of: Make new code and old code co-exist Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 03/33] thermal/drivers/rockchip: Switch to new of API Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 04/33] thermal/drivers/uniphier: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-15 14:18   ` Kunihiko Hayashi
2022-08-15 14:18     ` Kunihiko Hayashi
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 05/33] thermal/drivers/generic-adc: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 06/33] thermal/drivers/mmio: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 07/33] thermal/drivers/tegra: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-09-06 13:11   ` [PATCH v5 07/33] " Dmitry Osipenko
2022-08-04 22:43 ` [PATCH v5 08/33] thermal/drivers/sun8i: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 09/33] thermal/drivers/sprd: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 10/33] thermal/drivers/broadcom: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 11/33] thermal/drivers/qcom: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 12/33] thermal/drivers/st: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 13/33] thermal/drivers/amlogic: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-08  8:07   ` Neil Armstrong
2022-08-08  8:07     ` Neil Armstrong
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 14/33] thermal/drivers/armada: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 15/33] thermal/drivers/db8500: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 16/33] thermal/drivers/imx: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 17/33] thermal/drivers/rcar: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 18/33] thermal/drivers/rzg2l: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 19/33] thermal/drivers/qoriq: " Daniel Lezcano
2022-08-23 12:43   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 20/33] thermal/drivers/mtk: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 21/33] thermal/drivers/banggap: " Daniel Lezcano
2022-08-04 23:30   ` Bryan Brattlof
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 22/33] thermal/drivers/maxim: " Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 23/33] thermal/drivers/hisilicon: " Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 24/33] thermal/drivers/ti-soc: " Daniel Lezcano
2022-08-05  1:19   ` J, KEERTHY
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 25/33] ata/drivers/ahci_imx: Switch to new of thermal API Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 26/33] hwmon/drivers/pm_bus: " Daniel Lezcano
2022-08-08 10:29   ` Guenter Roeck
2022-08-10 10:23     ` Daniel Lezcano
2022-08-10 12:41       ` Guenter Roeck
2022-08-23 12:42   ` [thermal: thermal/next] hwmon: pm_bus: core: " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` Daniel Lezcano [this message]
2022-08-08 10:27   ` [PATCH v5 27/33] hwmon/drivers/core: " Guenter Roeck
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 28/33] iio/drivers/sun4i_gpadc: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-14 10:11   ` Jernej Škrabec
2022-08-14 10:11     ` Jernej Škrabec
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 29/33] Input: sun4i-ts - switch " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-14 10:12   ` Jernej Škrabec
2022-08-14 10:12     ` Jernej Škrabec
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 30/33] regulator/drivers/max8976: Switch " Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 31/33] thermal/drivers/samsung: " Daniel Lezcano
2022-08-04 22:43   ` Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 32/33] thermal/core: Move set_trip_temp ops to the sysfs code Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-04 22:43 ` [PATCH v5 33/33] thermal/of: Remove old OF code Daniel Lezcano
2022-08-23 12:42   ` [thermal: thermal/next] " thermal-bot for Daniel Lezcano
2022-08-08  9:42 ` [PATCH v5 00/33] New thermal " Michael Walle
2022-08-08 10:21   ` Daniel Lezcano
2022-08-08 10:55     ` Michael Walle
2022-08-08 11:06       ` Daniel Lezcano
2022-08-08 10:26   ` Guenter Roeck
2022-08-09  8:53     ` Daniel Lezcano
2022-08-09 14:32       ` Guenter Roeck
2022-08-09 15:37         ` Daniel Lezcano
2022-08-10  8:34     ` Daniel Lezcano
2022-08-10  9:56       ` Guenter Roeck
2022-08-08 13:09   ` Daniel Lezcano
2022-08-08 13:24     ` Michael Walle
2022-08-08 13:31       ` Daniel Lezcano
2022-08-09  8:23   ` Daniel Lezcano
2022-08-10  8:01     ` Michael Walle
2022-08-10  8:26       ` Daniel Lezcano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220804224349.1926752-28-daniel.lezcano@linexp.org \
    --to=daniel.lezcano@linexp.org \
    --cc=abailon@baylibre.com \
    --cc=anarsoul@gmail.com \
    --cc=baolin.wang7@gmail.com \
    --cc=bjorn.andersson@linaro.org \
    --cc=broonie@kernel.org \
    --cc=damien.lemoal@opensource.wdc.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=digetx@gmail.com \
    --cc=f.fainelli@gmail.com \
    --cc=glaroque@baylibre.com \
    --cc=hayashi.kunihiko@socionext.com \
    --cc=heiko@sntech.de \
    --cc=j-keerthy@ti.com \
    --cc=jdelvare@suse.com \
    --cc=jonathanh@nvidia.com \
    --cc=khilman@baylibre.com \
    --cc=linux-hwmon@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=lukasz.luba@arm.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=mhiramat@kernel.org \
    --cc=miquel.raynal@bootlin.com \
    --cc=niklas.soderlund@ragnatech.se \
    --cc=rafael@kernel.org \
    --cc=rui.zhang@intel.com \
    --cc=shawnguo@kernel.org \
    --cc=talel@amazon.com \
    --cc=thierry.reding@gmail.com \
    --cc=tiny.windzz@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.