thermal: gov_power_allocator: Use shorter paths to access data when possible
authorLukasz Luba <lukasz.luba@arm.com>
Wed, 25 Oct 2023 19:22:23 +0000 (20:22 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 28 Nov 2023 15:02:14 +0000 (16:02 +0100)
The 'cdev' pointer in allow_maximum_power() is valid, so there is no
need to use 'instance->cdev' instead of it.

This change is not expected to alter the general functionality.

Signed-off-by: Lukasz Luba <lukasz.luba@arm.com>
[ rjw: Subject and changelog edits ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/thermal/gov_power_allocator.c

index 79621b42ead38ac4adb8a899e71f6ea735dffe16..0f7f8278eacc581cac5e58dde41e1d2acd50dee1 100644 (file)
@@ -560,7 +560,7 @@ static void allow_maximum_power(struct thermal_zone_device *tz, bool update)
                        continue;
 
                instance->target = 0;
-               mutex_lock(&instance->cdev->lock);
+               mutex_lock(&cdev->lock);
                /*
                 * Call for updating the cooling devices local stats and avoid
                 * periods of dozen of seconds when those have not been
@@ -569,9 +569,9 @@ static void allow_maximum_power(struct thermal_zone_device *tz, bool update)
                cdev->ops->get_requested_power(cdev, &req_power);
 
                if (update)
-                       __thermal_cdev_update(instance->cdev);
+                       __thermal_cdev_update(cdev);
 
-               mutex_unlock(&instance->cdev->lock);
+               mutex_unlock(&cdev->lock);
        }
 }