Revert "CHROMIUM: thermal: of: Add support for hardware-tracked trip points"

This reverts commit 2f8e5324ef44d0842f3db3af5fc4da761440abfb.
Since there are the perfect patches in fromlist to instead of it.

Change-Id: I9f15478d0b0b94805d1bb9539a1cbea42a7af6a1
Signed-off-by: Caesar Wang <wxt@rock-chips.com>

diff --git a/drivers/thermal/of-thermal.c b/drivers/thermal/of-thermal.c
index 8eed2015038f2546bce895ebca73e0f37d09833d..57828841b6e6ac03c1ed5773d9281b07638fbf0f 100644 (file)
--- a/drivers/thermal/of-thermal.c
+++ b/drivers/thermal/of-thermal.c
@@ -60,8 +60,6 @@ struct __thermal_bind_params {
  * @polling_delay: zone polling interval
  * @slope: slope of the temperature adjustment curve
  * @offset: offset of the temperature adjustment curve
- * @prev_low_trip: previous low trip point temperature
- * @prev_high_trip: previous high trip point temperature
  * @ntrips: number of trip points
  * @trips: an array of trip points (0..ntrips - 1)
  * @num_tbps: number of thermal bind params
@@ -76,7 +74,6 @@ struct __thermal_zone {
        int polling_delay;
        int slope;
        int offset;
-       int prev_low_trip, prev_high_trip;
 
        /* trip data */
        int ntrips;
@@ -91,63 +88,17 @@ struct __thermal_zone {
        const struct thermal_zone_of_device_ops *ops;
 };
 
-/***   Automatic trip handling   ***/
-
-static int of_thermal_set_trips(struct thermal_zone_device *tz, int temp)
-{
-       struct __thermal_zone *data = tz->devdata;
-       int low = INT_MIN, high = INT_MAX;
-       int i;
-
-       /* Hardware trip points not supported */
-       if (!data->ops->set_trips)
-               return 0;
-
-       /* No need to change trip points */
-       if (temp > data->prev_low_trip && temp < data->prev_high_trip)
-               return 0;
-
-       for (i = 0; i < data->ntrips; ++i) {
-               struct thermal_trip *trip = data->trips + i;
-               int trip_low = trip->temperature - trip->hysteresis;
-
-               if (trip_low < temp && trip_low > low)
-                       low = trip_low;
-
-               if (trip->temperature > temp && trip->temperature < high)
-                       high = trip->temperature;
-       }
-
-       dev_dbg(&tz->device,
-               "temperature %d, updating trip points to %d, %d\n",
-               temp, low, high);
-
-       data->prev_low_trip = low;
-       data->prev_high_trip = high;
-
-       return data->ops->set_trips(data->sensor_data, low, high);
-}
-
 /***   DT thermal zone device callbacks   ***/
 
 static int of_thermal_get_temp(struct thermal_zone_device *tz,
                               int *temp)
 {
        struct __thermal_zone *data = tz->devdata;
-       int err;
 
        if (!data->ops->get_temp)
                return -EINVAL;
 
-       err = data->ops->get_temp(data->sensor_data, temp);
-       if (err)
-               return err;
-
-       err = of_thermal_set_trips(tz, *temp);
-       if (err)
-               return err;
-
-       return 0;
+       return data->ops->get_temp(data->sensor_data, temp);
 }
 
 /**
@@ -346,22 +297,6 @@ static int of_thermal_set_mode(struct thermal_zone_device *tz,
        return 0;
 }
 
-static int of_thermal_update_trips(struct thermal_zone_device *tz)
-{
-       int temp;
-       int err;
-
-       err = of_thermal_get_temp(tz, &temp);
-       if (err)
-               return err;
-
-       err = of_thermal_set_trips(tz, temp);
-       if (err)
-               return err;
-
-       return 0;
-}
-
 static int of_thermal_get_trip_type(struct thermal_zone_device *tz, int trip,
                                    enum thermal_trip_type *type)
 {
@@ -392,7 +327,6 @@ static int of_thermal_set_trip_temp(struct thermal_zone_device *tz, int trip,
                                    int temp)
 {
        struct __thermal_zone *data = tz->devdata;
-       int err;
 
        if (trip >= data->ntrips || trip < 0)
                return -EDOM;
@@ -408,10 +342,6 @@ static int of_thermal_set_trip_temp(struct thermal_zone_device *tz, int trip,
        /* thermal framework should take care of data->mask & (1 << trip) */
        data->trips[trip].temperature = temp;
 
-       err = of_thermal_update_trips(tz);
-       if (err)
-               return err;
-
        return 0;
 }
 
@@ -432,7 +362,6 @@ static int of_thermal_set_trip_hyst(struct thermal_zone_device *tz, int trip,
                                    int hyst)
 {
        struct __thermal_zone *data = tz->devdata;
-       int err;
 
        if (trip >= data->ntrips || trip < 0)
                return -EDOM;
@@ -440,10 +369,6 @@ static int of_thermal_set_trip_hyst(struct thermal_zone_device *tz, int trip,
        /* thermal framework should take care of data->mask & (1 << trip) */
        data->trips[trip].hysteresis = hyst;
 
-       err = of_thermal_update_trips(tz);
-       if (err)
-               return err;
-
        return 0;
 }
 
@@ -500,8 +425,6 @@ thermal_zone_of_add_sensor(struct device_node *zone,
        tz->ops = ops;
        tz->sensor_data = data;
 
-       of_thermal_update_trips(tzd);
-
        tzd->ops->get_temp = of_thermal_get_temp;
        tzd->ops->get_trend = of_thermal_get_trend;
        tzd->ops->set_emul_temp = of_thermal_set_emul_temp;
@@ -940,9 +863,6 @@ thermal_of_build_thermal_zone(struct device_node *np)
        /* trips */
        child = of_get_child_by_name(np, "trips");
 
-       tz->prev_high_trip = INT_MIN;
-       tz->prev_low_trip = INT_MAX;
-
        /* No trips provided */
        if (!child)
                goto finish;

diff --git a/include/linux/thermal.h b/include/linux/thermal.h
index 9ac210234084f5120551eb5ccb95121d3be94ebf..fcaab873b94f9a3b51f2f46a8d02b0b3b57589db 100644 (file)
--- a/include/linux/thermal.h
+++ b/include/linux/thermal.h
@@ -333,14 +333,12 @@ struct thermal_genl_event {
  * @get_trend: a pointer to a function that reads the sensor temperature trend.
  * @set_emul_temp: a pointer to a function that sets sensor emulated
  *                temperature.
- * @set_trips: a pointer to a function that set low/high trip temperature.
  */
 struct thermal_zone_of_device_ops {
        int (*get_temp)(void *, int *);
        int (*get_trend)(void *, long *);
        int (*set_emul_temp)(void *, int);
        int (*set_trip_temp)(void *, int, int);
-       int (*set_trips)(void *, int, int);
 };
 
 /**