[PATCH v5 3/4] test/py: Handle expected reboot while booting sandbox

[Heinrich Schuchardt]

On 2/26/22 19:37, Simon Glass wrote:
> Hi Masami,
>
> On Tue, 15 Feb 2022 at 23:16, Masami Hiramatsu
> <masami.hiramatsu at linaro.org> wrote:
>>
>> Add expected_reset optional argument to ConsoleBase::ensure_spawned(),
>> ConsoleBase::restart_uboot() and ConsoleSandbox::restart_uboot_with_flags()
>> so that it can handle a reset while the 1st boot process after main
>> boot logo before prompt correctly.
>>
>> Signed-off-by: Masami Hiramatsu <masami.hiramatsu at linaro.org>
>> ---
>>   Changes in v5:
>>    - Rename parameter to expect_reset and update the description to clarify
>>      the reset will happen between main boot and the command prompt.
>> ---
>>   test/py/u_boot_console_base.py    |   48 ++++++++++++++++++++++---------------
>>   test/py/u_boot_console_sandbox.py |    7 ++++-
>>   2 files changed, 33 insertions(+), 22 deletions(-)
>>
>
> Didn't I already comment on this patch? Why did it come back?

Dear Simon,

The discussion is in
https://patchwork.ozlabs.org/project/uboot/patch/164491595065.536855.9457820061065514578.stgit@localhost/

You suggested: "We have a means to avoid actually doing the reset, see
the reset driver."

We need a real reset on the sandbox and no fake reset as already said in
the referenced thread.

Best regards

Heinrich

>
> Regards,
> Simon