Hi Heinrich,
On Thu, 29 Dec 2022 at 03:18, Heinrich Schuchardt 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
