Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fd65ee5f authored by Chanwoo Choi's avatar Chanwoo Choi Committed by Anton Vorontsov
Browse files

charger-manager: Use replacement variable to check state of battery 



This patch remove unnecessary variable(cm->fullbatt_vchk_uV) by using
'desc->fullbatt_uV' field directly in fullbatt_handler() function
to check the state of battery.

Signed-off-by: default avatarChanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: default avatarMyungjoo Ham <myungjoo.ham@samsung.com>
Signed-off-by: default avatarKyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: default avatarAnton Vorontsov <anton.vorontsov@linaro.org>
parent dbb61fc7

drivers/power/charger-manager.c

+1 −1
Original line number Diff line number Diff line
@@ -415,7 +415,7 @@ static void fullbatt_vchk(struct work_struct *work) 
		return;

	}



	diff = cm->fullbatt_vchk_uV;

	diff = desc->fullbatt_uV;

	diff -= batt_uV;



	dev_dbg(cm->dev, "VBATT dropped %duV after full-batt.\n", diff);

include/linux/power/charger-manager.h

+0 −3
Original line number Diff line number Diff line
@@ -194,8 +194,6 @@ struct charger_desc { 
 * @charger_enabled: the state of charger

 * @fullbatt_vchk_jiffies_at:

 *	jiffies at the time full battery check will occur.

 * @fullbatt_vchk_uV: voltage in microvolt

 *	criteria for full battery

 * @fullbatt_vchk_work: work queue for full battery check

 * @emergency_stop:

 *	When setting true, stop charging
@@ -218,7 +216,6 @@ struct charger_manager { 
	bool charger_enabled;



	unsigned long fullbatt_vchk_jiffies_at;

	unsigned int fullbatt_vchk_uV;

	struct delayed_work fullbatt_vchk_work;



	int emergency_stop;