Commit 1174ee1f authored Jan 29, 2015 by Abhijeet Dharmapurikar

power: qpnp-smbcharger: fix the argument order while using is_between()

The is_between() function expects the value to be the third argument, the
first and the second arguments define the range.

Currently the value to be checked is passed as the first argument which
leads to selecting wrong voltage thresholds for Wipower dynamic
impedance limiting feature which in turn causes an ADC interrupt storm.

Fix this by passing arguments in the right order.

Change-Id: I525615ca2a2f4a4e3a0db15eb3e15ec965853552
Signed-off-by: Abhijeet Dharmapurikar <adharmap@codeaurora.org>

parent c84f1f0f

drivers/power/qpnp-smbcharger.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -1842,8 +1842,8 @@ static struct ilim_entry smbchg_wipower_find_entry(struct smbchg_chip chip,
		struct ilim_entry *ret = &(chip->wipower_default.entries[0]);

		for (i = 0; i < map->num; i++) {
		if (is_between(uv,
		map->entries[i].vmin_uv, map->entries[i].vmax_uv))
		if (is_between(map->entries[i].vmin_uv, map->entries[i].vmax_uv,
		uv))
		ret = &map->entries[i];
		}
		return ret;