* [PATCH] thermal: add trip parameter for of .get_trend
@ 2014-01-08 9:07 Wei Ni
2014-01-08 12:48 ` Eduardo Valentin
2014-01-08 12:48 ` Eduardo Valentin
0 siblings, 2 replies; 4+ messages in thread
From: Wei Ni @ 2014-01-08 9:07 UTC (permalink / raw)
To: eduardo.valentin, rui.zhang, mark.rutland
Cc: MLongnecker, swarren, linux-pm, linux-tegra, Wei Ni
Add trip parameter for the of-thermal's .get_trend(), so
that it can be compatible with current thermal framework.
Signed-off-by: Wei Ni <wni@nvidia.com>
---
drivers/thermal/of-thermal.c | 16 +++++-----------
drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 18 +++++++++---------
include/linux/thermal.h | 4 ++--
3 files changed, 16 insertions(+), 22 deletions(-)
diff --git a/drivers/thermal/of-thermal.c b/drivers/thermal/of-thermal.c
index 97c12cf..a503e61 100644
--- a/drivers/thermal/of-thermal.c
+++ b/drivers/thermal/of-thermal.c
@@ -97,7 +97,7 @@ struct __thermal_zone {
/* sensor interface */
void *sensor_data;
int (*get_temp)(void *, long *);
- int (*get_trend)(void *, long *);
+ int (*get_trend)(void *, int, long *);
};
/*** DT thermal zone device callbacks ***/
@@ -123,17 +123,11 @@ static int of_thermal_get_trend(struct thermal_zone_device *tz, int trip,
if (!data->get_trend)
return -EINVAL;
- r = data->get_trend(data->sensor_data, &dev_trend);
+ r = data->get_trend(data->sensor_data, trip, &dev_trend);
if (r)
return r;
- /* TODO: These intervals might have some thresholds, but in core code */
- if (dev_trend > 0)
- *trend = THERMAL_TREND_RAISING;
- else if (dev_trend < 0)
- *trend = THERMAL_TREND_DROPPING;
- else
- *trend = THERMAL_TREND_STABLE;
+ *trend = dev_trend;
return 0;
}
@@ -325,7 +319,7 @@ static struct thermal_zone_device *
thermal_zone_of_add_sensor(struct device_node *zone,
struct device_node *sensor, void *data,
int (*get_temp)(void *, long *),
- int (*get_trend)(void *, long *))
+ int (*get_trend)(void *, int, long *))
{
struct thermal_zone_device *tzd;
struct __thermal_zone *tz;
@@ -384,7 +378,7 @@ thermal_zone_of_add_sensor(struct device_node *zone,
struct thermal_zone_device *
thermal_zone_of_sensor_register(struct device *dev, int sensor_id,
void *data, int (*get_temp)(void *, long *),
- int (*get_trend)(void *, long *))
+ int (*get_trend)(void *, int, long *))
{
struct device_node *np, *child, *sensor_np;
diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
index 9eec26d..1a8ae03 100644
--- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
+++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
@@ -239,7 +239,7 @@ static int ti_thermal_get_trip_temp(struct thermal_zone_device *thermal,
return 0;
}
-static int __ti_thermal_get_trend(void *p, long *trend)
+static int __ti_thermal_get_trend(void *p, int trip, long *trend)
{
struct ti_thermal_data *data = p;
struct ti_bandgap *bgp;
@@ -252,7 +252,12 @@ static int __ti_thermal_get_trend(void *p, long *trend)
if (ret)
return ret;
- *trend = tr;
+ if (tr > 0)
+ *trend = THERMAL_TREND_RAISING;
+ else if (tr < 0)
+ *trend = THERMAL_TREND_DROPPING;
+ else
+ *trend = THERMAL_TREND_STABLE;
return 0;
}
@@ -264,16 +269,11 @@ static int ti_thermal_get_trend(struct thermal_zone_device *thermal,
int ret;
long tr;
- ret = __ti_thermal_get_trend(thermal->devdata, &tr);
+ ret = __ti_thermal_get_trend(thermal->devdata, trip, &tr);
if (ret)
return ret;
- if (tr > 0)
- *trend = THERMAL_TREND_RAISING;
- else if (tr < 0)
- *trend = THERMAL_TREND_DROPPING;
- else
- *trend = THERMAL_TREND_STABLE;
+ *trend = tr;
return 0;
}
diff --git a/include/linux/thermal.h b/include/linux/thermal.h
index a94de8c..37e4e03 100644
--- a/include/linux/thermal.h
+++ b/include/linux/thermal.h
@@ -249,14 +249,14 @@ struct thermal_genl_event {
struct thermal_zone_device *
thermal_zone_of_sensor_register(struct device *dev, int id,
void *data, int (*get_temp)(void *, long *),
- int (*get_trend)(void *, long *));
+ int (*get_trend)(void *, int, long *));
void thermal_zone_of_sensor_unregister(struct device *dev,
struct thermal_zone_device *tz);
#else
static inline struct thermal_zone_device *
thermal_zone_of_sensor_register(struct device *dev, int id,
void *data, int (*get_temp)(void *, long *),
- int (*get_trend)(void *, long *))
+ int (*get_trend)(void *, int, long *))
{
return NULL;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] thermal: add trip parameter for of .get_trend
2014-01-08 9:07 [PATCH] thermal: add trip parameter for of .get_trend Wei Ni
@ 2014-01-08 12:48 ` Eduardo Valentin
[not found] ` <52CD492C.2030508-l0cyMroinI0@public.gmane.org>
2014-01-08 12:48 ` Eduardo Valentin
1 sibling, 1 reply; 4+ messages in thread
From: Eduardo Valentin @ 2014-01-08 12:48 UTC (permalink / raw)
To: Wei Ni
Cc: eduardo.valentin, rui.zhang, mark.rutland, MLongnecker, swarren,
linux-pm, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 4993 bytes --]
On 08-01-2014 05:07, Wei Ni wrote:
> Add trip parameter for the of-thermal's .get_trend(), so
> that it can be compatible with current thermal framework.
You need to have a better look on the existing users of an API before
changing it.
>
> Signed-off-by: Wei Ni <wni@nvidia.com>
> ---
> drivers/thermal/of-thermal.c | 16 +++++-----------
> drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 18 +++++++++---------
> include/linux/thermal.h | 4 ++--
> 3 files changed, 16 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/thermal/of-thermal.c b/drivers/thermal/of-thermal.c
> index 97c12cf..a503e61 100644
> --- a/drivers/thermal/of-thermal.c
> +++ b/drivers/thermal/of-thermal.c
> @@ -97,7 +97,7 @@ struct __thermal_zone {
> /* sensor interface */
> void *sensor_data;
> int (*get_temp)(void *, long *);
> - int (*get_trend)(void *, long *);
> + int (*get_trend)(void *, int, long *);
> };
>
> /*** DT thermal zone device callbacks ***/
> @@ -123,17 +123,11 @@ static int of_thermal_get_trend(struct thermal_zone_device *tz, int trip,
> if (!data->get_trend)
> return -EINVAL;
>
> - r = data->get_trend(data->sensor_data, &dev_trend);
> + r = data->get_trend(data->sensor_data, trip, &dev_trend);
> if (r)
> return r;
>
> - /* TODO: These intervals might have some thresholds, but in core code */
> - if (dev_trend > 0)
> - *trend = THERMAL_TREND_RAISING;
> - else if (dev_trend < 0)
> - *trend = THERMAL_TREND_DROPPING;
> - else
> - *trend = THERMAL_TREND_STABLE;
> + *trend = dev_trend;
>
> return 0;
> }
> @@ -325,7 +319,7 @@ static struct thermal_zone_device *
> thermal_zone_of_add_sensor(struct device_node *zone,
> struct device_node *sensor, void *data,
> int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *))
> + int (*get_trend)(void *, int, long *))
> {
> struct thermal_zone_device *tzd;
> struct __thermal_zone *tz;
> @@ -384,7 +378,7 @@ thermal_zone_of_add_sensor(struct device_node *zone,
> struct thermal_zone_device *
> thermal_zone_of_sensor_register(struct device *dev, int sensor_id,
> void *data, int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *))
> + int (*get_trend)(void *, int, long *))
> {
> struct device_node *np, *child, *sensor_np;
>
> diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> index 9eec26d..1a8ae03 100644
> --- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> +++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> @@ -239,7 +239,7 @@ static int ti_thermal_get_trip_temp(struct thermal_zone_device *thermal,
> return 0;
> }
>
> -static int __ti_thermal_get_trend(void *p, long *trend)
> +static int __ti_thermal_get_trend(void *p, int trip, long *trend)
> {
> struct ti_thermal_data *data = p;
> struct ti_bandgap *bgp;
> @@ -252,7 +252,12 @@ static int __ti_thermal_get_trend(void *p, long *trend)
> if (ret)
> return ret;
>
> - *trend = tr;
> + if (tr > 0)
> + *trend = THERMAL_TREND_RAISING;
> + else if (tr < 0)
> + *trend = THERMAL_TREND_DROPPING;
> + else
> + *trend = THERMAL_TREND_STABLE;
>
> return 0;
> }
> @@ -264,16 +269,11 @@ static int ti_thermal_get_trend(struct thermal_zone_device *thermal,
> int ret;
> long tr;
>
> - ret = __ti_thermal_get_trend(thermal->devdata, &tr);
> + ret = __ti_thermal_get_trend(thermal->devdata, trip, &tr);
> if (ret)
> return ret;
>
> - if (tr > 0)
> - *trend = THERMAL_TREND_RAISING;
> - else if (tr < 0)
> - *trend = THERMAL_TREND_DROPPING;
> - else
> - *trend = THERMAL_TREND_STABLE;
> + *trend = tr;
This enforces users of this API (device drivers) to be compatible to
thermal framework, which is not always the case. For instance, hwmon
drivers are not aware of thermal constants.
>
> return 0;
> }
> diff --git a/include/linux/thermal.h b/include/linux/thermal.h
> index a94de8c..37e4e03 100644
> --- a/include/linux/thermal.h
> +++ b/include/linux/thermal.h
> @@ -249,14 +249,14 @@ struct thermal_genl_event {
> struct thermal_zone_device *
> thermal_zone_of_sensor_register(struct device *dev, int id,
> void *data, int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *));
> + int (*get_trend)(void *, int, long *));
> void thermal_zone_of_sensor_unregister(struct device *dev,
> struct thermal_zone_device *tz);
> #else
> static inline struct thermal_zone_device *
> thermal_zone_of_sensor_register(struct device *dev, int id,
> void *data, int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *))
> + int (*get_trend)(void *, int, long *))
> {
> return NULL;
> }
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] thermal: add trip parameter for of .get_trend
2014-01-08 9:07 [PATCH] thermal: add trip parameter for of .get_trend Wei Ni
2014-01-08 12:48 ` Eduardo Valentin
@ 2014-01-08 12:48 ` Eduardo Valentin
1 sibling, 0 replies; 4+ messages in thread
From: Eduardo Valentin @ 2014-01-08 12:48 UTC (permalink / raw)
To: Wei Ni
Cc: eduardo.valentin, rui.zhang, mark.rutland, MLongnecker, swarren,
linux-pm, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 4993 bytes --]
On 08-01-2014 05:07, Wei Ni wrote:
> Add trip parameter for the of-thermal's .get_trend(), so
> that it can be compatible with current thermal framework.
You need to have a better look on the existing users of an API before
changing it.
>
> Signed-off-by: Wei Ni <wni@nvidia.com>
> ---
> drivers/thermal/of-thermal.c | 16 +++++-----------
> drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 18 +++++++++---------
> include/linux/thermal.h | 4 ++--
> 3 files changed, 16 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/thermal/of-thermal.c b/drivers/thermal/of-thermal.c
> index 97c12cf..a503e61 100644
> --- a/drivers/thermal/of-thermal.c
> +++ b/drivers/thermal/of-thermal.c
> @@ -97,7 +97,7 @@ struct __thermal_zone {
> /* sensor interface */
> void *sensor_data;
> int (*get_temp)(void *, long *);
> - int (*get_trend)(void *, long *);
> + int (*get_trend)(void *, int, long *);
> };
>
> /*** DT thermal zone device callbacks ***/
> @@ -123,17 +123,11 @@ static int of_thermal_get_trend(struct thermal_zone_device *tz, int trip,
> if (!data->get_trend)
> return -EINVAL;
>
> - r = data->get_trend(data->sensor_data, &dev_trend);
> + r = data->get_trend(data->sensor_data, trip, &dev_trend);
> if (r)
> return r;
>
> - /* TODO: These intervals might have some thresholds, but in core code */
> - if (dev_trend > 0)
> - *trend = THERMAL_TREND_RAISING;
> - else if (dev_trend < 0)
> - *trend = THERMAL_TREND_DROPPING;
> - else
> - *trend = THERMAL_TREND_STABLE;
> + *trend = dev_trend;
>
> return 0;
> }
> @@ -325,7 +319,7 @@ static struct thermal_zone_device *
> thermal_zone_of_add_sensor(struct device_node *zone,
> struct device_node *sensor, void *data,
> int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *))
> + int (*get_trend)(void *, int, long *))
> {
> struct thermal_zone_device *tzd;
> struct __thermal_zone *tz;
> @@ -384,7 +378,7 @@ thermal_zone_of_add_sensor(struct device_node *zone,
> struct thermal_zone_device *
> thermal_zone_of_sensor_register(struct device *dev, int sensor_id,
> void *data, int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *))
> + int (*get_trend)(void *, int, long *))
> {
> struct device_node *np, *child, *sensor_np;
>
> diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> index 9eec26d..1a8ae03 100644
> --- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> +++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> @@ -239,7 +239,7 @@ static int ti_thermal_get_trip_temp(struct thermal_zone_device *thermal,
> return 0;
> }
>
> -static int __ti_thermal_get_trend(void *p, long *trend)
> +static int __ti_thermal_get_trend(void *p, int trip, long *trend)
> {
> struct ti_thermal_data *data = p;
> struct ti_bandgap *bgp;
> @@ -252,7 +252,12 @@ static int __ti_thermal_get_trend(void *p, long *trend)
> if (ret)
> return ret;
>
> - *trend = tr;
> + if (tr > 0)
> + *trend = THERMAL_TREND_RAISING;
> + else if (tr < 0)
> + *trend = THERMAL_TREND_DROPPING;
> + else
> + *trend = THERMAL_TREND_STABLE;
>
> return 0;
> }
> @@ -264,16 +269,11 @@ static int ti_thermal_get_trend(struct thermal_zone_device *thermal,
> int ret;
> long tr;
>
> - ret = __ti_thermal_get_trend(thermal->devdata, &tr);
> + ret = __ti_thermal_get_trend(thermal->devdata, trip, &tr);
> if (ret)
> return ret;
>
> - if (tr > 0)
> - *trend = THERMAL_TREND_RAISING;
> - else if (tr < 0)
> - *trend = THERMAL_TREND_DROPPING;
> - else
> - *trend = THERMAL_TREND_STABLE;
> + *trend = tr;
This enforces users of this API (device drivers) to be compatible to
thermal framework, which is not always the case. For instance, hwmon
drivers are not aware of thermal constants.
>
> return 0;
> }
> diff --git a/include/linux/thermal.h b/include/linux/thermal.h
> index a94de8c..37e4e03 100644
> --- a/include/linux/thermal.h
> +++ b/include/linux/thermal.h
> @@ -249,14 +249,14 @@ struct thermal_genl_event {
> struct thermal_zone_device *
> thermal_zone_of_sensor_register(struct device *dev, int id,
> void *data, int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *));
> + int (*get_trend)(void *, int, long *));
> void thermal_zone_of_sensor_unregister(struct device *dev,
> struct thermal_zone_device *tz);
> #else
> static inline struct thermal_zone_device *
> thermal_zone_of_sensor_register(struct device *dev, int id,
> void *data, int (*get_temp)(void *, long *),
> - int (*get_trend)(void *, long *))
> + int (*get_trend)(void *, int, long *))
> {
> return NULL;
> }
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] thermal: add trip parameter for of .get_trend
[not found] ` <52CD492C.2030508-l0cyMroinI0@public.gmane.org>
@ 2014-01-09 7:55 ` Wei Ni
0 siblings, 0 replies; 4+ messages in thread
From: Wei Ni @ 2014-01-09 7:55 UTC (permalink / raw)
To: Eduardo Valentin
Cc: rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
mark.rutland-5wv7dgnIgG8@public.gmane.org, Matthew Longnecker,
swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org,
linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
On 01/08/2014 08:48 PM, Eduardo Valentin wrote:
> * PGP Signed by an unknown key
>
> On 08-01-2014 05:07, Wei Ni wrote:
>> Add trip parameter for the of-thermal's .get_trend(), so
>> that it can be compatible with current thermal framework.
>
> You need to have a better look on the existing users of an API before
> changing it.
>
>>
>> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> ---
>> drivers/thermal/of-thermal.c | 16 +++++-----------
>> drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 18 +++++++++---------
>> include/linux/thermal.h | 4 ++--
>> 3 files changed, 16 insertions(+), 22 deletions(-)
>>
>> diff --git a/drivers/thermal/of-thermal.c b/drivers/thermal/of-thermal.c
>> index 97c12cf..a503e61 100644
>> --- a/drivers/thermal/of-thermal.c
>> +++ b/drivers/thermal/of-thermal.c
>> @@ -97,7 +97,7 @@ struct __thermal_zone {
>> /* sensor interface */
>> void *sensor_data;
>> int (*get_temp)(void *, long *);
>> - int (*get_trend)(void *, long *);
>> + int (*get_trend)(void *, int, long *);
>> };
>>
>> /*** DT thermal zone device callbacks ***/
>> @@ -123,17 +123,11 @@ static int of_thermal_get_trend(struct thermal_zone_device *tz, int trip,
>> if (!data->get_trend)
>> return -EINVAL;
>>
>> - r = data->get_trend(data->sensor_data, &dev_trend);
>> + r = data->get_trend(data->sensor_data, trip, &dev_trend);
>> if (r)
>> return r;
>>
>> - /* TODO: These intervals might have some thresholds, but in core code */
>> - if (dev_trend > 0)
>> - *trend = THERMAL_TREND_RAISING;
>> - else if (dev_trend < 0)
>> - *trend = THERMAL_TREND_DROPPING;
>> - else
>> - *trend = THERMAL_TREND_STABLE;
>> + *trend = dev_trend;
>>
>> return 0;
>> }
>> @@ -325,7 +319,7 @@ static struct thermal_zone_device *
>> thermal_zone_of_add_sensor(struct device_node *zone,
>> struct device_node *sensor, void *data,
>> int (*get_temp)(void *, long *),
>> - int (*get_trend)(void *, long *))
>> + int (*get_trend)(void *, int, long *))
>> {
>> struct thermal_zone_device *tzd;
>> struct __thermal_zone *tz;
>> @@ -384,7 +378,7 @@ thermal_zone_of_add_sensor(struct device_node *zone,
>> struct thermal_zone_device *
>> thermal_zone_of_sensor_register(struct device *dev, int sensor_id,
>> void *data, int (*get_temp)(void *, long *),
>> - int (*get_trend)(void *, long *))
>> + int (*get_trend)(void *, int, long *))
>> {
>> struct device_node *np, *child, *sensor_np;
>>
>> diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
>> index 9eec26d..1a8ae03 100644
>> --- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
>> +++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
>> @@ -239,7 +239,7 @@ static int ti_thermal_get_trip_temp(struct thermal_zone_device *thermal,
>> return 0;
>> }
>>
>> -static int __ti_thermal_get_trend(void *p, long *trend)
>> +static int __ti_thermal_get_trend(void *p, int trip, long *trend)
>> {
>> struct ti_thermal_data *data = p;
>> struct ti_bandgap *bgp;
>> @@ -252,7 +252,12 @@ static int __ti_thermal_get_trend(void *p, long *trend)
>> if (ret)
>> return ret;
>>
>> - *trend = tr;
>> + if (tr > 0)
>> + *trend = THERMAL_TREND_RAISING;
>> + else if (tr < 0)
>> + *trend = THERMAL_TREND_DROPPING;
>> + else
>> + *trend = THERMAL_TREND_STABLE;
>>
>> return 0;
>> }
>> @@ -264,16 +269,11 @@ static int ti_thermal_get_trend(struct thermal_zone_device *thermal,
>> int ret;
>> long tr;
>>
>> - ret = __ti_thermal_get_trend(thermal->devdata, &tr);
>> + ret = __ti_thermal_get_trend(thermal->devdata, trip, &tr);
>> if (ret)
>> return ret;
>>
>> - if (tr > 0)
>> - *trend = THERMAL_TREND_RAISING;
>> - else if (tr < 0)
>> - *trend = THERMAL_TREND_DROPPING;
>> - else
>> - *trend = THERMAL_TREND_STABLE;
>> + *trend = tr;
>
> This enforces users of this API (device drivers) to be compatible to
> thermal framework, which is not always the case. For instance, hwmon
> drivers are not aware of thermal constants.
I think the of-thermal driver is used to improve the thermal framework,
if the user want to use the .get_trend, and handle the parameter "trip",
then they can not use use this new DT framework to register into thermal
framework.
Different thermal driver may use different logic to calculate the trend,
I think we should not calculate it in framework.
For the hwmon driver, yes they are not aware of thermal constants, but
it can set the .get_trend to NULL, or doesn't need to touch the "trip"
in it's get_trend callback function.
Thanks.
Wei.
>
>>
>> return 0;
>> }
>> diff --git a/include/linux/thermal.h b/include/linux/thermal.h
>> index a94de8c..37e4e03 100644
>> --- a/include/linux/thermal.h
>> +++ b/include/linux/thermal.h
>> @@ -249,14 +249,14 @@ struct thermal_genl_event {
>> struct thermal_zone_device *
>> thermal_zone_of_sensor_register(struct device *dev, int id,
>> void *data, int (*get_temp)(void *, long *),
>> - int (*get_trend)(void *, long *));
>> + int (*get_trend)(void *, int, long *));
>> void thermal_zone_of_sensor_unregister(struct device *dev,
>> struct thermal_zone_device *tz);
>> #else
>> static inline struct thermal_zone_device *
>> thermal_zone_of_sensor_register(struct device *dev, int id,
>> void *data, int (*get_temp)(void *, long *),
>> - int (*get_trend)(void *, long *))
>> + int (*get_trend)(void *, int, long *))
>> {
>> return NULL;
>> }
>>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-01-09 7:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-08 9:07 [PATCH] thermal: add trip parameter for of .get_trend Wei Ni
2014-01-08 12:48 ` Eduardo Valentin
[not found] ` <52CD492C.2030508-l0cyMroinI0@public.gmane.org>
2014-01-09 7:55 ` Wei Ni
2014-01-08 12:48 ` Eduardo Valentin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).