quoted

quoted

And modify thermal_zone_device_register to Struct

thermal_zone_device

quoted

quoted

*thermal_zone_device_register(const char *type, struct
thermal_zone_params *params);

The first 3 fields are necessary for registering a zone, the
thermal_bind_params can either be filled by platform thermal driver,
or be NULL and filled by thermal layer later, when user invokes
thermal_zone_bind_cooling_devices.

In this way, we do not need this API at all.

We can do it either ways. In this case, we need to modify all the
tzd_register calls. If we are Ok doing that, I am happy to change.

Just that we are adding one more to the already existing 7 args :-)

No, we are trying to reducing the args by moving them into
thermal_zone_params.

quoted

quoted

quoted

 static int get_idr(struct idr *idr, struct mutex *lock, int *id)

{

quoted

quoted

 	int err;

diff --git a/include/linux/thermal.h b/include/linux/thermal.h

index

quoted

quoted

32af124..b644b8e 100644

--- a/include/linux/thermal.h
+++ b/include/linux/thermal.h

@@ -67,6 +67,12 @@ enum thermal_trend {
 	THERMAL_TREND_DROPPING, /* temperature is dropping */  };

+enum thermal_throttle_policy {
+	THERMAL_USER_SPACE,
+	THERMAL_FAIR_SHARE,
+	THERMAL_STEP_WISE,
+};
+
 /* Events supported by Thermal Netlink */  enum events {
 	THERMAL_AUX0,

@@ -162,6 +168,37 @@ 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;
+};
+
+/* Structure that holds binding parameters for a zone */ struct
+thermal_bind_params {
+	struct thermal_cooling_device *cdev;
+
+	/*
+	 * 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 weight;
+
+	/*
+	 * 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;
+	int (*match) (struct thermal_zone_device *tz,
+			struct thermal_cooling_device *cdev); };

You should start a new line here.

Again, not sure what you meant here. The new line is already there.

		struct thermal_cooling_device *cdev);
	 };

I'm not sure what it is in your original patch, but I see something like
"struct thermal_cooling_device *cdev); };"
May be this is because I'm using outlook?

Looks like it is !!

Thanks,
Durga