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

[Simon Glass]

Hi Heinrich,

On Sun, 27 Feb 2022 at 01:22, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> 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.

Why?

The fake reset is there for use by tests. We don't need this load of
Python code at all for sandbox. We should worry about it later.

Regards,
Simon