Dear Akshay Saraswat,

On 26/05/14 22:51, Akshay Saraswat wrote:
> From: Doug Anderson <diand...@chromium.org>
> 
> Setting ps_hold ought to be one of the first things we do when we
> first boot up. If we wait until the main u-boot runs we won't set it
> in time and the PMIC may power us back off.
> 
> Moving ps_hold setup into the generic power_init() which
> should contain code that's currently duplicated in the
> board_power_init() of several boards.
> 
> Signed-off-by: Doug Anderson <diand...@chromium.org>
> Signed-off-by: Akshay Saraswat <aksha...@samsung.com>
> Acked-by: Simon Glass <s...@chromium.org>
> ---
> Changes since v1:
>       - Added "Acked-by".
> 
>  arch/arm/cpu/armv7/exynos/lowlevel_init.c |    6 +++++-
>  arch/arm/cpu/armv7/exynos/power.c         |   14 ++++++++++++++
>  arch/arm/include/asm/arch-exynos/power.h  |    8 ++++++++
>  3 files changed, 27 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/cpu/armv7/exynos/lowlevel_init.c 
> b/arch/arm/cpu/armv7/exynos/lowlevel_init.c
> index 11fe5b8..ed966bc 100644
> --- a/arch/arm/cpu/armv7/exynos/lowlevel_init.c
> +++ b/arch/arm/cpu/armv7/exynos/lowlevel_init.c
> @@ -39,6 +39,7 @@ enum {
>       DO_CLOCKS       = 1 << 1,
>       DO_MEM_RESET    = 1 << 2,
>       DO_UART         = 1 << 3,
> +     DO_POWER        = 1 << 4,
>  };
>  
>  int do_lowlevel_init(void)
> @@ -60,9 +61,12 @@ int do_lowlevel_init(void)
>               break;
>       default:
>               /* This is a normal boot (not a wake from sleep) */
> -             actions = DO_CLOCKS | DO_MEM_RESET;
> +             actions = DO_CLOCKS | DO_MEM_RESET | DO_POWER;
>       }
>  
> +     if (actions & DO_POWER)
> +             power_init();

I think we don't have to make new function.
Please call set_ps_hold_ctrl directly.

Thanks,
Minkyu Kang.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to