[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3267346.aV6nBDHxoP@rjwysocki.net>
Date: Mon, 12 Aug 2024 16:27:30 +0200
From: "Rafael J. Wysocki" <rjw@...ysocki.net>
To: Linux PM <linux-pm@...r.kernel.org>
Cc: LKML <linux-kernel@...r.kernel.org>,
Daniel Lezcano <daniel.lezcano@...aro.org>,
Lukasz Luba <lukasz.luba@....com>, Zhang Rui <rui.zhang@...el.com>
Subject: [PATCH v2 16/17] thermal: code: Clean up trip bind/unbind functions
From: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
Make thermal_bind_cdev_to_trip() take a struct cooling_spec pointer
to reduce the number of its arguments, change the return type of
thermal_unbind_cdev_from_trip() to void and rearrange the code in
thermal_zone_cdev_binding() to reduce the indentation level.
No intentional functional impact.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
---
v1 -> v2: No changes
---
drivers/thermal/thermal_core.c | 54 +++++++++++++++--------------------------
1 file changed, 21 insertions(+), 33 deletions(-)
Index: linux-pm/drivers/thermal/thermal_core.c
===================================================================
--- linux-pm.orig/drivers/thermal/thermal_core.c
+++ linux-pm/drivers/thermal/thermal_core.c
@@ -761,15 +761,7 @@ struct thermal_zone_device *thermal_zone
* @tz: pointer to struct thermal_zone_device
* @trip: trip point the cooling devices is associated with in this zone.
* @cdev: pointer to struct thermal_cooling_device
- * @upper: the Maximum cooling state for this trip point.
- * THERMAL_NO_LIMIT means no upper limit,
- * and the cooling device can be in max_state.
- * @lower: the Minimum cooling state can be used for this trip point.
- * THERMAL_NO_LIMIT means no lower limit,
- * and the cooling device can be in cooling state 0.
- * @weight: The weight of the cooling device to be bound to the
- * thermal zone. Use THERMAL_WEIGHT_DEFAULT for the
- * default value
+ * @c: cooling specification for @trip and @cdev
*
* This interface function bind a thermal cooling device to the certain trip
* point of a thermal zone device.
@@ -780,8 +772,7 @@ struct thermal_zone_device *thermal_zone
static int thermal_bind_cdev_to_trip(struct thermal_zone_device *tz,
struct thermal_trip *trip,
struct thermal_cooling_device *cdev,
- unsigned long upper, unsigned long lower,
- unsigned int weight)
+ struct cooling_spec *c)
{
struct thermal_trip_desc *td = trip_to_trip_desc(trip);
struct thermal_instance *dev, *instance;
@@ -795,17 +786,17 @@ static int thermal_bind_cdev_to_trip(str
return -EINVAL;
/* lower default 0, upper default max_state */
- if (lower == THERMAL_NO_LIMIT)
- lower = 0;
+ if (c->lower == THERMAL_NO_LIMIT)
+ c->lower = 0;
- if (upper == THERMAL_NO_LIMIT) {
- upper = cdev->max_state;
+ if (c->upper == THERMAL_NO_LIMIT) {
+ c->upper = cdev->max_state;
upper_no_limit = true;
} else {
upper_no_limit = false;
}
- if (lower > upper || upper > cdev->max_state)
+ if (c->lower > c->upper || c->upper > cdev->max_state)
return -EINVAL;
dev = kzalloc(sizeof(*dev), GFP_KERNEL);
@@ -815,11 +806,11 @@ static int thermal_bind_cdev_to_trip(str
dev->tz = tz;
dev->cdev = cdev;
dev->trip = trip;
- dev->upper = upper;
+ dev->upper = c->upper;
dev->upper_no_limit = upper_no_limit;
- dev->lower = lower;
+ dev->lower = c->lower;
dev->target = THERMAL_NO_TARGET;
- dev->weight = weight;
+ dev->weight = c->weight;
result = ida_alloc(&tz->ida, GFP_KERNEL);
if (result < 0)
@@ -893,12 +884,10 @@ free_mem:
* This interface function unbind a thermal cooling device from the certain
* trip point of a thermal zone device.
* This function is usually called in the thermal zone device .unbind callback.
- *
- * Return: 0 on success, the proper error value otherwise.
*/
-static int thermal_unbind_cdev_from_trip(struct thermal_zone_device *tz,
- struct thermal_trip *trip,
- struct thermal_cooling_device *cdev)
+static void thermal_unbind_cdev_from_trip(struct thermal_zone_device *tz,
+ struct thermal_trip *trip,
+ struct thermal_cooling_device *cdev)
{
struct thermal_trip_desc *td = trip_to_trip_desc(trip);
struct thermal_instance *pos, *next;
@@ -921,7 +910,7 @@ static int thermal_unbind_cdev_from_trip
mutex_unlock(&cdev->lock);
- return -ENODEV;
+ return;
free:
device_remove_file(&tz->device, &pos->weight_attr);
@@ -929,7 +918,6 @@ free:
sysfs_remove_link(&tz->device.kobj, pos->name);
ida_free(&tz->ida, pos->id);
kfree(pos);
- return 0;
}
static void thermal_release(struct device *dev)
@@ -968,7 +956,6 @@ static void thermal_zone_cdev_binding(st
struct thermal_cooling_device *cdev)
{
struct thermal_trip_desc *td;
- int ret;
if (!tz->ops.should_bind)
return;
@@ -982,13 +969,14 @@ static void thermal_zone_cdev_binding(st
.lower = THERMAL_NO_LIMIT,
.weight = THERMAL_WEIGHT_DEFAULT
};
+ int ret;
- if (tz->ops.should_bind(tz, trip, cdev, &c)) {
- ret = thermal_bind_cdev_to_trip(tz, trip, cdev, c.upper,
- c.lower, c.weight);
- if (ret)
- print_bind_err_msg(tz, trip, cdev, ret);
- }
+ if (!tz->ops.should_bind(tz, trip, cdev, &c))
+ continue;
+
+ ret = thermal_bind_cdev_to_trip(tz, trip, cdev, &c);
+ if (ret)
+ print_bind_err_msg(tz, trip, cdev, ret);
}
mutex_unlock(&tz->lock);
Powered by blists - more mailing lists