[PATCH v1 1/2] imx: scu_api: add implementation of sc_pm_reset_reason

Add implementation of sc_pm_reset_reason() call for obtaining reset reason.
Signed-off-by: Igor Opaniuk igor.opaniuk@foundries.io ---
drivers/misc/imx8/scu_api.c | 22 ++++++++++++++++++++++ include/firmware/imx/sci/sci.h | 6 ++++++ 2 files changed, 28 insertions(+)
diff --git a/drivers/misc/imx8/scu_api.c b/drivers/misc/imx8/scu_api.c index 3e3002ba6df..6e2c678e614 100644 --- a/drivers/misc/imx8/scu_api.c +++ b/drivers/misc/imx8/scu_api.c @@ -882,6 +882,28 @@ void sc_pm_reboot(sc_ipc_t ipc, sc_pm_reset_type_t type) misc_call(dev, SC_TRUE, &msg, size, &msg, size); }
+int sc_pm_reset_reason(sc_ipc_t ipc, sc_pm_reset_reason_t *reason) +{ + struct udevice *dev = gd->arch.scu_dev; + struct sc_rpc_msg_s msg; + int size = sizeof(struct sc_rpc_msg_s); + int ret; + + RPC_VER(&msg) = SC_RPC_VERSION; + RPC_SVC(&msg) = (u8)SC_RPC_SVC_PM; + RPC_FUNC(&msg) = (u8)PM_FUNC_RESET_REASON; + RPC_SIZE(&msg) = 1U; + + ret = misc_call(dev, SC_FALSE, &msg, size, &msg, size); + if (ret) + printf("%s: res:%d\n", __func__, RPC_U8(&msg, 0U)); + + if (reason) + *reason = RPC_U8(&msg, 0U); + + return ret; +} + int sc_pm_get_resource_power_mode(sc_ipc_t ipc, sc_rsrc_t resource, sc_pm_power_mode_t *mode) { diff --git a/include/firmware/imx/sci/sci.h b/include/firmware/imx/sci/sci.h index f832982b3de..7d8499f070a 100644 --- a/include/firmware/imx/sci/sci.h +++ b/include/firmware/imx/sci/sci.h @@ -75,6 +75,7 @@ int sc_pm_set_clock_parent(sc_ipc_t ipc, sc_rsrc_t resource, sc_pm_clk_t clk, int sc_pm_cpu_start(sc_ipc_t ipc, sc_rsrc_t resource, sc_bool_t enable, sc_faddr_t address); void sc_pm_reboot(sc_ipc_t ipc, sc_pm_reset_type_t type); +int sc_pm_reset_reason(sc_ipc_t ipc, sc_pm_reset_reason_t *reason); sc_bool_t sc_pm_is_partition_started(sc_ipc_t ipc, sc_rm_pt_t pt); int sc_pm_resource_reset(sc_ipc_t ipc, sc_rsrc_t resource);
@@ -385,6 +386,11 @@ static inline void sc_pm_reboot(sc_ipc_t ipc, sc_pm_reset_type_t type) { }
+static inline int sc_pm_reset_reason(sc_ipc_t ipc, sc_pm_reset_reason_t *reason) +{ + return -EOPNOTSUPP; +} + static inline int sc_seco_v2x_build_info(sc_ipc_t ipc, u32 *version, u32 *commit) { return -EOPNOTSUPP;

Add support for printing reset cause during boot.
Signed-off-by: Igor Opaniuk igor.opaniuk@foundries.io ---
arch/arm/mach-imx/imx8/cpu.c | 41 ++++++++++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+)
diff --git a/arch/arm/mach-imx/imx8/cpu.c b/arch/arm/mach-imx/imx8/cpu.c index 39ac0bc4140..0b91e448a5d 100644 --- a/arch/arm/mach-imx/imx8/cpu.c +++ b/arch/arm/mach-imx/imx8/cpu.c @@ -45,6 +45,45 @@ struct pass_over_info_t *get_pass_over_info(void) return p; }
+static char *get_reset_cause(void) +{ + sc_pm_reset_reason_t reason; + + if (sc_pm_reset_reason(-1, &reason) != SC_ERR_NONE) + return "Unknown reset"; + + switch (reason) { + case SC_PM_RESET_REASON_POR: + return "POR"; + case SC_PM_RESET_REASON_JTAG: + return "JTAG reset "; + case SC_PM_RESET_REASON_SW: + return "Software reset"; + case SC_PM_RESET_REASON_WDOG: + return "Watchdog reset"; + case SC_PM_RESET_REASON_LOCKUP: + return "SCU lockup reset"; + case SC_PM_RESET_REASON_SNVS: + return "SNVS reset"; + case SC_PM_RESET_REASON_TEMP: + return "Temp panic reset"; + case SC_PM_RESET_REASON_MSI: + return "MSI reset"; + case SC_PM_RESET_REASON_UECC: + return "ECC reset"; + case SC_PM_RESET_REASON_SCFW_WDOG: + return "SCFW watchdog reset"; + case SC_PM_RESET_REASON_ROM_WDOG: + return "SCU ROM watchdog reset"; + case SC_PM_RESET_REASON_SECO: + return "SECO reset"; + case SC_PM_RESET_REASON_SCFW_FAULT: + return "SCFW fault reset"; + default: + return "Unknown reset"; + } +} + int arch_cpu_init(void) { #if defined(CONFIG_SPL_BUILD) && defined(CONFIG_SPL_RECOVER_DATA_SECTION) @@ -322,6 +361,8 @@ int print_bootinfo(void) break; }
+ printf("Reset cause: %s\n", get_reset_cause()); + return 0; }

On Wed, Jan 31, 2024 at 9:49 AM Igor Opaniuk igor.opaniuk@foundries.io wrote:
Add support for printing reset cause during boot.
Signed-off-by: Igor Opaniuk igor.opaniuk@foundries.io
Applied, both, thanks.
participants (2)
-
Fabio Estevam
-
Igor Opaniuk