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

Commit a81e7206 authored by Harry Yang's avatar Harry Yang
Browse files

qcom-charger: smb-lib: fix battery health status 



Battery health shows 'good' in the ranges of both cold and
overheat temperatures. This change fixes a bug in its
get_prop function.

CRs-Fixed: 1040848
Change-Id: If6e6f60170cfb4769456de4c4428831a29489da2
Signed-off-by: default avatarHarry Yang <harryy@codeaurora.org>
parent 400520a6

drivers/power/qcom-charger/smb-lib.c

+5 −11
Original line number Diff line number Diff line
@@ -783,22 +783,16 @@ int smblib_get_prop_batt_health(struct smb_charger *chg, 
		goto done;

	}



	switch (stat & BAT_TEMP_STATUS_MASK) {

	case BAT_TEMP_STATUS_TOO_COLD_BIT:

	if (stat & BAT_TEMP_STATUS_TOO_COLD_BIT)

		val->intval = POWER_SUPPLY_HEALTH_COLD;

		break;

	case BAT_TEMP_STATUS_TOO_HOT_BIT:

	else if (stat & BAT_TEMP_STATUS_TOO_HOT_BIT)

		val->intval = POWER_SUPPLY_HEALTH_OVERHEAT;

		break;

	case BAT_TEMP_STATUS_COLD_SOFT_LIMIT_BIT:

	else if (stat & BAT_TEMP_STATUS_COLD_SOFT_LIMIT_BIT)

		val->intval = POWER_SUPPLY_HEALTH_COOL;

		break;

	case BAT_TEMP_STATUS_HOT_SOFT_LIMIT_BIT:

	else if (stat & BAT_TEMP_STATUS_HOT_SOFT_LIMIT_BIT)

		val->intval = POWER_SUPPLY_HEALTH_WARM;

		break;

	default:

	else

		val->intval = POWER_SUPPLY_HEALTH_GOOD;

	}



done:

	return rc;