[PATCH v4 06/14] test: dm: ecdsa.c: clean this test as software ecdsa is now implemented
Simon Glass
sjg at chromium.org
Mon Apr 20 04:11:58 CEST 2026
Hi Philippe,
On 2026-04-17T13:02:04, Philippe Reynes <philippe.reynes at softathome.com> wrote:
> test: dm: ecdsa.c: clean this test as software ecdsa is now implemented
>
> The test ecdsa was done when ecdsa was only supported by hardware.
> So it wasn't possible to test ecdsa on sandbox, and there is a test
> to check that ecdsa is not supported on sandbox.
> Now, there is a software implementation of ecdsa. So we remove
> this outdated test.
>
> Signed-off-by: Philippe Reynes <philippe.reynes at softathome.com>
>
> test/dm/ecdsa.c | 104 ++++++++++++++++++++++++++++++++++++++++++++++++++------
> 1 file changed, 94 insertions(+), 10 deletions(-)
> diff --git a/test/dm/ecdsa.c b/test/dm/ecdsa.c
> @@ -3,36 +3,120 @@
> + struct image_sign_info info = {
> + .checksum = image_get_checksum_algo(full_name),
> + .crypto = image_get_crypto_algo(full_name),
> + .fdt_blob = fdt,
> + };
If the crypto algo name is invalid, info.crypto will be NULL - please
can you add ut_assertnonnull() checks for both checksum and crypto to
catch this earlier?
Regards,
Simon
More information about the U-Boot
mailing list