sandbox: "Reset Status: WARM Reset Status: COLD"

Simon Glass sjg at chromium.org
Thu Dec 29 23:38:31 CET 2022


Hi Heinrich,

On Thu, 29 Dec 2022 at 03:18, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> Hello Simon,
>
> the sandbox prints the following line when started:
> "Reset Status: WARM Reset Status: COLD"

This seems to be when you use the -T flag.

>
> This does not make much sense. Either the last reset was WARM or COLD
> but not both.
>

Indeed!

> sandbox_sysreset_get_status() and sandbox_warm_sysreset_get_status()
> should determine the reset status before printing anything incorrect.
>
> After executing the reset command
>
>     state->last_sysreset == 4
>
> This cannot be correct. It seems to be due to:
>
> arch/sandbox/cpu/state.c:365:   state->last_sysreset = SYSRESET_COUNT;
>
> state_reset_for_test() should only be called in tests.

It needs to be called at init also. But actually the name is
confusing. How about state_reset_common() or something like that?

For testing we have two sysreset devices. Perhaps print_resetinfo()
should only print the first?

Regards,
Simon


More information about the U-Boot mailing list