Skip to contentSkip to conversation

thermal: trip: Drop redundant __thermal_zone_get_trip() header

PATCH In review Rafael J. Wysocki 3 participants 3 messages Dec 4, 2023 – Dec 13, 2023 Thermal

lore
Share & tools
lore

Rafael J. Wysocki

author
#0 lore raw

PATCH 1/1 thermal: trip: Drop redundant __thermal_zone_get_trip() header

From: Rafael J. Wysocki <[email protected]>

The __thermal_zone_get_trip() header in drivers/thermal/thermal_core.h is redundant, because there is one already in thermal.h, so drop it.

No functional impact.

Signed-off-by Rafael J. Wysocki <[email protected]>
---
 drivers/thermal/thermal_core.h |    2 --
 1 file changed, 2 deletions(-)
drivers/thermal/thermal_core.h +0 -2
Index: linux-pm/drivers/thermal/thermal_core.h
===================================================================
@@ -120,8 +120,6 @@ void __thermal_zone_device_update(struct
 	for (__trip = __tz->trips; __trip - __tz->trips < __tz->num_trips; __trip++)
 
 void __thermal_zone_set_trips(struct thermal_zone_device *tz);
-int __thermal_zone_get_trip(struct thermal_zone_device *tz, int trip_id,
-			    struct thermal_trip *trip);
 int thermal_zone_trip_id(struct thermal_zone_device *tz,
 			 const struct thermal_trip *trip);
 void thermal_zone_trip_updated(struct thermal_zone_device *tz,
maintainer
#1 lore raw
Quoted 9 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

The __thermal_zone_get_trip() header in drivers/thermal/thermal_core.h is redundant, because there is one already in thermal.h, so drop it.

No functional impact.

Signed-off-by Rafael J. Wysocki <[email protected]>

From the lack of comments I gather that this change is fine with everyone, so I'm going to queue it up for 6.8.

Thanks!

Quoted 21 lines
---
 drivers/thermal/thermal_core.h |    2 --
 1 file changed, 2 deletions(-)

Index: linux-pm/drivers/thermal/thermal_core.h
===================================================================

drivers/thermal/thermal_core.h +0 -2
@@ -120,8 +120,6 @@ void __thermal_zone_device_update(struct
        for (__trip = __tz->trips; __trip - __tz->trips < __tz->num_trips; __trip++)
 
 void __thermal_zone_set_trips(struct thermal_zone_device *tz);
-int __thermal_zone_get_trip(struct thermal_zone_device *tz, int trip_id,
-                           struct thermal_trip *trip);
 int thermal_zone_trip_id(struct thermal_zone_device *tz,
                         const struct thermal_trip *trip);
 void thermal_zone_trip_updated(struct thermal_zone_device *tz,
Acked-by on v1 1/1
#2 lore raw
In reply to Rafael J. Wysocki
Quoted 9 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

The __thermal_zone_get_trip() header in drivers/thermal/thermal_core.h is redundant, because there is one already in thermal.h, so drop it.

No functional impact.

Signed-off-by Rafael J. Wysocki <[email protected]>

---

Acked-by Daniel Lezcano <[email protected]>
Signature · 5 lines
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog