Hi Denis,

On 2026-05-29T03:48:33, None <[email protected]> wrote:
> tests: reset: add cold/warm reset types
>
> Extend reset tests to cover both -c/-w reset command flags.
> Make sure sandbox reset tests are passing.
>
> Signed-off-by: Denis Mukhin <[email protected]>
>
> test/py/tests/test_reset.py | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)

> diff --git a/test/py/tests/test_reset.py b/test/py/tests/test_reset.py
> @@ -52,12 +52,12 @@ def test_reset(ubman):
> [email protected]('reset_type', ['c', 'w'])
> +def test_reset_type(ubman, reset_type):
> +    """Test the reset -{c,w} command in non-JTAG bootmode.
>      """
>      setup_reset_env(ubman)
> -    ubman.run_command('reset -w', wait_for_reboot=True)
> +    ubman.run_command(f"reset -{reset_type}", wait_for_reboot=True)

U-Boot uses single quotes throughout ('hush_parser', 'reset -w',
etc.). Please keep that style - @pytest.mark.parametrize('reset_type',
['c', 'w']) and f'reset -{reset_type}'.

Regards,
Simon

Reply via email to