* Re: [RFC PATCH 7/12] thermal: rename structure thermal_cooling_device_instance to thermal_instance
[not found] <1339384821.1492.161.camel@rui.sh.intel.com>
@ 2012-06-11 6:21 ` Zhang Rui
2012-06-12 14:39 ` Eduardo Valentin
0 siblings, 1 reply; 2+ messages in thread
From: Zhang Rui @ 2012-06-11 6:21 UTC (permalink / raw)
To: linux-pm; +Cc: linux-acpi
seems this patch is missed. Resend it.
On 一, 2012-06-11 at 11:20 +0800, Zhang Rui wrote:
> Rename structure thermal_cooling_device_instance to thermal_instance.
>
> This struct is used to describe the behavior for a thermal
> cooling device on a certain trip point for a certain thremal zone.
>
> thermal_cooling_device_instance is not accurate, as a cooling device
> can be used for more than one trip point in one thermal zone device.
>
> Signed-off-by: Zhang Rui <rui.zhang@intel.com>
> ---
> drivers/thermal/thermal_sys.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> Index: rtd3/drivers/thermal/thermal_sys.c
> ===================================================================
> --- rtd3.orig/drivers/thermal/thermal_sys.c
> +++ rtd3/drivers/thermal/thermal_sys.c
> @@ -46,7 +46,7 @@ MODULE_LICENSE("GPL");
> * a certain cooling device on a certain trip point
> * in a certain thermal zone
> */
> -struct thermal_cooling_device_instance {
> +struct thermal_instance {
> int id;
> char name[THERMAL_NAME_LENGTH];
> struct thermal_zone_device *tz;
> @@ -388,10 +388,10 @@ static ssize_t
> thermal_cooling_device_trip_point_show(struct device *dev,
> struct device_attribute *attr, char *buf)
> {
> - struct thermal_cooling_device_instance *instance;
> + struct thermal_instance *instance;
>
> instance =
> - container_of(attr, struct thermal_cooling_device_instance, attr);
> + container_of(attr, struct thermal_instance, attr);
>
> if (instance->trip == THERMAL_TRIPS_NONE)
> return sprintf(buf, "-1\n");
> @@ -686,7 +686,7 @@ static void thermal_zone_device_passive(
> int temp, int trip_temp, int trip)
> {
> int trend = 0;
> - struct thermal_cooling_device_instance *instance;
> + struct thermal_instance *instance;
> struct thermal_cooling_device *cdev;
> long state, max_state;
>
> @@ -771,8 +771,8 @@ int thermal_zone_bind_cooling_device(str
> struct thermal_cooling_device *cdev,
> long upper, long lower)
> {
> - struct thermal_cooling_device_instance *dev;
> - struct thermal_cooling_device_instance *pos;
> + struct thermal_instance *dev;
> + struct thermal_instance *pos;
> struct thermal_zone_device *pos1;
> struct thermal_cooling_device *pos2;
> unsigned long max_state;
> @@ -803,7 +803,7 @@ int thermal_zone_bind_cooling_device(str
> return -EINVAL;
>
> dev =
> - kzalloc(sizeof(struct thermal_cooling_device_instance), GFP_KERNEL);
> + kzalloc(sizeof(struct thermal_instance), GFP_KERNEL);
> if (!dev)
> return -ENOMEM;
> dev->tz = tz;
> @@ -868,7 +868,7 @@ int thermal_zone_unbind_cooling_device(s
> int trip,
> struct thermal_cooling_device *cdev)
> {
> - struct thermal_cooling_device_instance *pos, *next;
> + struct thermal_instance *pos, *next;
>
> mutex_lock(&tz->lock);
> list_for_each_entry_safe(pos, next, &tz->cooling_devices, node) {
> @@ -1054,7 +1054,7 @@ EXPORT_SYMBOL(thermal_cooling_device_unr
> static void thermal_zone_trip_update(struct thermal_zone_device *tz,
> int trip, long temp)
> {
> - struct thermal_cooling_device_instance *instance;
> + struct thermal_instance *instance;
> struct thermal_cooling_device *cdev = NULL;
> unsigned long cur_state, max_state;
> long trip_temp;
>
_______________________________________________
linux-pm mailing list
linux-pm@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-pm
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [RFC PATCH 7/12] thermal: rename structure thermal_cooling_device_instance to thermal_instance
2012-06-11 6:21 ` [RFC PATCH 7/12] thermal: rename structure thermal_cooling_device_instance to thermal_instance Zhang Rui
@ 2012-06-12 14:39 ` Eduardo Valentin
0 siblings, 0 replies; 2+ messages in thread
From: Eduardo Valentin @ 2012-06-12 14:39 UTC (permalink / raw)
To: Zhang Rui
Cc: linux-pm, Amit Kachhap, eduardo, R, Durgadoss, Len, Brown,
Rafael J. Wysocki, Matthew Garrett, linux-acpi
Hello Rui,
On Mon, Jun 11, 2012 at 02:21:36PM +0800, Zhang Rui wrote:
> seems this patch is missed. Resend it.
>
> On 一, 2012-06-11 at 11:20 +0800, Zhang Rui wrote:
> > Rename structure thermal_cooling_device_instance to thermal_instance.
> >
> > This struct is used to describe the behavior for a thermal
> > cooling device on a certain trip point for a certain thremal zone.
typo
> >
> > thermal_cooling_device_instance is not accurate, as a cooling device
> > can be used for more than one trip point in one thermal zone device.
'thermal_instance' seams to be a bit misleading though...
It looks a bit too generic. One reading it won't figure it out that
it is supposed the mapping between cooling device vs. trip vs. thermal zone...
How about 'thermal_mapping' or 'thermal_binding'?
> >
> > Signed-off-by: Zhang Rui <rui.zhang@intel.com>
> > ---
> > drivers/thermal/thermal_sys.c | 18 +++++++++---------
> > 1 file changed, 9 insertions(+), 9 deletions(-)
> >
> > Index: rtd3/drivers/thermal/thermal_sys.c
> > ===================================================================
> > --- rtd3.orig/drivers/thermal/thermal_sys.c
> > +++ rtd3/drivers/thermal/thermal_sys.c
> > @@ -46,7 +46,7 @@ MODULE_LICENSE("GPL");
> > * a certain cooling device on a certain trip point
> > * in a certain thermal zone
> > */
> > -struct thermal_cooling_device_instance {
> > +struct thermal_instance {
> > int id;
> > char name[THERMAL_NAME_LENGTH];
> > struct thermal_zone_device *tz;
> > @@ -388,10 +388,10 @@ static ssize_t
> > thermal_cooling_device_trip_point_show(struct device *dev,
> > struct device_attribute *attr, char *buf)
> > {
> > - struct thermal_cooling_device_instance *instance;
> > + struct thermal_instance *instance;
> >
> > instance =
> > - container_of(attr, struct thermal_cooling_device_instance, attr);
> > + container_of(attr, struct thermal_instance, attr);
> >
> > if (instance->trip == THERMAL_TRIPS_NONE)
> > return sprintf(buf, "-1\n");
> > @@ -686,7 +686,7 @@ static void thermal_zone_device_passive(
> > int temp, int trip_temp, int trip)
> > {
> > int trend = 0;
> > - struct thermal_cooling_device_instance *instance;
> > + struct thermal_instance *instance;
> > struct thermal_cooling_device *cdev;
> > long state, max_state;
> >
> > @@ -771,8 +771,8 @@ int thermal_zone_bind_cooling_device(str
> > struct thermal_cooling_device *cdev,
> > long upper, long lower)
> > {
> > - struct thermal_cooling_device_instance *dev;
> > - struct thermal_cooling_device_instance *pos;
> > + struct thermal_instance *dev;
> > + struct thermal_instance *pos;
> > struct thermal_zone_device *pos1;
> > struct thermal_cooling_device *pos2;
> > unsigned long max_state;
> > @@ -803,7 +803,7 @@ int thermal_zone_bind_cooling_device(str
> > return -EINVAL;
> >
> > dev =
> > - kzalloc(sizeof(struct thermal_cooling_device_instance), GFP_KERNEL);
> > + kzalloc(sizeof(struct thermal_instance), GFP_KERNEL);
> > if (!dev)
> > return -ENOMEM;
> > dev->tz = tz;
> > @@ -868,7 +868,7 @@ int thermal_zone_unbind_cooling_device(s
> > int trip,
> > struct thermal_cooling_device *cdev)
> > {
> > - struct thermal_cooling_device_instance *pos, *next;
> > + struct thermal_instance *pos, *next;
> >
> > mutex_lock(&tz->lock);
> > list_for_each_entry_safe(pos, next, &tz->cooling_devices, node) {
> > @@ -1054,7 +1054,7 @@ EXPORT_SYMBOL(thermal_cooling_device_unr
> > static void thermal_zone_trip_update(struct thermal_zone_device *tz,
> > int trip, long temp)
> > {
> > - struct thermal_cooling_device_instance *instance;
> > + struct thermal_instance *instance;
> > struct thermal_cooling_device *cdev = NULL;
> > unsigned long cur_state, max_state;
> > long trip_temp;
> >
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-06-12 14:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1339384821.1492.161.camel@rui.sh.intel.com>
2012-06-11 6:21 ` [RFC PATCH 7/12] thermal: rename structure thermal_cooling_device_instance to thermal_instance Zhang Rui
2012-06-12 14:39 ` Eduardo Valentin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.