[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