thermal/core: Add dedicated release callback for cooling devices

The thermal class release callback currently handles both thermal
zones and cooling devices by checking the device name prefix.

Move the cooling device cleanup to a dedicated struct device release
callback.  This avoids relying on device names to select the release
path and keeps the cooling device lifetime handling local to the
cooling device object.

Signed-off-by: Daniel Lezcano <daniel.lezcano@oss.qualcomm.com>
Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
Link: https://patch.msgid.link/20260508180511.1306659-2-daniel.lezcano@oss.qualcomm.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
index 427a76223c6fedd53570466108128fc16ee77314..207b86f54c4d5f9ddcdc3d8b686899a657eb7069 100644 (file)
--- a/drivers/thermal/thermal_core.c
+++ b/drivers/thermal/thermal_core.c
@@ -892,7 +892,6 @@ unbind:
 static void thermal_release(struct device *dev)
 {
        struct thermal_zone_device *tz;
-       struct thermal_cooling_device *cdev;
 
        if (!strncmp(dev_name(dev), "thermal_zone",
                     sizeof("thermal_zone") - 1)) {
@@ -902,13 +901,6 @@ static void thermal_release(struct device *dev)
                ida_destroy(&tz->ida);
                mutex_destroy(&tz->lock);
                complete(&tz->removal);
-       } else if (!strncmp(dev_name(dev), "cooling_device",
-                           sizeof("cooling_device") - 1)) {
-               cdev = to_cooling_device(dev);
-               thermal_cooling_device_destroy_sysfs(cdev);
-               kfree_const(cdev->type);
-               ida_free(&thermal_cdev_ida, cdev->id);
-               kfree(cdev);
        }
 }
 
@@ -980,6 +972,16 @@ static void thermal_cooling_device_init_complete(struct thermal_cooling_device *
                thermal_zone_cdev_bind(tz, cdev);
 }
 
+static void thermal_cdev_release(struct device *dev)
+{
+       struct thermal_cooling_device *cdev = to_cooling_device(dev);
+
+       thermal_cooling_device_destroy_sysfs(cdev);
+       kfree_const(cdev->type);
+       ida_free(&thermal_cdev_ida, cdev->id);
+       kfree(cdev);
+}
+
 /**
  * __thermal_cooling_device_register() - register a new thermal cooling device
  * @np:                a pointer to a device tree node.
@@ -1033,6 +1035,7 @@ __thermal_cooling_device_register(struct device_node *np,
        cdev->ops = ops;
        cdev->updated = false;
        cdev->device.class = &thermal_class;
+       cdev->device.release = thermal_cdev_release;
        cdev->devdata = devdata;
 
        ret = cdev->ops->get_max_state(cdev, &cdev->max_state);