[v3,06/15] test: dm: ecdsa.c: clean this test as software ecdsa is now implemented

Simon Glass sjg at chromium.org
Thu Apr 2 00:09:24 CEST 2026


Hi Philippe,

On 2026-03-31T10:00:34, Philippe Reynes <philippe.reynes at softathome.com> 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 <sjg at chromium.org>

Regards,
Simon


More information about the U-Boot mailing list