Skip to content
Snippets Groups Projects
Commit 3cc293e2 authored by Rask Ingemann Lambertsen's avatar Rask Ingemann Lambertsen Committed by Jaehoon Chung
Browse files

sunxi: power: axp809.c: Fix aldo1-2 being disabled for mvolt != 0


The execution flow is currently like this for aldo_num == 1 or 2:

int axp_set_aldo(int aldo_num, unsigned int mvolt)
{
...
	if (mvolt == 0)
		return pmic_bus_clrbits(AXP809_OUTPUT_CTRL1,
				AXP809_OUTPUT_CTRL1_ALDO1_EN << (aldo_num - 1));
...
	return pmic_bus_clrbits(AXP809_OUTPUT_CTRL1,
 				AXP809_OUTPUT_CTRL1_ALDO1_EN << (aldo_num - 1));
 }

I.e. aldo1 and aldo2 will always be disabled. This patch fixes it by
setting (rather than clearing) the enable bit when mvolt != 0.

Signed-off-by: default avatarRask Ingemann Lambertsen <rask@formelder.dk>
Fixes: 795857df ("sunxi: power: add AXP809 support")
parent 0574f786
No related branches found
No related tags found
No related merge requests found
...@@ -143,7 +143,7 @@ int axp_set_aldo(int aldo_num, unsigned int mvolt) ...@@ -143,7 +143,7 @@ int axp_set_aldo(int aldo_num, unsigned int mvolt)
if (aldo_num == 3) if (aldo_num == 3)
return pmic_bus_setbits(AXP809_OUTPUT_CTRL2, return pmic_bus_setbits(AXP809_OUTPUT_CTRL2,
AXP809_OUTPUT_CTRL2_ALDO3_EN); AXP809_OUTPUT_CTRL2_ALDO3_EN);
return pmic_bus_clrbits(AXP809_OUTPUT_CTRL1, return pmic_bus_setbits(AXP809_OUTPUT_CTRL1,
AXP809_OUTPUT_CTRL1_ALDO1_EN << (aldo_num - 1)); AXP809_OUTPUT_CTRL1_ALDO1_EN << (aldo_num - 1));
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment