From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhang Rui Subject: Re: [PATCH 04/13] Thermal: Add platform level information to thermal.h Date: Thu, 16 Aug 2012 14:16:30 +0800 Message-ID: <1345097790.1682.856.camel@rui.sh.intel.com> References: <1344516365-7230-1-git-send-email-durgadoss.r@intel.com> <1344516365-7230-5-git-send-email-durgadoss.r@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mga11.intel.com ([192.55.52.93]:27280 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753347Ab2HPGP0 (ORCPT ); Thu, 16 Aug 2012 02:15:26 -0400 In-Reply-To: <1344516365-7230-5-git-send-email-durgadoss.r@intel.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Durgadoss R Cc: lenb@kernel.org, rjw@sisk.pl, linux-acpi@vger.kernel.org, linux-pm@vger.kernel.org, eduardo.valentin@ti.com, amit.kachhap@linaro.org, wni@nvidia.com On =E5=9B=9B, 2012-08-09 at 18:15 +0530, Durgadoss R wrote: > This patch creates a structure to hold a thermal zone's > platform level info, and also defines an extern function to > retrieve zone parameters from thermal_sys.c. >=20 > Signed-off-by: Durgadoss R > --- > drivers/thermal/thermal_sys.c | 3 +++ > include/linux/thermal.h | 43 +++++++++++++++++++++++++++++++= ++++++++++ > 2 files changed, 46 insertions(+) >=20 > diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_= sys.c > index 998c16e..f043cd6 100644 > --- a/drivers/thermal/thermal_sys.c > +++ b/drivers/thermal/thermal_sys.c > @@ -49,6 +49,9 @@ static LIST_HEAD(thermal_tz_list); > static LIST_HEAD(thermal_cdev_list); > static DEFINE_MUTEX(thermal_list_lock); > =20 > +int (*get_platform_thermal_params)(struct thermal_zone_device *); > +EXPORT_SYMBOL(get_platform_thermal_params); > + > static int get_idr(struct idr *idr, struct mutex *lock, int *id) > { > int err; > diff --git a/include/linux/thermal.h b/include/linux/thermal.h > index 757a007..f9ce1e2 100644 > --- a/include/linux/thermal.h > +++ b/include/linux/thermal.h > @@ -33,6 +33,8 @@ > #define THERMAL_MAX_TRIPS 12 > #define THERMAL_NAME_LENGTH 20 > =20 > +#define MAX_COOLING_DEVS THERMAL_MAX_TRIPS > + > /* Initial state of a cooling device during binding */ > #define THERMAL_NO_TARGET -1UL > =20 > @@ -49,6 +51,11 @@ > #define THERMAL_GENL_VERSION 0x01 > #define THERMAL_GENL_MCAST_GROUP_NAME "thermal_mc_group" > =20 > +/* Thermal policies */ > +#define THERMAL_USER_SPACE 0 > +#define THERMAL_FAIR_SHARE 1 > +#define THERMAL_STEP_WISE 2 > + I'd prefer using enum here, for thermal policies. > struct thermal_zone_device; > struct thermal_cooling_device; > =20 > @@ -165,6 +172,7 @@ struct thermal_zone_device { > struct mutex lock; /* protect thermal_instances list */ > struct list_head node; > struct delayed_work poll_queue; > + struct thermal_zone_params *tzp; > }; > =20 > /* > @@ -187,6 +195,31 @@ struct thermal_instance { > struct list_head cdev_node; /* node in cdev->thermal_instances */ > }; > =20 > +/* Platform level parameters associated with a thermal zone */ > +struct thermal_zone_params { > + char *thermal_zone_name; > + int throttle_policy; > + > + /* Number of cooling devices associated with this thermal zone */ > + int num_cdevs; > + char *cdevs_name[MAX_COOLING_DEVS]; > + > + /* > + * This is a measure of 'how effectively these devices can > + * cool 'this' thermal zone. The shall be determined by platform > + * characterization. This is on a 'percentage' scale. > + * See Documentation/thermal/sysfs-api.txt for more information. > + */ > + int weights[MAX_COOLING_DEVS]; > + > + /* > + * This is a bit mask that gives the binding relation between this > + * thermal zone and cdev, for a particular trip point. > + * See Documentation/thermal/sysfs-api.txt for more information. > + */ > + int trip_mask[MAX_COOLING_DEVS]; > +}; > + > struct thermal_genl_event { > u32 orig; > enum events event; > @@ -213,6 +246,16 @@ struct thermal_cooling_device *get_cdev_by_name(= const char *); > struct thermal_instance *get_thermal_instance(struct thermal_zone_de= vice *, > struct thermal_cooling_device *, int); > =20 > +/* > + * The platform layer shall define a 'function' that provides the > + * parameters for all thermal zones in the platform. This pointer > + * should point to that 'function'. > + * > + * In thermal_zone_device_register() we update the parameters > + * for the particular thermal zone. > + */ > +extern int (*get_platform_thermal_params)(struct thermal_zone_device= *); > + > #ifdef CONFIG_NET > extern int thermal_generate_netlink_event(u32 orig, enum events even= t); > #else -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html