On 5.6.2017 17:43, Tom Rini wrote:
> From: Michal Simek <michal.si...@xilinx.com>
> 
> Add support for calling poweroff in case of psci is wired.
> Based on the same solution as is used for reset.
> 
> Signed-off-by: Michal Simek <michal.si...@xilinx.com>
> Reviewed-by: Simon Glass <s...@chromium.org>
> [trini: Move all logic in to fwcall.c as other ARMs implement poweroff
> via PMIC]
> Signed-off-by: Tom Rini <tr...@konsulko.com>
> ---
>  arch/arm/cpu/armv8/fwcall.c | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/arch/arm/cpu/armv8/fwcall.c b/arch/arm/cpu/armv8/fwcall.c
> index 7dfd27002916..c2202675366f 100644
> --- a/arch/arm/cpu/armv8/fwcall.c
> +++ b/arch/arm/cpu/armv8/fwcall.c
> @@ -1,5 +1,6 @@
>  /**
>   * (C) Copyright 2014, Cavium Inc.
> + * (C) Copyright 2017, Xilinx Inc.
>   *
>   * SPDX-License-Identifier:  GPL-2.0+
>  **/
> @@ -114,6 +115,22 @@ void __noreturn __efi_runtime psci_system_off(void)
>               ;
>  }
>  
> +#ifdef CONFIG_CMD_POWEROFF
> +int do_poweroff(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> +{
> +     puts("poweroff ...\n");
> +
> +     udelay(50000); /* wait 50 ms */
> +
> +     disable_interrupts();
> +
> +     psci_system_off();
> +
> +     /*NOTREACHED*/
> +     return 0;
> +}
> +#endif
> +
>  #ifdef CONFIG_PSCI_RESET
>  void reset_misc(void)
>  {
> 

Looks good to me.
BTW: there was a question from Simon if we should use puts or printf.

Thanks,
Michal






_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to