diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2024-08-26 18:30:42 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2024-09-05 12:33:15 +0200 |
| commit | 49029b507e3ad9de35faca497c7bde0e83fcc036 (patch) | |
| tree | e89134cf3cda8ebcea378d4cffc8f8ae5e676ac4 /drivers/thermal | |
| parent | thermal: gov_bang_bang: Adjust states of all uninitialized instances (diff) | |
| download | linux-49029b507e3ad9de35faca497c7bde0e83fcc036.tar.xz linux-49029b507e3ad9de35faca497c7bde0e83fcc036.zip | |
thermal: core: Drop redundant lockdep_assert_held()
Along the lines of commit 24aad192c671 ("thermal: core: Drop
redundant checks from thermal_bind_cdev_to_trip()") notice that
thermal_unbind_cdev_from_trip() is only called by
thermal_zone_cdev_unbind() under the thermal zone lock, so it
need not use lockdep_assert_held() for that lock.
No functional impact.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Link: https://patch.msgid.link/3341369.44csPzL39Z@rjwysocki.net
Diffstat (limited to 'drivers/thermal')
| -rw-r--r-- | drivers/thermal/thermal_core.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index 4187f207bce9..9215a7bc92c7 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal/thermal_core.c @@ -891,8 +891,6 @@ static void thermal_unbind_cdev_from_trip(struct thermal_zone_device *tz, { struct thermal_instance *pos, *next; - lockdep_assert_held(&tz->lock); - mutex_lock(&cdev->lock); list_for_each_entry_safe(pos, next, &tz->thermal_instances, tz_node) { if (pos->trip == trip && pos->cdev == cdev) { |