Hi Philippe,

On 2026-03-31T10:00:34, Philippe Reynes <[email protected]> wrote:
> diff --git a/test/dm/ecdsa.c b/test/dm/ecdsa.c
> @@ -9,29 +9,15 @@ static int dm_test_ecdsa_verify(struct unit_test_state *uts)
> -     ut_asserteq(-ENODEV, ecdsa_verify(&info, NULL, 0, NULL, 0));

With the software ECDSA driver now available, this test would no
longer return -ENODEV. Rather than just removing this assertion, could
the test call ecdsa_verify() with valid test data and check for
success? A DM test that exercises the full ecdsa_verify() path would
be useful to ensure the driver binding works correctly.

Reviewed-by: Simon Glass <[email protected]>

Regards,
Simon

Reply via email to