Hi Jorge,

On Tue, 9 Feb 2021 at 13:10, Jorge Ramirez-Ortiz <jo...@foundries.io> wrote:
>
> From: Igor Opaniuk <igor.opan...@foundries.io>
>
> Add initial test coverage for SCP03 command.
>
> Signed-off-by: Igor Opaniuk <igor.opan...@foundries.io>
> ---
>  test/py/tests/test_scp03.py | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
>  create mode 100644 test/py/tests/test_scp03.py

Reviewed-by: Simon Glass <s...@chromium.org>

This is fine, but note you can also write this test in C for sandbox,
and this might be easier if it gets more complicated. For example you
can check the state of devices from C.

Regards,
Simon

Reply via email to