lkml.org 
[lkml]   [2013]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH V2 2/2] thermal: exynos: Use the framework for temperature emulation support
From
Date
On Mon, 2013-02-04 at 10:14 +0800, Zhang Rui 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@samsung.com>
> > Acked-by: Kukjin Kim <kgene.kim@samsung.com>
> > ---
> > 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?
>
maybe you missed this email.
IMO, the code above should be much cleaner.

thanks,
rui
> 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();




\
 
 \ /
  Last update: 2013-02-08 04:21    [W:0.084 / U:0.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site