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

Commit 7ffa9ad3 authored by Kiran Gunda's avatar Kiran Gunda Committed by Gerrit - the friendly Code Review server
Browse files

power: smb5-lib: Populate the soft jeita thresholds 



Currently if the soft jeita and soft jeita hysteresis thresholds
are not configured in the device tree file, the corresponding
jeita registers are programmed to '0', which is not expected
behaviour. Fix it by populating the soft jeita thresholds with
the corresponding register values in the absence of soft jeita
and soft jeita hysteresis properties in device tree file.

Change-Id: I985121ec1b8897549b3256a5de1cdf2effee5921
Signed-off-by: default avatarKiran Gunda <kgunda@codeaurora.org>
parent e710aa3f

drivers/power/supply/qcom/smb5-lib.c

+31 −0
Original line number Diff line number Diff line
@@ -7458,6 +7458,7 @@ static void apsd_timer_cb(struct timer_list *tm) 
	chg->apsd_ext_timeout = true;

}



#define SOFT_JEITA_HYSTERESIS_OFFSET	0x200

static void jeita_update_work(struct work_struct *work)

{

	struct smb_charger *chg = container_of(work, struct smb_charger,
@@ -7467,6 +7468,8 @@ static void jeita_update_work(struct work_struct *work) 
	union power_supply_propval val;

	int rc, tmp[2], max_fcc_ma, max_fv_uv;

	u32 jeita_hard_thresholds[2];

	u16 addr;

	u8 buff[2];



	batt_node = of_find_node_by_name(node, "qcom,battery-data");

	if (!batt_node) {
@@ -7529,6 +7532,34 @@ static void jeita_update_work(struct work_struct *work) 
					rc);

			goto out;

		}

	} else {

		/* Populate the jeita-soft-thresholds */

		addr = CHGR_JEITA_THRESHOLD_BASE_REG(JEITA_SOFT);

		rc = smblib_batch_read(chg, addr, buff, 2);

		if (rc < 0) {

			pr_err("failed to read 0x%4X, rc=%d\n", addr, rc);

			goto out;

		}



		chg->jeita_soft_thlds[1] = buff[1] | buff[0] << 8;



		rc = smblib_batch_read(chg, addr + 2, buff, 2);

		if (rc < 0) {

			pr_err("failed to read 0x%4X, rc=%d\n", addr + 2, rc);

			goto out;

		}



		chg->jeita_soft_thlds[0] = buff[1] | buff[0] << 8;



		/*

		 * Update the soft jeita hysteresis 2 DegC less for warm and

		 * 2 DegC more for cool than the soft jeita thresholds to avoid

		 * overwriting the registers with invalid values.

		 */

		chg->jeita_soft_hys_thlds[0] =

			chg->jeita_soft_thlds[0] - SOFT_JEITA_HYSTERESIS_OFFSET;

		chg->jeita_soft_hys_thlds[1] =

			chg->jeita_soft_thlds[1] + SOFT_JEITA_HYSTERESIS_OFFSET;

	}



	chg->jeita_soft_fcc[0] = chg->jeita_soft_fcc[1] = -EINVAL;