Re: [PATCH V2 2/2] thermal: exynos: Use the framework for temperature emulation support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rui,

Please merge this patch also. The 1st series of this patchset is
already accepted by you. This is just a adaptation of the earlier one
and does code cleanup.

Thanks,
Amit Daniel

On Sun, Feb 3, 2013 at 6:14 PM, Zhang Rui <rui.zhang@xxxxxxxxx> wrote:
> On Sun, 2013-01-27 at 19:28 -0800, Amit Daniel Kachhap wrote:
>> This removes the driver specific sysfs support of the temperature
>> emulation and uses the newly added core thermal framework for thermal
>> emulation. A platform specific handler is added to support this.
>>
>> Signed-off-by: Amit Daniel Kachhap <amit.daniel@xxxxxxxxxxx>
>> Acked-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
>> ---
>> Changes in V2:
>> * Added config option CONFIG_THERMAL_EMULATION instead of
>>  CONFIG_EXYNOS_THERMAL_EMUL
>>
>> This patchset is based on thermal maintainer next tree.
>> git://git.kernel.org/pub/scm/linux/kernel/git/rzhang/linux.git next
>>
>>  Documentation/thermal/exynos_thermal_emulation |    8 +-
>>  drivers/thermal/Kconfig                        |    9 --
>>  drivers/thermal/exynos_thermal.c               |  158 ++++++++++--------------
>>  3 files changed, 67 insertions(+), 108 deletions(-)
>>
>> diff --git a/Documentation/thermal/exynos_thermal_emulation b/Documentation/thermal/exynos_thermal_emulation
>> index b73bbfb..36a3e79 100644
>> --- a/Documentation/thermal/exynos_thermal_emulation
>> +++ b/Documentation/thermal/exynos_thermal_emulation
>> @@ -13,11 +13,11 @@ Thermal emulation mode supports software debug for TMU's operation. User can set
>>  manually with software code and TMU will read current temperature from user value not from
>>  sensor's value.
>>
>> -Enabling CONFIG_EXYNOS_THERMAL_EMUL option will make this support in available.
>> -When it's enabled, sysfs node will be created under
>> -/sys/bus/platform/devices/'exynos device name'/ with name of 'emulation'.
>> +Enabling CONFIG_THERMAL_EMULATION option will make this support available.
>> +When it's enabled, sysfs node will be created as
>> +/sys/devices/virtual/thermal/thermal_zone'zone id'/emul_temp.
>>
>> -The sysfs node, 'emulation', will contain value 0 for the initial state. When you input any
>> +The sysfs node, 'emul_node', will contain value 0 for the initial state. When you input any
>>  temperature you want to update to sysfs node, it automatically enable emulation mode and
>>  current temperature will be changed into it.
>>  (Exynos also supports user changable delay time which would be used to delay of
>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>> index e4cf7fb..2a79510 100644
>> --- a/drivers/thermal/Kconfig
>> +++ b/drivers/thermal/Kconfig
>> @@ -109,15 +109,6 @@ config EXYNOS_THERMAL
>>         If you say yes here you get support for TMU (Thermal Management
>>         Unit) on SAMSUNG EXYNOS series of SoC.
>>
>> -config EXYNOS_THERMAL_EMUL
>> -     bool "EXYNOS TMU emulation mode support"
>> -     depends on EXYNOS_THERMAL
>> -     help
>> -       Exynos 4412 and 4414 and 5 series has emulation mode on TMU.
>> -       Enable this option will be make sysfs node in exynos thermal platform
>> -       device directory to support emulation mode. With emulation mode sysfs
>> -       node, you can manually input temperature to TMU for simulation purpose.
>> -
>>  config DB8500_THERMAL
>>       bool "DB8500 thermal management"
>>       depends on ARCH_U8500
>> diff --git a/drivers/thermal/exynos_thermal.c b/drivers/thermal/exynos_thermal.c
>> index 327102a..afe9c2a 100644
>> --- a/drivers/thermal/exynos_thermal.c
>> +++ b/drivers/thermal/exynos_thermal.c
>> @@ -99,13 +99,13 @@
>>  #define IDLE_INTERVAL 10000
>>  #define MCELSIUS     1000
>>
>> -#ifdef CONFIG_EXYNOS_THERMAL_EMUL
>> +#ifdef CONFIG_THERMAL_EMULATION
>>  #define EXYNOS_EMUL_TIME     0x57F0
>>  #define EXYNOS_EMUL_TIME_SHIFT       16
>>  #define EXYNOS_EMUL_DATA_SHIFT       8
>>  #define EXYNOS_EMUL_DATA_MASK        0xFF
>>  #define EXYNOS_EMUL_ENABLE   0x1
>> -#endif /* CONFIG_EXYNOS_THERMAL_EMUL */
>> +#endif /* CONFIG_THERMAL_EMULATION */
>>
>>  /* CPU Zone information */
>>  #define PANIC_ZONE      4
>> @@ -143,6 +143,7 @@ struct    thermal_cooling_conf {
>>  struct thermal_sensor_conf {
>>       char name[SENSOR_NAME_LEN];
>>       int (*read_temperature)(void *data);
>> +     int (*write_emul_temp)(void *data, unsigned long temp);
>>       struct thermal_trip_point_conf trip_data;
>>       struct thermal_cooling_conf cooling_data;
>>       void *private_data;
>> @@ -366,6 +367,23 @@ static int exynos_get_temp(struct thermal_zone_device *thermal,
>>       return 0;
>>  }
>>
>> +/* Get temperature callback functions for thermal zone */
>> +static int exynos_set_emul_temp(struct thermal_zone_device *thermal,
>> +                                             unsigned long temp)
>> +{
>> +     void *data;
>> +     int ret = -EINVAL;
>> +
>> +     if (!th_zone->sensor_conf) {
>> +             pr_info("Temperature sensor not initialised\n");
>> +             return -EINVAL;
>> +     }
>> +     data = th_zone->sensor_conf->private_data;
>> +     if (th_zone->sensor_conf->write_emul_temp)
>> +             ret = th_zone->sensor_conf->write_emul_temp(data, temp);
>> +     return ret;
>> +}
>> +
>
>>  /* Get the temperature trend */
>>  static int exynos_get_trend(struct thermal_zone_device *thermal,
>>                       int trip, enum thermal_trend *trend)
>> @@ -382,6 +400,7 @@ static struct thermal_zone_device_ops const exynos_dev_ops = {
>>       .bind = exynos_bind,
>>       .unbind = exynos_unbind,
>>       .get_temp = exynos_get_temp,
>> +     .set_emul_temp = exynos_set_emul_temp,
>>       .get_trend = exynos_get_trend,
>>       .get_mode = exynos_get_mode,
>>       .set_mode = exynos_set_mode,
>> @@ -700,6 +719,44 @@ static int exynos_tmu_read(struct exynos_tmu_data *data)
>>       return temp;
>>  }
>>
>> +#ifdef CONFIG_THERMAL_EMULATION
>> +static int exynos_tmu_set_emulation(struct exynos_tmu_data *data,
>> +                                     unsigned long temp)
>> +{
>> +     unsigned int reg;
>> +     int ret = -EINVAL;
>> +
>> +     if (data->soc == SOC_ARCH_EXYNOS4210)
>> +             goto out;
>> +
>> +     if (temp && temp < MCELSIUS)
>> +             goto out;
>> +
>> +     mutex_lock(&data->lock);
>> +     clk_enable(data->clk);
>> +
>> +     reg = readl(data->base + EXYNOS_EMUL_CON);
>> +
>> +     if (temp) {
>> +             temp /= MCELSIUS;
>> +
>> +             reg = (EXYNOS_EMUL_TIME << EXYNOS_EMUL_TIME_SHIFT) |
>> +                     (temp_to_code(data, temp)
>> +                      << EXYNOS_EMUL_DATA_SHIFT) | EXYNOS_EMUL_ENABLE;
>> +     } else {
>> +             reg &= ~EXYNOS_EMUL_ENABLE;
>> +     }
>> +
>> +     writel(reg, data->base + EXYNOS_EMUL_CON);
>> +
>> +     clk_disable(data->clk);
>> +     mutex_unlock(&data->lock);
>> +     return 0;
>> +out:
>> +     return ret;
>> +}
>> +#endif/*CONFIG_THERMAL_EMULATION*/
>> +
>>  static void exynos_tmu_work(struct work_struct *work)
>>  {
>>       struct exynos_tmu_data *data = container_of(work,
>> @@ -839,93 +896,6 @@ static inline struct  exynos_tmu_platform_data *exynos_get_driver_data(
>>                       platform_get_device_id(pdev)->driver_data;
>>  }
>>
>> -#ifdef CONFIG_EXYNOS_THERMAL_EMUL
>> -static ssize_t exynos_tmu_emulation_show(struct device *dev,
>> -                                      struct device_attribute *attr,
>> -                                      char *buf)
>> -{
>> -     struct platform_device *pdev = container_of(dev,
>> -                                     struct platform_device, dev);
>> -     struct exynos_tmu_data *data = platform_get_drvdata(pdev);
>> -     unsigned int reg;
>> -     u8 temp_code;
>> -     int temp = 0;
>> -
>> -     if (data->soc == SOC_ARCH_EXYNOS4210)
>> -             goto out;
>> -
>> -     mutex_lock(&data->lock);
>> -     clk_enable(data->clk);
>> -     reg = readl(data->base + EXYNOS_EMUL_CON);
>> -     clk_disable(data->clk);
>> -     mutex_unlock(&data->lock);
>> -
>> -     if (reg & EXYNOS_EMUL_ENABLE) {
>> -             reg >>= EXYNOS_EMUL_DATA_SHIFT;
>> -             temp_code = reg & EXYNOS_EMUL_DATA_MASK;
>> -             temp = code_to_temp(data, temp_code);
>> -     }
>> -out:
>> -     return sprintf(buf, "%d\n", temp * MCELSIUS);
>> -}
>> -
>> -static ssize_t exynos_tmu_emulation_store(struct device *dev,
>> -                                     struct device_attribute *attr,
>> -                                     const char *buf, size_t count)
>> -{
>> -     struct platform_device *pdev = container_of(dev,
>> -                                     struct platform_device, dev);
>> -     struct exynos_tmu_data *data = platform_get_drvdata(pdev);
>> -     unsigned int reg;
>> -     int temp;
>> -
>> -     if (data->soc == SOC_ARCH_EXYNOS4210)
>> -             goto out;
>> -
>> -     if (!sscanf(buf, "%d\n", &temp) || temp < 0)
>> -             return -EINVAL;
>> -
>> -     mutex_lock(&data->lock);
>> -     clk_enable(data->clk);
>> -
>> -     reg = readl(data->base + EXYNOS_EMUL_CON);
>> -
>> -     if (temp) {
>> -             /* Both CELSIUS and MCELSIUS type are available for input */
>> -             if (temp > MCELSIUS)
>> -                     temp /= MCELSIUS;
>> -
>> -             reg = (EXYNOS_EMUL_TIME << EXYNOS_EMUL_TIME_SHIFT) |
>> -                     (temp_to_code(data, (temp / MCELSIUS))
>> -                      << EXYNOS_EMUL_DATA_SHIFT) | EXYNOS_EMUL_ENABLE;
>> -     } else {
>> -             reg &= ~EXYNOS_EMUL_ENABLE;
>> -     }
>> -
>> -     writel(reg, data->base + EXYNOS_EMUL_CON);
>> -
>> -     clk_disable(data->clk);
>> -     mutex_unlock(&data->lock);
>> -
>> -out:
>> -     return count;
>> -}
>> -
>> -static DEVICE_ATTR(emulation, 0644, exynos_tmu_emulation_show,
>> -                                     exynos_tmu_emulation_store);
>> -static int create_emulation_sysfs(struct device *dev)
>> -{
>> -     return device_create_file(dev, &dev_attr_emulation);
>> -}
>> -static void remove_emulation_sysfs(struct device *dev)
>> -{
>> -     device_remove_file(dev, &dev_attr_emulation);
>> -}
>> -#else
>> -static inline int create_emulation_sysfs(struct device *dev) { return 0; }
>> -static inline void remove_emulation_sysfs(struct device *dev) {}
>> -#endif
>> -
>>  static int __devinit exynos_tmu_probe(struct platform_device *pdev)
>>  {
>>       struct exynos_tmu_data *data;
>> @@ -1002,6 +972,10 @@ static int __devinit exynos_tmu_probe(struct platform_device *pdev)
>>
>>       /* Register the sensor with thermal management interface */
>>       (&exynos_sensor_conf)->private_data = data;
>> +#ifdef CONFIG_THERMAL_EMULATION
>> +     (&exynos_sensor_conf)->write_emul_temp =
>> +             (int (*)(void *, unsigned long))exynos_tmu_set_emulation;
>> +#endif
>
> this is a little complicate.
> to enable emulation, you just need
>
> #ifdef CONFIG_THERMAL_EMUL
> static int exynos_set_emul_temp(struct thermal_zone_device *thermal,
>                                                 unsigned long temp)
> {
>         remove exynos_tmu_set_emulation and copy its code here.
> }
> #else
> static int exynos_set_emul_temp() { return -EINVAL; }
> #endif
>
> and
> + .set_emul_temp = exynos_set_emul_temp,
>
> or do I miss something?

The above code just assigns the emulated function pointer. The purpose
in doing that is to keep the senor related and framework
related(thermal zone, cooling devices) separate. I am current working
on adding some more sensor support using the same framework code and
the above way of doing it is helpful.

>
> thanks,
> rui
>>       exynos_sensor_conf.trip_data.trip_count = pdata->trigger_level0_en +
>>                       pdata->trigger_level1_en + pdata->trigger_level2_en +
>>                       pdata->trigger_level3_en;
>> @@ -1025,10 +999,6 @@ static int __devinit exynos_tmu_probe(struct platform_device *pdev)
>>               goto err_clk;
>>       }
>>
>> -     ret = create_emulation_sysfs(&pdev->dev);
>> -     if (ret)
>> -             dev_err(&pdev->dev, "Failed to create emulation mode sysfs node\n");
>> -
>>       return 0;
>>  err_clk:
>>       platform_set_drvdata(pdev, NULL);
>> @@ -1040,8 +1010,6 @@ static int __devexit exynos_tmu_remove(struct platform_device *pdev)
>>  {
>>       struct exynos_tmu_data *data = platform_get_drvdata(pdev);
>>
>> -     remove_emulation_sysfs(&pdev->dev);
>> -
>>       exynos_tmu_control(pdev, false);
>>
>>       exynos_unregister_thermal();
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux