[PATCH] mxs: power: Change sequence of enabling DCDC switches

[Jaehoon Chung]

On 10/13/21 10:40 PM, Lukasz Majewski wrote:
> The imx28 uses following voltage supplies hierarchy:
> 
> VDD_5V (VDD_BAT) -> VDDIO -> VDDA -> VDDMEM
> 			     \-----> VDDD
> 
> One shall first enable DCDC on the parent source (VDDIO) and then
> follow with its children.
> 
> Signed-off-by: Lukasz Majewski <lukma at denx.de>

Reviewed-by: Jaehoon Chung <jh80.chung at samsung.com>

Best Regards,
Jaehoon Chung

> ---
>  arch/arm/cpu/arm926ejs/mxs/spl_power_init.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c b/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c
> index 35ea71a5ba8..c33170f06d3 100644
> --- a/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c
> +++ b/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c
> @@ -627,11 +627,11 @@ static void mxs_power_enable_4p2(void)
>  
>  	mxs_power_init_dcdc_4p2_source();
>  
> -	writel(vdddctrl, &power_regs->hw_power_vdddctrl);
> +	writel(vddioctrl, &power_regs->hw_power_vddioctrl);
>  	early_delay(20);
>  	writel(vddactrl, &power_regs->hw_power_vddactrl);
>  	early_delay(20);
> -	writel(vddioctrl, &power_regs->hw_power_vddioctrl);
> +	writel(vdddctrl, &power_regs->hw_power_vdddctrl);
>  
>  	/*
>  	 * Check if FET is enabled on either powerout and if so,
>