From: Anton Vorontsov <anton.vorontsov@linaro.org>
Date: Tue, 31 Jul 2012 11:59:42 +0000 (-0700)
Subject: Merge with upstream to accommodate with thermal changes
X-Git-Tag: upstream/snapshot3+hdmi~6903^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e6db06a53b1dcf4e9da4aba143e2eb4d63418abb;hp=-c;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

Merge ... upstream to accommodate with thermal changes

This merge is performed to take commit c56f5c0342dfee11a1 ("Thermal: Make
Thermal trip points writeable") out of Linus' tree and then fixup power
supply class. This is needed since thermal stuff added a new argument:

  CC      drivers/power/power_supply_core.o
drivers/power/power_supply_core.c: In function ‘psy_register_thermal’:
drivers/power/power_supply_core.c:204:6: warning: passing argument 3 of ‘thermal_zone_device_register’ makes integer from pointer without a cast [enabled by default]
include/linux/thermal.h:154:29: note: expected ‘int’ but argument is of type ‘struct power_supply *’
drivers/power/power_supply_core.c:204:6: error: too few arguments to function ‘thermal_zone_device_register’
include/linux/thermal.h:154:29: note: declared here
make[1]: *** [drivers/power/power_supply_core.o] Error 1
make: *** [drivers/power/] Error 2

Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
---

e6db06a53b1dcf4e9da4aba143e2eb4d63418abb
diff --combined drivers/power/power_supply_core.c
index ff990d2,6ad6127..08cc8a3
--- a/drivers/power/power_supply_core.c
+++ b/drivers/power/power_supply_core.c
@@@ -17,7 -17,6 +17,7 @@@
  #include <linux/device.h>
  #include <linux/err.h>
  #include <linux/power_supply.h>
 +#include <linux/thermal.h>
  #include "power_supply.h"
  
  /* exported for the APM Power driver, APM emulation */
@@@ -170,63 -169,6 +170,63 @@@ static void power_supply_dev_release(st
  	kfree(dev);
  }
  
 +#ifdef CONFIG_THERMAL
 +static int power_supply_read_temp(struct thermal_zone_device *tzd,
 +		unsigned long *temp)
 +{
 +	struct power_supply *psy;
 +	union power_supply_propval val;
 +	int ret;
 +
 +	WARN_ON(tzd == NULL);
 +	psy = tzd->devdata;
 +	ret = psy->get_property(psy, POWER_SUPPLY_PROP_TEMP, &val);
 +
 +	/* Convert tenths of degree Celsius to milli degree Celsius. */
 +	if (!ret)
 +		*temp = val.intval * 100;
 +
 +	return ret;
 +}
 +
 +static struct thermal_zone_device_ops psy_tzd_ops = {
 +	.get_temp = power_supply_read_temp,
 +};
 +
 +static int psy_register_thermal(struct power_supply *psy)
 +{
 +	int i;
 +
 +	/* Register battery zone device psy reports temperature */
 +	for (i = 0; i < psy->num_properties; i++) {
 +		if (psy->properties[i] == POWER_SUPPLY_PROP_TEMP) {
- 			psy->tzd = thermal_zone_device_register(psy->name, 0,
++			psy->tzd = thermal_zone_device_register(psy->name, 0, 0,
 +					psy, &psy_tzd_ops, 0, 0, 0, 0);
 +			if (IS_ERR(psy->tzd))
 +				return PTR_ERR(psy->tzd);
 +			break;
 +		}
 +	}
 +	return 0;
 +}
 +
 +static void psy_unregister_thermal(struct power_supply *psy)
 +{
 +	if (IS_ERR_OR_NULL(psy->tzd))
 +		return;
 +	thermal_zone_device_unregister(psy->tzd);
 +}
 +#else
 +static int psy_register_thermal(struct power_supply *psy)
 +{
 +	return 0;
 +}
 +
 +static void psy_unregister_thermal(struct power_supply *psy)
 +{
 +}
 +#endif
 +
  int power_supply_register(struct device *parent, struct power_supply *psy)
  {
  	struct device *dev;
@@@ -255,10 -197,6 +255,10 @@@
  	if (rc)
  		goto device_add_failed;
  
 +	rc = psy_register_thermal(psy);
 +	if (rc)
 +		goto register_thermal_failed;
 +
  	rc = power_supply_create_triggers(psy);
  	if (rc)
  		goto create_triggers_failed;
@@@ -268,8 -206,6 +268,8 @@@
  	goto success;
  
  create_triggers_failed:
 +	psy_unregister_thermal(psy);
 +register_thermal_failed:
  	device_del(dev);
  kobject_set_name_failed:
  device_add_failed:
@@@ -284,7 -220,6 +284,7 @@@ void power_supply_unregister(struct pow
  	cancel_work_sync(&psy->changed_work);
  	sysfs_remove_link(&psy->dev->kobj, "powers");
  	power_supply_remove_triggers(psy);
 +	psy_unregister_thermal(psy);
  	device_unregister(psy->dev);
  }
  EXPORT_SYMBOL_GPL(power_supply_unregister);