thermal: trip: Drop a redundant check from thermal_zone_set_trip()
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 29 Nov 2023 13:36:07 +0000 (14:36 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 30 Nov 2023 13:38:06 +0000 (14:38 +0100)
After recent changes in the thermal framework, a trip points array is
required for registering a thermal zone that is not tripless, so the
tz->trips pointer in thermal_zone_set_trip() is never NULL and the
check involving it is redundant.  Drop that check.

No functional impact.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
drivers/thermal/thermal_trip.c

index e42456442c68e0f7e9faa1498d5cfd4b845a7925..e3dd583234ddecb390c55aadab956b73d12624d9 100644 (file)
@@ -153,9 +153,6 @@ int thermal_zone_set_trip(struct thermal_zone_device *tz, int trip_id,
        struct thermal_trip t;
        int ret;
 
-       if (!tz->ops->set_trip_temp && !tz->ops->set_trip_hyst && !tz->trips)
-               return -EINVAL;
-
        ret = __thermal_zone_get_trip(tz, trip_id, &t);
        if (ret)
                return ret;