thermal: exynos: Modify exynos thermal code to use device tree for cpu cooling config...
authorLukasz Majewski <l.majewski@samsung.com>
Fri, 23 Jan 2015 12:10:00 +0000 (13:10 +0100)
committerEduardo Valentin <edubezval@gmail.com>
Sat, 24 Jan 2015 21:32:03 +0000 (17:32 -0400)
Up till now exynos_tmu_data.c was used for storing CPU cooling configuration
data. Now the Exynos thermal core code uses device tree to get this data.
For this purpose generic thermal code for configuring CPU cooling was
used.

Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
drivers/thermal/samsung/exynos_thermal_common.c
drivers/thermal/samsung/exynos_tmu.c
drivers/thermal/samsung/exynos_tmu.h
drivers/thermal/samsung/exynos_tmu_data.c

index 6dc3815cc73f514c71d16fab40552609fa12ba6d..00aa68862a522e2ed19de5cb384c3f965773f775 100644 (file)
@@ -133,47 +133,62 @@ static int exynos_get_crit_temp(struct thermal_zone_device *thermal,
 static int exynos_bind(struct thermal_zone_device *thermal,
                        struct thermal_cooling_device *cdev)
 {
-       int ret = 0, i, tab_size, level;
-       struct freq_clip_table *tab_ptr, *clip_data;
        struct exynos_thermal_zone *th_zone = thermal->devdata;
        struct thermal_sensor_conf *data = th_zone->sensor_conf;
+       struct device_node *child, *gchild, *np;
+       struct of_phandle_args cooling_spec;
+       unsigned long max, state = 0;
+       int ret = 0, i = 0;
 
-       tab_ptr = (struct freq_clip_table *)data->cooling_data.freq_data;
-       tab_size = data->cooling_data.freq_clip_count;
-
-       if (tab_ptr == NULL || tab_size == 0)
+       /*
+        * Below code is necessary to skip binding when cpufreq's
+        * frequency table is not yet initialized.
+        */
+       cdev->ops->get_max_state(cdev, &state);
+       if (!state && !th_zone->cool_dev_size) {
+               th_zone->cool_dev_size = 1;
+               th_zone->cool_dev[0] = cdev;
+               th_zone->bind = false;
                return 0;
+       }
 
-       /* find the cooling device registered*/
-       for (i = 0; i < th_zone->cool_dev_size; i++)
-               if (cdev == th_zone->cool_dev[i])
-                       break;
+       np = of_find_node_by_path("/thermal-zones/cpu-thermal");
+       if (!np) {
+               pr_err("failed to find thmerla-zones/cpu-thermal node\n");
+               return -ENOENT;
+       }
 
-       /* No matching cooling device */
-       if (i == th_zone->cool_dev_size)
-               return 0;
+       child = of_get_child_by_name(np, "cooling-maps");
 
-       /* Bind the thermal zone to the cpufreq cooling device */
-       for (i = 0; i < tab_size; i++) {
-               clip_data = (struct freq_clip_table *)&(tab_ptr[i]);
-               level = cpufreq_cooling_get_level(0, clip_data->freq_clip_max);
-               if (level == THERMAL_CSTATE_INVALID)
-                       return 0;
-               switch (GET_ZONE(i)) {
-               case MONITOR_ZONE:
-               case WARN_ZONE:
-                       if (thermal_zone_bind_cooling_device(thermal, i, cdev,
-                                                               level, 0)) {
-                               dev_err(data->dev,
-                                       "error unbinding cdev inst=%d\n", i);
-                               ret = -EINVAL;
-                       }
-                       th_zone->bind = true;
-                       break;
-               default:
+       for_each_child_of_node(child, gchild) {
+               ret = of_parse_phandle_with_args(gchild, "cooling-device",
+                                                "#cooling-cells",
+                                                0, &cooling_spec);
+               if (ret < 0) {
+                       pr_err("missing cooling_device property\n");
+                       goto end;
+               }
+
+               if (cooling_spec.args_count < 2) {
                        ret = -EINVAL;
+                       goto end;
                }
+
+               max = cooling_spec.args[0];
+               if (thermal_zone_bind_cooling_device(thermal, i, cdev,
+                                                    max, 0)) {
+                       dev_err(data->dev,
+                               "thermal error unbinding cdev inst=%d\n", i);
+
+                       ret = -EINVAL;
+                       goto end;
+               }
+               i++;
        }
+       th_zone->bind = true;
+end:
+       of_node_put(child);
+       of_node_put(np);
 
        return ret;
 }
@@ -182,16 +197,12 @@ static int exynos_bind(struct thermal_zone_device *thermal,
 static int exynos_unbind(struct thermal_zone_device *thermal,
                        struct thermal_cooling_device *cdev)
 {
-       int ret = 0, i, tab_size;
+       int ret = 0, i;
        struct exynos_thermal_zone *th_zone = thermal->devdata;
        struct thermal_sensor_conf *data = th_zone->sensor_conf;
+       struct device_node *child, *gchild, *np;
 
-       if (th_zone->bind == false)
-               return 0;
-
-       tab_size = data->cooling_data.freq_clip_count;
-
-       if (tab_size == 0)
+       if (th_zone->bind == false || !th_zone->cool_dev_size)
                return 0;
 
        /* find the cooling device registered*/
@@ -203,23 +214,30 @@ static int exynos_unbind(struct thermal_zone_device *thermal,
        if (i == th_zone->cool_dev_size)
                return 0;
 
-       /* Bind the thermal zone to the cpufreq cooling device */
-       for (i = 0; i < tab_size; i++) {
-               switch (GET_ZONE(i)) {
-               case MONITOR_ZONE:
-               case WARN_ZONE:
-                       if (thermal_zone_unbind_cooling_device(thermal, i,
-                                                               cdev)) {
-                               dev_err(data->dev,
-                                       "error unbinding cdev inst=%d\n", i);
-                               ret = -EINVAL;
-                       }
-                       th_zone->bind = false;
-                       break;
-               default:
+       np = of_find_node_by_path("/thermal-zones/cpu-thermal");
+       if (!np) {
+               pr_err("failed to find thmerla-zones/cpu-thermal node\n");
+               return -ENOENT;
+       }
+
+       child = of_get_child_by_name(np, "cooling-maps");
+
+       i = 0;
+       for_each_child_of_node(child, gchild) {
+               if (thermal_zone_unbind_cooling_device(thermal, i,
+                                                      cdev)) {
+                       dev_err(data->dev,
+                               "error unbinding cdev inst=%d\n", i);
                        ret = -EINVAL;
+                       goto end;
                }
+               i++;
        }
+       th_zone->bind = false;
+end:
+       of_node_put(child);
+       of_node_put(np);
+
        return ret;
 }
 
index 5000727c8c2b87a0a34ff819b62e264735bb265b..ae30f6af05e0183d4073f8f7ece6882eb95684d5 100644 (file)
@@ -916,13 +916,6 @@ static int exynos_tmu_probe(struct platform_device *pdev)
 
        sensor_conf->trip_data.trigger_falling = pdata->threshold_falling;
 
-       sensor_conf->cooling_data.freq_clip_count = pdata->freq_tab_count;
-       for (i = 0; i < pdata->freq_tab_count; i++) {
-               sensor_conf->cooling_data.freq_data[i].freq_clip_max =
-                                       pdata->freq_tab[i].freq_clip_max;
-               sensor_conf->cooling_data.freq_data[i].temp_level =
-                                       pdata->freq_tab[i].temp_level;
-       }
        sensor_conf->dev = &pdev->dev;
        /* Register the sensor with thermal management interface */
        ret = exynos_register_thermal(sensor_conf);
index 7f880d2e53f56a606d833fc84fedc2da24ff3fdd..627dec92ec1bfb628aae2433df919af2b681f244 100644 (file)
@@ -83,9 +83,6 @@ enum soc_type {
  * @second_point_trim: temp value of the second point trimming
  * @default_temp_offset: default temperature offset in case of no trimming
  * @cal_type: calibration type for temperature
- * @freq_clip_table: Table representing frequency reduction percentage.
- * @freq_tab_count: Count of the above table as frequency reduction may
- *     applicable to only some of the trigger levels.
  *
  * This structure is required for configuration of exynos_tmu driver.
  */
@@ -111,8 +108,6 @@ struct exynos_tmu_platform_data {
        enum soc_type type;
        u32 cal_type;
        u32 cal_mode;
-       struct freq_clip_table freq_tab[4];
-       unsigned int freq_tab_count;
 };
 
 /**
index b23910069f6877bda04353b8d19f345e31ac596c..a993f3d33f9b6ad3f2bc127e8ee6e2490c2221b6 100644 (file)
@@ -47,15 +47,6 @@ struct exynos_tmu_init_data const exynos4210_default_tmu_data = {
                .first_point_trim = 25,
                .second_point_trim = 85,
                .default_temp_offset = 50,
-               .freq_tab[0] = {
-                       .freq_clip_max = 800 * 1000,
-                       .temp_level = 85,
-                       },
-               .freq_tab[1] = {
-                       .freq_clip_max = 200 * 1000,
-                       .temp_level = 100,
-               },
-               .freq_tab_count = 2,
                .type = SOC_ARCH_EXYNOS4210,
                },
        },
@@ -87,16 +78,7 @@ struct exynos_tmu_init_data const exynos4210_default_tmu_data = {
        .max_efuse_value = 100, \
        .first_point_trim = 25, \
        .second_point_trim = 85, \
-       .default_temp_offset = 50, \
-       .freq_tab[0] = { \
-               .freq_clip_max = 800 * 1000, \
-               .temp_level = 70, \
-       }, \
-       .freq_tab[1] = { \
-               .freq_clip_max = 400 * 1000, \
-               .temp_level = 95, \
-       }, \
-       .freq_tab_count = 2
+       .default_temp_offset = 50
 
 struct exynos_tmu_init_data const exynos3250_default_tmu_data = {
        .tmu_data = {
@@ -133,16 +115,7 @@ struct exynos_tmu_init_data const exynos3250_default_tmu_data = {
        .max_efuse_value = 100, \
        .first_point_trim = 25, \
        .second_point_trim = 85, \
-       .default_temp_offset = 50, \
-       .freq_tab[0] = { \
-               .freq_clip_max = 1400 * 1000, \
-               .temp_level = 70, \
-       }, \
-       .freq_tab[1] = { \
-               .freq_clip_max = 400 * 1000, \
-               .temp_level = 95, \
-       }, \
-       .freq_tab_count = 2
+       .default_temp_offset = 50
 
 struct exynos_tmu_init_data const exynos4412_default_tmu_data = {
        .tmu_data = {
@@ -189,16 +162,7 @@ struct exynos_tmu_init_data const exynos5250_default_tmu_data = {
        .max_efuse_value = 100, \
        .first_point_trim = 25, \
        .second_point_trim = 85, \
-       .default_temp_offset = 50, \
-       .freq_tab[0] = { \
-               .freq_clip_max = 800 * 1000, \
-               .temp_level = 85, \
-       }, \
-       .freq_tab[1] = { \
-               .freq_clip_max = 200 * 1000, \
-               .temp_level = 103, \
-       }, \
-       .freq_tab_count = 2, \
+       .default_temp_offset = 50,
 
 #define EXYNOS5260_TMU_DATA \
        __EXYNOS5260_TMU_DATA \