Commit afbeb99e authored by Daniel Lezcano's avatar Daniel Lezcano
Browse files

thermal/drivers/tegra: Remove get_trend function 



The get_trend function does already what the generic framework does.

Remove it.

Signed-off-by: default avatarDaniel Lezcano <daniel.lezcano@linaro.org>
Tested-by: default avatarDmitry Osipenko <dmitry.osipenko@collabora.com>
Link: https://lore.kernel.org/r/20220616202537.303655-2-daniel.lezcano@linaro.org


Signed-off-by: default avatarDaniel Lezcano <daniel.lezcano@linaro.org>
parent 198fa452

drivers/thermal/tegra/soctherm.c

+0 −32
Original line number Diff line number Diff line
@@ -633,37 +633,6 @@ static int tegra_thermctl_set_trip_temp(void *data, int trip, int temp) 
	return 0;

}



static int tegra_thermctl_get_trend(void *data, int trip,

				    enum thermal_trend *trend)

{

	struct tegra_thermctl_zone *zone = data;

	struct thermal_zone_device *tz = zone->tz;

	int trip_temp, temp, last_temp, ret;



	if (!tz)

		return -EINVAL;



	ret = tz->ops->get_trip_temp(zone->tz, trip, &trip_temp);

	if (ret)

		return ret;



	temp = READ_ONCE(tz->temperature);

	last_temp = READ_ONCE(tz->last_temperature);



	if (temp > trip_temp) {

		if (temp >= last_temp)

			*trend = THERMAL_TREND_RAISING;

		else

			*trend = THERMAL_TREND_STABLE;

	} else if (temp < trip_temp) {

		*trend = THERMAL_TREND_DROPPING;

	} else {

		*trend = THERMAL_TREND_STABLE;

	}



	return 0;

}



static void thermal_irq_enable(struct tegra_thermctl_zone *zn)

{

	u32 r;
@@ -716,7 +685,6 @@ static int tegra_thermctl_set_trips(void *data, int lo, int hi) 
static const struct thermal_zone_of_device_ops tegra_of_thermal_ops = {

	.get_temp = tegra_thermctl_get_temp,

	.set_trip_temp = tegra_thermctl_set_trip_temp,

	.get_trend = tegra_thermctl_get_trend,

	.set_trips = tegra_thermctl_set_trips,

};