Skip to content

Commit

Permalink
ab8500-btemp: Defer btemp filtering while initialising
Browse files Browse the repository at this point in the history
Due to btemp filtering enabled during init, temp values reported
to charge algorithm driver started from 0. As a result,charge
algorithm was going into wrong state and charging was stopped.

Signed-off-by: Rupesh Kumar <rupesh.kumar@stericsson.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Reviewed-by: Marcus COOPER <marcus.xm.cooper@stericsson.com>
Reviewed-by: Martin SJOBLOM <martin.w.sjoblom@stericsson.com>
Reviewed-by: Philippe LANGLAIS <philippe.langlais@stericsson.com>
  • Loading branch information
Rupesh Kumar authored and Lee Jones committed Mar 7, 2013
1 parent db43e6c commit 2c4c40a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions drivers/power/ab8500_btemp.c
Original file line number Diff line number Diff line change
Expand Up @@ -609,7 +609,6 @@ static void ab8500_btemp_periodic_work(struct work_struct *work)
struct ab8500_btemp, btemp_periodic_work.work);

if (!di->initialized) {
di->initialized = true;
/* Identify the battery */
if (ab8500_btemp_id(di) < 0)
dev_warn(di->dev, "failed to identify the battery\n");
Expand All @@ -622,8 +621,9 @@ static void ab8500_btemp_periodic_work(struct work_struct *work)
* same temperature. Else only allow 1 degree change from previous
* reported value in the direction of the new measurement.
*/
if (bat_temp == di->prev_bat_temp || !di->initialized) {
if (di->bat_temp != di->prev_bat_temp || !di->initialized) {
if ((bat_temp == di->prev_bat_temp) || !di->initialized) {
if ((di->bat_temp != di->prev_bat_temp) || !di->initialized) {
di->initialized = true;
di->bat_temp = bat_temp;
power_supply_changed(&di->btemp_psy);
}
Expand Down

0 comments on commit 2c4c40a

Please sign in to comment.