Hi Denis,
On 2026-05-22T01:23:09, None <[email protected]> wrote:
> reset: Print reset type on diagnostic console
>
> Add a diagnostic console trace indicating the reset type.
>
> Signed-off-by: Denis Mukhin <[email protected]>
>
> drivers/sysreset/sysreset-uclass.c | 18 +++++++++++++++++-
> 1 file changed, 17 insertions(+), 1 deletion(-)
> diff --git a/drivers/sysreset/sysreset-uclass.c
> b/drivers/sysreset/sysreset-uclass.c
> @@ -22,6 +22,22 @@
> +static const char *get_reset_type_str(enum sysreset_t reset_type)
> +{
> + switch (reset_type) {
> + case SYSRESET_WARM:
> + return 'warm';
> + case SYSRESET_COLD:
> + return 'cold';
> + case SYSRESET_POWER:
> + return 'PMIC';
> + case SYSRESET_POWER_OFF:
> + return "power off";
> + default:
> + return 'unknown';
> + }
> +}
PMIC is an odd label for SYSRESET_POWER - the enum doc describes it as
"remove and restore power", so 'power' reads more naturally and lines
up with the enum name. What do you think?
Regards,
Simon
