{"id":2224486,"url":"http://patchwork.ozlabs.org/api/1.2/patches/2224486/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20260417130204.49896-7-philippe.reynes@softathome.com/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/1.2/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260417130204.49896-7-philippe.reynes@softathome.com>","list_archive_url":null,"date":"2026-04-17T13:01:56","name":"[v4,06/14] test: dm: ecdsa.c: clean this test as software ecdsa is now implemented","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"3b7d263829a5339b48b230b6852d1b98f4104ab9","submitter":{"id":74351,"url":"http://patchwork.ozlabs.org/api/1.2/people/74351/?format=json","name":"Philippe Reynes","email":"philippe.reynes@softathome.com"},"delegate":{"id":161313,"url":"http://patchwork.ozlabs.org/api/1.2/users/161313/?format=json","username":"raymo200915","first_name":"Raymond","last_name":"Mao","email":"raymondmaoca@gmail.com"},"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20260417130204.49896-7-philippe.reynes@softathome.com/mbox/","series":[{"id":500332,"url":"http://patchwork.ozlabs.org/api/1.2/series/500332/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=500332","date":"2026-04-17T13:02:04","name":"add software ecdsa support","version":4,"mbox":"http://patchwork.ozlabs.org/series/500332/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2224486/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2224486/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=softathome1.onmicrosoft.com header.i=@softathome1.onmicrosoft.com\n header.a=rsa-sha256 header.s=selector1-softathome1-onmicrosoft-com\n header.b=GFVKLi2y;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de\n (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de;\n envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org)","phobos.denx.de;\n dmarc=none (p=none dis=none) header.from=softathome.com","phobos.denx.de;\n spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de","phobos.denx.de;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=softathome1.onmicrosoft.com header.i=@softathome1.onmicrosoft.com\n header.b=\"GFVKLi2y\";\n\tdkim-atps=neutral","phobos.denx.de; dmarc=none (p=none dis=none)\n header.from=softathome.com","phobos.denx.de;\n spf=pass smtp.mailfrom=philippe.reynes@softathome.com"],"Received":["from phobos.denx.de (phobos.denx.de\n [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fxw7M4X42z1yCv\n\tfor ; Fri, 17 Apr 2026 23:03:59 +1000 (AEST)","from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 2C08984329;\n\tFri, 17 Apr 2026 15:02:23 +0200 (CEST)","by phobos.denx.de (Postfix, from userid 109)\n id 2590784262; Fri, 17 Apr 2026 15:02:20 +0200 (CEST)","from PA5P264CU001.outbound.protection.outlook.com\n (mail-francecentralazlp170100000.outbound.protection.outlook.com\n [IPv6:2a01:111:f403:c20a::])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))\n (No client certificate requested)\n by phobos.denx.de (Postfix) with ESMTPS id 21BF0842C9\n for ; Fri, 17 Apr 2026 15:02:18 +0200 (CEST)","from PR1P264CA0126.FRAP264.PROD.OUTLOOK.COM (2603:10a6:102:2cd::20)\n by MR1PPFAFCFF2AA2.FRAP264.PROD.OUTLOOK.COM (2603:10a6:508:1::2ca)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9745.45; Fri, 17 Apr\n 2026 13:02:10 +0000","from PA1PEPF000CC3FB.FRAP264.PROD.OUTLOOK.COM\n (2603:10a6:102:2cd:cafe::77) by PR1P264CA0126.outlook.office365.com\n (2603:10a6:102:2cd::20) with Microsoft SMTP Server (version=TLS1_3,\n cipher=TLS_AES_256_GCM_SHA384) id 15.20.9769.52 via Frontend Transport; Fri,\n 17 Apr 2026 13:02:10 +0000","from proxy.softathome.com (149.6.166.170) by\n PA1PEPF000CC3FB.mail.protection.outlook.com (10.167.242.6) with Microsoft\n SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9769.17\n via Frontend Transport; Fri, 17 Apr 2026 13:02:10 +0000","from sah1lpt726.home (unknown [192.168.72.39])\n by proxy.softathome.com (Postfix) with ESMTPSA id 0BB0120EC3;\n Fri, 17 Apr 2026 15:02:10 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n DKIM_VALID,RCVD_IN_DNSWL_BLOCKED,SPF_HELO_PASS,SPF_PASS autolearn=ham\n autolearn_force=no version=3.4.2","ARC-Seal":"i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;\n b=QZZc8Vt6AugSzqLQB+kcu8cXD2NcMf9mkWeT8MkN+GmJlU4rRWBLC1U6fcPQts1DEp+NNzTK2GhiWrk2k3qtM3qHpEj8eu2UiX8keb7dDzf1H3OZX+mt/eI0an3vFn7khuloQl3Hd7DEeVKvSiZcamrYKGK31FwUu2IwifSTPDs7P5KS6Vww5YEdmwhQ+LGBWjRQuIKFV2bUgshd7Uedcb/R3pKP5WYzcSpvymvetmErBGX9dRxyrFb+YXsm9nPLN6Mh47299fGLW0qAdH+DkDgQOk5oFLs6idaQThvf4YuUq1G1obinKCHEw/aj8l8P/X4Rnqk1Pc50oxnx8zdF8A==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector10001;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=e+dsiUpY+xIMy3dX2yySZrgzLjxTheVc9mEpOl9n1zM=;\n b=Y26/KE5x6WLx+b8aA6pwSxft6xmhQ1ylkQzjDfaYyMXM5gHeRHa4ujmxvh7q++0hn/pegbJ/S0JVuKA1HdccagcnJHRsLVoUOifs8VVmmLqsNTeUMrAJRF90qDM1moeJxIFth8dlM2ohi+JUBf/DWEaLKiMKtufIe84pBiBXykrEfPfPG9KNHRnz8kjzNozkkemTHbhPre8RGqlauD+hZVJ9UD7ebBYJcyEOZmFlxzY5HbMEvCg8bfZiCkMszQG/Z0Gs5TUr+pwhofO64nGcSeB/aDd7JkTpuVXzatupxoj9uhA5gIPnSqT8WI8QEABIGW5UeIfkE/uXpvoKsMe5eQ==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass (sender ip is\n 149.6.166.170) smtp.rcpttodomain=canonical.com smtp.mailfrom=softathome.com;\n dmarc=bestguesspass action=none header.from=softathome.com; dkim=none\n (message not signed); arc=none (0)","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=softathome1.onmicrosoft.com; s=selector1-softathome1-onmicrosoft-com;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=e+dsiUpY+xIMy3dX2yySZrgzLjxTheVc9mEpOl9n1zM=;\n b=GFVKLi2yrkDJ9ta7K3hcAg+RP++//YiKE631QIDrSrnJIiPsRKn64Xa0kQIqke/T02lsKjXNnj8X0HWkgc+ovJkt5AsuGklN1diW+jFxoXc9zpYxR1dggUswmcvZg+1+MudpFsDYOvHL1WQNap7I22wBmCdkBcthNh+S9sNHcSdNWGWw6blpjWa1lHyn/F65kQ+MR4YIBEunT09bPi0E1PJUtEjoJcYrN+yekX9UQ9feJUSd1hpEG9U2PltrJDjWnJrm9EPCItFynzHcq4gbNMoBZQ/xvUO+bxM32FFnkFVxhnWaYgr2TRXGrSpnoo7a1g3IYqdankQw3DOkwDPgLw==","X-MS-Exchange-Authentication-Results":"spf=pass (sender IP is 149.6.166.170)\n smtp.mailfrom=softathome.com; dkim=none (message not signed)\n header.d=none;dmarc=bestguesspass action=none header.from=softathome.com;","Received-SPF":"Pass (protection.outlook.com: domain of softathome.com\n designates 149.6.166.170 as permitted sender)\n receiver=protection.outlook.com; client-ip=149.6.166.170;\n helo=proxy.softathome.com; pr=C","From":"Philippe Reynes ","To":"marko.makela@iki.fi, jonny.green@keytechinc.com, raymondmaoca@gmail.com,\n trini@konsulko.com, simon.glass@canonical.com","Cc":"u-boot@lists.denx.de,\n\tPhilippe Reynes ","Subject":"[PATCH v4 06/14] test: dm: ecdsa.c: clean this test as software ecdsa\n is now implemented","Date":"Fri, 17 Apr 2026 15:01:56 +0200","Message-ID":"<20260417130204.49896-7-philippe.reynes@softathome.com>","X-Mailer":"git-send-email 2.43.0","In-Reply-To":"<20260417130204.49896-1-philippe.reynes@softathome.com>","References":"<20260417130204.49896-1-philippe.reynes@softathome.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","X-EOPAttributedMessage":"0","X-MS-PublicTrafficType":"Email","X-MS-TrafficTypeDiagnostic":"PA1PEPF000CC3FB:EE_|MR1PPFAFCFF2AA2:EE_","Content-Type":"text/plain","X-MS-Office365-Filtering-Correlation-Id":"9d7c551f-0949-45b4-8b90-08de9c81896c","X-MS-Exchange-SenderADCheck":"1","X-MS-Exchange-AntiSpam-Relay":"0","X-Microsoft-Antispam":"BCL:0;\n ARA:13230040|82310400026|376014|36860700016|1800799024|56012099003|22082099003|18002099003;","X-Microsoft-Antispam-Message-Info":"\n 1tPEB3FU5aCvIW/DN1tOelydB6VkgFLi3rHR1RkNt5Cgd+xTD8SKycLAc09WPjk0P7P4ne33ztijbvW/gfH+qVXnPfEPN1otnm+6u2mUEmhvf151VhzclTW64rSOky4vbG9f/75vo91nl3zp+9dWU/PgFPtn84dN/DhBhto6ARBTqWOVriMuBxLgMjE6hwA4Osxjvy1h9KPecKKYva6aeKPxJ4p2KwA3oqiE+JJmWi+vFZd0ioDrbv0r9KrxDxq3g2MCBYaoUHKi6kbikm21Rd0IeAYMBNX1AfD9mVMwiB3rLZ5zSfzDfO7qWc2lNB01EjUrxN5HwmKak1VWSQ3yp2pxq7HSdWWZ1UMiwXUJm+fKOyXBc1ukNz59fRS2AGh3tOIQosreyF86sh/M4gLTZYBuoBQIRPy58KUiWSHmZlK09UGOon8MfHI+1q+lNZI8USqB88KfKlGgze5UTjnQ/UUlp1J0uC451t3AXvjUbtci+UaaUkroXd20f8zUbfkMmjijEE/ITGavG1dH4HStRLCH774WdowoEVFYjyoZLb165v11rKDE5o0uERIlUse7YeNpYNPKwgFCmr8AgNoalVARjUlrlKSsuIhJQ7ftctXbyKigqmrrh/jNuzD9xa18noLUerzD7voFs2vv5oYehJlCyNM2E8knKDpBvX+3ZUb88S/frclqHxOMlslYNEpbuwxcnPrHMCp7YSHknxpVmKdC5bBHbAGTrT/auQZWvGPH1siLsp8VVl/3e24FEuqpTWVdXz2wweCbxtlCnaQ1Bg==","X-Forefront-Antispam-Report":"CIP:149.6.166.170; CTRY:FR; LANG:en; SCL:1; SRV:;\n IPV:CAL; SFV:NSPM; H:proxy.softathome.com; PTR:InfoDomainNonexistent;\n CAT:NONE;\n SFS:(13230040)(82310400026)(376014)(36860700016)(1800799024)(56012099003)(22082099003)(18002099003);\n DIR:OUT; SFP:1101;","X-MS-Exchange-AntiSpam-MessageData-ChunkCount":"1","X-MS-Exchange-AntiSpam-MessageData-0":"\n 8KPs05HQ9tWipqTMG9rqbcEPKA7io9A6z5MWcQhMbMijQXPGCfHHzHuaDxuaTS8jPz90+ATW+SehvZcp8H2GwhRcc3ejsZl7YmkoSHpPO75+mVT4R4Om9ZMswzZJIGYj7A7d0bgsmHSNNaTJNWPVGJWjgivbtI9aofg1XrMrv3yWi3YKqHrKVIr7ajgUyW1hDlwkPAqH+OHZTa2ipyaWlDikKR/uNKqBlB7sGoiOeQYxAXaCwRrbRBZ2gy1ioHKpHgxdCdLmJDM1LULN6z1R+AiDDGWeNxwaPGvxCvm150wmwsilz/dc6drbA1OcVcuuy9WyRJUuXkcQV66fg8sFvsE2BpGlmmE7QUawLm/rCTVXVT2ZbjeGpu6P/q+CLvsnBk/UD8xbmYjAaE7itarovRTzt8RhASINNvuV+xEl2cqewQ5iHuYh4Vlmxy5vWtAh","X-OriginatorOrg":"softathome.com","X-MS-Exchange-CrossTenant-OriginalArrivalTime":"17 Apr 2026 13:02:10.3404 (UTC)","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n 9d7c551f-0949-45b4-8b90-08de9c81896c","X-MS-Exchange-CrossTenant-Id":"aa10e044-e405-4c10-8353-36b4d0cce511","X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp":"\n TenantId=aa10e044-e405-4c10-8353-36b4d0cce511; Ip=[149.6.166.170];\n Helo=[proxy.softathome.com]","X-MS-Exchange-CrossTenant-AuthSource":"PA1PEPF000CC3FB.FRAP264.PROD.OUTLOOK.COM","X-MS-Exchange-CrossTenant-AuthAs":"Anonymous","X-MS-Exchange-CrossTenant-FromEntityHeader":"HybridOnPrem","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"MR1PPFAFCFF2AA2","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.39","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" ","X-Virus-Scanned":"clamav-milter 0.103.8 at phobos.denx.de","X-Virus-Status":"Clean"},"content":"The test ecdsa was done when ecdsa was only supported by hardware.\nSo it wasn't possible to test ecdsa on sandbox, and there is a test\nto check that ecdsa is not supported on sandbox.\nNow, there is a software implementation of ecdsa. So we remove\nthis outdated test.\n\nSigned-off-by: Philippe Reynes \n---\nv2:\n- initial version\nv3:\n- no change\nv4:\n- add a test that use ecdsa_verify\n\n\n test/dm/ecdsa.c | 104 +++++++++++++++++++++++++++++++++++++++++++-----\n 1 file changed, 94 insertions(+), 10 deletions(-)","diff":"diff --git a/test/dm/ecdsa.c b/test/dm/ecdsa.c\nindex d7eac7115f7..946d5ab1d81 100644\n--- a/test/dm/ecdsa.c\n+++ b/test/dm/ecdsa.c\n@@ -3,36 +3,120 @@\n #include \n #include \n #include \n+#include \n #include \n #include \n \n+#define CHECK(op) ({\t\t\t\t\t\t\t\\\n+\t\tint err = op;\t\t\t\t\t\t\\\n+\t\tif (err < 0) {\t\t\t\t\t\t\\\n+\t\t\tprintf(\"%s: %s: %s\\n\", __func__, #op,\t\t\\\n+\t\t\t fdt_strerror(err));\t\t\t\\\n+\t\t\treturn err;\t\t\t\t\t\\\n+\t\t}\t\t\t\t\t\t\t\\\n+\t\t\t\t\t\t\t\t\t\\\n+\t\terr;\t\t\t\t\t\t\t\\\n+\t})\n+\n+static int set_fdt_ecdsa_point(char *fdt, char *name, const char *data, size_t len)\n+{\n+\tchar *value = NULL;\n+\tint ret = 0;\n+\n+\tif (!fdt || !name || !data) {\n+\t\tret = -EINVAL;\n+\t\tgoto out;\n+\t}\n+\n+\tvalue = malloc(len / 2);\n+\tif (!value) {\n+\t\tret = -EINVAL;\n+\t\tgoto out;\n+\t}\n+\n+\tret = hex2bin(value, data, len / 2);\n+\tif (ret)\n+\t\tgoto out;\n+\n+\tret = fdt_property(fdt, name, value, len / 2);\n+\tif (ret)\n+\t\tgoto out;\n+\n+ out:\n+\tfree(value);\n+\treturn ret;\n+}\n+\n+static int create_fdt_with_ecdsa_key(char *fdt, size_t size,\n+\t\t\t\t const char *name, const char *curve,\n+\t\t\t\t const char *x, const char *y)\n+{\n+\tCHECK(fdt_create(fdt, size));\n+\tCHECK(fdt_finish_reservemap(fdt));\n+\tCHECK(fdt_begin_node(fdt, \"\"));\n+\tCHECK(fdt_begin_node(fdt, \"signature\"));\n+\tCHECK(fdt_begin_node(fdt, name));\n+\tCHECK(fdt_property_string(fdt, \"algo\", \"sha256,ecdsa256\"));\n+\tCHECK(set_fdt_ecdsa_point(fdt, \"ecdsa,y-point\", y, strlen(y)));\n+\tCHECK(set_fdt_ecdsa_point(fdt, \"ecdsa,x-point\", x, strlen(x)));\n+\tCHECK(fdt_property_string(fdt, \"ecdsa,curve\", curve));\n+\tCHECK(fdt_property_string(fdt, \"key-name-hint\", name));\n+\tCHECK(fdt_end_node(fdt)); /* name */\n+\tCHECK(fdt_end_node(fdt)); /* \"signature\" */\n+\tCHECK(fdt_end_node(fdt)); /* \"\" */\n+\tCHECK(fdt_finish(fdt));\n+\tCHECK(fdt_pack(fdt));\n+\n+\treturn 0;\n+}\n+\n /*\n * Basic test of the ECDSA uclass and ecdsa_verify()\n *\n- * ECDSA implementations in u-boot are hardware-dependent. Until we have a\n- * software implementation that can be compiled into the sandbox, all we can\n- * test is the uclass support.\n+ * ECDSA software implementation is tested in another test,\n+ * so we only check that the class UCLASS_ECDSA may be used.\n *\n- * The uclass_get() test is redundant since ecdsa_verify() would also fail. We\n- * run both functions in order to isolate the cause more clearly. i.e. is\n- * ecdsa_verify() failing because the UCLASS is absent/broken?\n+ * The data used in this test come from RFC6979\n */\n static int dm_test_ecdsa_verify(struct unit_test_state *uts)\n {\n \tstruct uclass *ucp;\n+\tconst char *full_name = \"sha256,secp256r1\";\n+\tconst char *name = \"key-ecdsa-256\";\n+\tconst char *curve = \"prime256v1\";\n+\tconst char *x = \"60FED4BA255A9D31C961EB74C6356D68C049B8923B61FA6CE669622E60F29FB6\";\n+\tconst char *y = \"7903FE1008B8BC99A41AE9E95628BC64F2F1B20C2D7E9F5177A3C294D4462299\";\n+\tconst char *r = \"EFD48B2AACB6A8FD1140DD9CD45E81D69D2C877B56AAF991C34D0EA84EAF3716\";\n+\tconst char *s = \"F7CB1C942D657C41D436C7A1B6E29F65F3E900DBB9AFF4064DC4AB2F843ACDA8\";\n+\tuint8_t sig[64];\n+\tsize_t fdt_size = 512;\n+\tchar fdt[512];\n \n-\tstruct checksum_algo algo = {\n-\t\t.checksum_len = 256,\n+\tstruct image_region region[] = {\n+\t\t{\n+\t\t\t.data = \"sample\",\n+\t\t\t.size = strlen(\"sample\"),\n+\t\t},\n \t};\n \n \tstruct image_sign_info info = {\n-\t\t.checksum = &algo,\n+\t\t.checksum = image_get_checksum_algo(full_name),\n+\t\t.crypto = image_get_crypto_algo(full_name),\n+\t\t.fdt_blob = fdt,\n \t};\n \n+\t/* create a fdt with the public key */\n+\tut_asserteq(0, create_fdt_with_ecdsa_key(fdt, fdt_size, name, curve, x, y));\n+\n+\t/* prepare the signature */\n+\tut_asserteq(0, hex2bin(sig + 0, r, strlen(r) / 2));\n+\tut_asserteq(0, hex2bin(sig + 32, s, strlen(s) / 2));\n+\n \tut_assertok(uclass_get(UCLASS_ECDSA, &ucp));\n \tut_assertnonnull(ucp);\n-\tut_asserteq(-ENODEV, ecdsa_verify(&info, NULL, 0, NULL, 0));\n+\tut_asserteq(0, ecdsa_verify(&info, region, 1, sig, sizeof(sig)));\n \n \treturn 0;\n }\n+\n DM_TEST(dm_test_ecdsa_verify, UTF_SCAN_PDATA | UTF_SCAN_FDT);\n","prefixes":["v4","06/14"]}