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

Commit 75f2a219 authored by Hakan Berg's avatar Hakan Berg Committed by Lee Jones
Browse files

ab8500-fg: Adjust for RF bursts voltage drops 



Changed conditions for restarting low battery measurements counter
and adjusted the interval between measurements to avoid RF burst
induced voltage drops, and to shorten time to decide to shut down.

Signed-off-by: default avatarHakan Berg <hakan.berg@stericsson.com>
Signed-off-by: default avatarMartin Bergstrom <martin.bergstrom@stericsson.com>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Reviewed-by: default avatarJonas ABERG <jonas.aberg@stericsson.com>
Reviewed-by: default avatarMarcus COOPER <marcus.xm.cooper@stericsson.com>
parent ff38090a

drivers/power/ab8500_fg.c

+27 −14
Original line number Diff line number Diff line
@@ -43,7 +43,7 @@ 


#define NBR_AVG_SAMPLES			20



#define LOW_BAT_CHECK_INTERVAL		(2 * HZ)

#define LOW_BAT_CHECK_INTERVAL		(HZ / 16) /* 62.5 ms */



#define VALID_CAPACITY_SEC		(45 * 60) /* 45 minutes */

#define BATT_OK_MIN			2360 /* mV */
@@ -169,6 +169,7 @@ struct inst_curr_result_list { 
 * @recovery_cnt:	Counter for recovery mode

 * @high_curr_cnt:	Counter for high current mode

 * @init_cnt:		Counter for init mode

 * @low_bat_cnt		Counter for number of consecutive low battery measures

 * @nbr_cceoc_irq_cnt	Counter for number of CCEOC irqs received since enabled

 * @recovery_needed:	Indicate if recovery is needed

 * @high_curr_mode:	Indicate if we're in high current mode
@@ -210,6 +211,7 @@ struct ab8500_fg { 
	int recovery_cnt;

	int high_curr_cnt;

	int init_cnt;

	int low_bat_cnt;

	int nbr_cceoc_irq_cnt;

	bool recovery_needed;

	bool high_curr_mode;
@@ -1879,25 +1881,29 @@ static void ab8500_fg_low_bat_work(struct work_struct *work) 


	/* Check if LOW_BAT still fulfilled */

	if (vbat < di->bm->fg_params->lowbat_threshold) {

		/* Is it time to shut down? */

		if (di->low_bat_cnt < 1) {

			di->flags.low_bat = true;

		dev_warn(di->dev, "Battery voltage still LOW\n");



			dev_warn(di->dev, "Shut down pending...\n");

		} else {

			/*

		 * We need to re-schedule this check to be able to detect

		 * if the voltage increases again during charging

			* Else we need to re-schedule this check to be able to detect

			* if the voltage increases again during charging or

			* due to decreasing load.

			*/

			di->low_bat_cnt--;

			dev_warn(di->dev, "Battery voltage still LOW\n");

			queue_delayed_work(di->fg_wq, &di->fg_low_bat_work,

				round_jiffies(LOW_BAT_CHECK_INTERVAL));

		}

	} else {

		di->flags.low_bat = false;

		di->flags.low_bat_delay = false;

		di->low_bat_cnt = 10;

		dev_warn(di->dev, "Battery voltage OK again\n");

	}



	/* This is needed to dispatch LOW_BAT */

	ab8500_fg_check_capacity_limits(di, false);



	/* Set this flag to check if LOW_BAT IRQ still occurs */

	di->flags.low_bat_delay = false;

}



/**
@@ -2056,6 +2062,7 @@ static irqreturn_t ab8500_fg_lowbatf_handler(int irq, void *_di) 
{

	struct ab8500_fg *di = _di;



	/* Initiate handling in ab8500_fg_low_bat_work() if not already initiated. */

	if (!di->flags.low_bat_delay) {

		dev_warn(di->dev, "Battery voltage is below LOW threshold\n");

		di->flags.low_bat_delay = true;
@@ -2698,6 +2705,12 @@ static int ab8500_fg_probe(struct platform_device *pdev) 
	INIT_DEFERRABLE_WORK(&di->fg_check_hw_failure_work,

		ab8500_fg_check_hw_failure_work);



	/* Reset battery low voltage flag */

	di->flags.low_bat = false;



	/* Initialize low battery counter */

	di->low_bat_cnt = 10;



	/* Initialize OVV, and other registers */

	ret = ab8500_fg_init_hw_registers(di);

	if (ret) {