[PATCH] arch: arm: mach-socfpga: To notify SDM when SPL pass control to U-Boot

[Chee, Tien Fong]

> -----Original Message-----
> From: Lim, Jit Loon <jit.loon.lim at intel.com>
> Sent: Tuesday, 30 August, 2022 11:52 PM
> To: u-boot at lists.denx.de
> Cc: Jagan Teki <jagan at amarulasolutions.com>; Vignesh R <vigneshr at ti.com>;
> Vasut, Marek <marex at denx.de>; Simon <simon.k.r.goldschmidt at gmail.com>;
> Chee, Tien Fong <tien.fong.chee at intel.com>; Hea, Kok Kiang
> <kok.kiang.hea at intel.com>; Lim, Elly Siew Chin <elly.siew.chin.lim at intel.com>;
> Kho, Sin Hui <sin.hui.kho at intel.com>; Lokanathan, Raaj
> <raaj.lokanathan at intel.com>; Maniyam, Dinesh <dinesh.maniyam at intel.com>;
> Ng, Boon Khai <boon.khai.ng at intel.com>; Yuslaimi, Alif Zakuan
> <alif.zakuan.yuslaimi at intel.com>; Chong, Teik Heng
> <teik.heng.chong at intel.com>; Zamri, Muhammad Hazim Izzat
> <muhammad.hazim.izzat.zamri at intel.com>; Tang, Sieu Mun
> <sieu.mun.tang at intel.com>; Lim, Jit Loon <jit.loon.lim at intel.com>; Chin Liang
> See <chin.liang.see at intel.com>
> Subject: [PATCH] arch: arm: mach-socfpga: To notify SDM when SPL pass
> control to U-Boot
> 
> From: Chin Liang See <chin.liang.see at intel.com>
> 
> Prior SPL pass control to U-Boot, SPL will send a mailbox command
> "HPS_STAGE_NOTIFY" to notify Secure Device Manager (SDM) on HPS SW
> transition.
> 
> The purpose is for debug as user can query SDM on HPS error details when HPS
> enters a warm reset due to error such as watchdog.
> 
> Signed-off-by: Jit Loon Lim <jit.loon.lim at intel.com>
> ---
>  arch/arm/mach-socfpga/spl_s10.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/mach-socfpga/spl_s10.c b/arch/arm/mach-
> socfpga/spl_s10.c index c20e87cdbe..9bfd1a0f6a 100644
> --- a/arch/arm/mach-socfpga/spl_s10.c
> +++ b/arch/arm/mach-socfpga/spl_s10.c
> @@ -92,3 +92,9 @@ void board_init_f(ulong dummy)
>  	mbox_qspi_open();
>  #endif
>  }
> +
> +/* board specific function prior loading SSBL / U-Boot */ void
> +spl_board_prepare_for_boot(void) {
> +	mbox_hps_stage_notify(HPS_EXECUTION_STATE_SSBL);
> +}

You should put this notify into spl_perform_fixups(), so this can used by both boot flows:
SPL -> U-Boot and SPL -> ATF -> U-Boot.

Regards
Tien Fong