{"id":2223614,"url":"http://patchwork.ozlabs.org/api/1.1/patches/2223614/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20260415183250.3031030-6-dario.binacchi@amarulasolutions.com/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/1.1/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},"msgid":"<20260415183250.3031030-6-dario.binacchi@amarulasolutions.com>","date":"2026-04-15T18:31:32","name":"[v3,5/7] test: cmd: part: add UUID lookup tests","commit_ref":null,"pull_url":null,"state":"changes-requested","archived":true,"hash":"f404b5289d1aa8da0410f62ca79274d2b29c8c22","submitter":{"id":83038,"url":"http://patchwork.ozlabs.org/api/1.1/people/83038/?format=json","name":"Dario Binacchi","email":"dario.binacchi@amarulasolutions.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20260415183250.3031030-6-dario.binacchi@amarulasolutions.com/mbox/","series":[{"id":500027,"url":"http://patchwork.ozlabs.org/api/1.1/series/500027/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=500027","date":"2026-04-15T18:31:27","name":"Support metadata-driven A/B boot for STM32MP25","version":3,"mbox":"http://patchwork.ozlabs.org/series/500027/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2223614/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2223614/checks/","tags":{},"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 (1024-bit key;\n unprotected) header.d=amarulasolutions.com header.i=@amarulasolutions.com\n header.a=rsa-sha256 header.s=google header.b=oBwWJab0;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de\n (client-ip=85.214.62.61; helo=phobos.denx.de;\n envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org)","phobos.denx.de;\n dmarc=pass (p=none dis=none) header.from=amarulasolutions.com","phobos.denx.de;\n spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de","phobos.denx.de;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=amarulasolutions.com header.i=@amarulasolutions.com\n header.b=\"oBwWJab0\";\n\tdkim-atps=neutral","phobos.denx.de; dmarc=pass (p=none dis=none)\n header.from=amarulasolutions.com","phobos.denx.de; spf=pass\n smtp.mailfrom=dario.binacchi@amarulasolutions.com"],"Received":["from phobos.denx.de (phobos.denx.de [85.214.62.61])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fwqXx39qkz1yHM\n\tfor ; Thu, 16 Apr 2026 04:33:53 +1000 (AEST)","from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 0437C8421D;\n\tWed, 15 Apr 2026 20:33:15 +0200 (CEST)","by phobos.denx.de (Postfix, from userid 109)\n id 564308420C; Wed, 15 Apr 2026 20:33:14 +0200 (CEST)","from mail-wm1-x32f.google.com (mail-wm1-x32f.google.com\n [IPv6:2a00:1450:4864:20::32f])\n (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits))\n (No client certificate requested)\n by phobos.denx.de (Postfix) with ESMTPS id 4DBF5841CD\n for ; Wed, 15 Apr 2026 20:33:12 +0200 (CEST)","by mail-wm1-x32f.google.com with SMTP id\n 5b1f17b1804b1-48897fd88ebso72756655e9.2\n for ; Wed, 15 Apr 2026 11:33:12 -0700 (PDT)","from dario-ThinkPad-P14s-Gen-5.amarulasolutions.com ([2.196.40.93])\n by smtp.gmail.com with ESMTPSA id\n 5b1f17b1804b1-488f0e72980sm34953115e9.5.2026.04.15.11.33.10\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Wed, 15 Apr 2026 11:33:10 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=amarulasolutions.com; s=google; t=1776277991; x=1776882791;\n darn=lists.denx.de;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=KMrUAAtuax7uvVus6CWUVq2R8GJCStf3Di3MtyLYARE=;\n b=oBwWJab0eHS/sepcDsMGuWVvcianrYs6QZAAaz6PDEvJ8EsfAmtynNs9ZDHVrM4QCr\n 0xz8dLTnMa5rTa9xbdOh7UJRBJJ6vHWwHDZOhsnr56f73diMJTLSTEwryMhJt0Go9JPc\n 1v83Be6vdF5M7hLnih4oorq0vQ025QbXdQ7Rw=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1776277991; x=1776882791;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=KMrUAAtuax7uvVus6CWUVq2R8GJCStf3Di3MtyLYARE=;\n b=LSMnzd/Zp5e65e2xLUXKEpJG2zj12uvCVREM/rkXUSL3q6nSoTqRq6XYM+uqM9jlBN\n xFnfbJH4gWW/fRkPIBmeHVU1MmpWbTKvzktN7t+U5I9OqH7HYAZvtKFlCzBo39XENU/W\n 0EiZQbITQcmNt+mhdHaak6LlSc4kFsZ/Rtgd8BQ2GFyv2+dzg0EQ2Aaen2ZhNIFx6sqQ\n 4AYLy0+SFREK2Cn7FDPNfLTdfdAohtfyi9Ll4CNG7fcHfcalzHrwdyuTZBzO1NINRc1w\n iltA9kmdXKbW0qR+4/hIXmjBJmfD46miWHdFiTy5bzgn0RJHi/9ZWb/VfMZXhT4Wgsn0\n BPnA==","X-Gm-Message-State":"AOJu0Yz7P2ggMOiz+9Q7Ivsq7qxeRKP8dRBfYPmmncQU06tFi2OHyrbG\n z/wESoFPge9aahW26jVXYaLCmakAa2HPzryVM0fPUJP1nFRn6G6M0N1HPtdOJK2kPUQubJTnYqd\n gu+/z","X-Gm-Gg":"AeBDievlrWhfAXyOCBP7RBE/1rLuMIAlYhSpiOr9efzEfwfVapAgLPcqZN8oVn3ONyV\n Xx/EheFUcimmgwfrblQzqheEQME6aE1k0F7V3xVzZEydw4DC0IahPTHbANrQ9g70+qK5IZ533Ad\n kvpn8MopdsCUgNun7VJneB85V2Rd1djuNjzdlJrEQkjeUq3GZuAn7lDqgnGkhEZTvqYKbdMOITq\n Nxa/XJw9zQNDhaB03UquLFMXVZpgv+L6rRxP4TLd+Pwj06GV10+94awhOAFAjXUoaKCUd8ZAKwE\n 1ZwYUqK38DcA8qebYG5XgJF6F7SK2K9T0nn58PRm5yNG6ebKm+5trX5pwkE/xN89tySVGvHwqsV\n cRId8nPL7JG7xHNwjD9qk1PAeblRWKt0I+0uAjoq7pCoxWyfxmv7X93/QdL3SQOcquZjCdEKICW\n GQlNXT6WqWYlH5dIMuFNy+ypcTNP7FBO2n0jotISt+muo54WhKnAnBrDSr3TsP12+RY+OkwVKBp\n MwSo1CbUyUORqAL7KzCGMxK121aE+ZhZneEyuX9j2MzijGAciJ0V/qzQg==","X-Received":"by 2002:a05:600c:450a:b0:488:a882:b7 with SMTP id\n 5b1f17b1804b1-488d6ad17e6mr304461325e9.29.1776277991125;\n Wed, 15 Apr 2026 11:33:11 -0700 (PDT)","From":"Dario Binacchi ","To":"u-boot@lists.denx.de","Cc":"linux-amarula@amarulasolutions.com,\n Dario Binacchi ,\n Simon Glass , Tom Rini ","Subject":"[PATCH v3 5/7] test: cmd: part: add UUID lookup tests","Date":"Wed, 15 Apr 2026 20:31:32 +0200","Message-ID":"<20260415183250.3031030-6-dario.binacchi@amarulasolutions.com>","X-Mailer":"git-send-email 2.43.0","In-Reply-To":"<20260415183250.3031030-1-dario.binacchi@amarulasolutions.com>","References":"<20260415183250.3031030-1-dario.binacchi@amarulasolutions.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","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":"Extend the 'part' command unit tests to include partition lookup via\nUUID.\n\nThis ensures that the 'number', 'start', and 'size' subcommands\nconsistently handle UUIDs as partition identifiers, maintaining\nparity with the name-based lookup functionality.\n\nSigned-off-by: Dario Binacchi \nReviewed-by: Simon Glass \n\n---\n\nChanges in v3:\n- Combine run_command() and ut_asserteq() -> ut_asserteq(1, run_command(\n- Add Reviewed-by tag of Simon Glass\n\n test/cmd/part.c | 44 ++++++++++++++++++++++++++++++++++++++++++++\n 1 file changed, 44 insertions(+)","diff":"diff --git a/test/cmd/part.c b/test/cmd/part.c\nindex f808ddbc87eb..3ac6b02cc368 100644\n--- a/test/cmd/part.c\n+++ b/test/cmd/part.c\n@@ -76,6 +76,20 @@ static int dm_test_cmd_part_number(struct unit_test_state *uts)\n \t\t\t\t mmc_dev_num));\n \tut_assertnull(env_get(\"partnum\"));\n \n+\tfor (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {\n+\t\tenv_set(\"partnum\", NULL);\n+\t\tut_assertok(run_commandf(\"part number mmc %u %s partnum\",\n+\t\t\t\t\t mmc_dev_num, gpt_parts[i].uuid));\n+\t\tsnprintf(expected, sizeof(expected), \"0x%x\", i + 1);\n+\t\tut_asserteq_str(expected, env_get(\"partnum\"));\n+\t}\n+\n+\tenv_set(\"partnum\", NULL);\n+\tut_asserteq(1, run_commandf(\"part number mmc %u %s partnum\",\n+\t\t\t\t mmc_dev_num,\n+\t\t\t\t \"00000000-0000-0000-0000-000000000000\"));\n+\tut_assertnull(env_get(\"partnum\"));\n+\n \treturn 0;\n }\n DM_TEST(dm_test_cmd_part_number, UTF_SCAN_PDATA | UTF_SCAN_FDT);\n@@ -116,6 +130,21 @@ static int dm_test_cmd_part_start(struct unit_test_state *uts)\n \t\t\t\t mmc_dev_num));\n \tut_assertnull(env_get(\"partstart\"));\n \n+\tfor (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {\n+\t\tenv_set(\"partstart\", NULL);\n+\t\tut_assertok(run_commandf(\"part start mmc %u %s partstart\",\n+\t\t\t\t\t mmc_dev_num, gpt_parts[i].uuid));\n+\t\tsnprintf(expected, sizeof(expected), \"%lx\",\n+\t\t\t (unsigned long)gpt_parts[i].start);\n+\t\tut_asserteq_str(expected, env_get(\"partstart\"));\n+\t}\n+\n+\tenv_set(\"partstart\", NULL);\n+\tut_asserteq(1, run_commandf(\"part start mmc %u %s partstart\",\n+\t\t\t\t mmc_dev_num,\n+\t\t\t\t \"00000000-0000-0000-0000-000000000000\"));\n+\tut_assertnull(env_get(\"partstart\"));\n+\n \treturn 0;\n }\n DM_TEST(dm_test_cmd_part_start, UTF_SCAN_PDATA | UTF_SCAN_FDT);\n@@ -156,6 +185,21 @@ static int dm_test_cmd_part_size(struct unit_test_state *uts)\n \t\t\t\t mmc_dev_num));\n \tut_assertnull(env_get(\"partsize\"));\n \n+\tfor (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {\n+\t\tenv_set(\"partsize\", NULL);\n+\t\tut_assertok(run_commandf(\"part size mmc %u %s partsize\",\n+\t\t\t\t\t mmc_dev_num, gpt_parts[i].uuid));\n+\t\tsnprintf(expected, sizeof(expected), \"%lx\",\n+\t\t\t (unsigned long)gpt_parts[i].size);\n+\t\tut_asserteq_str(expected, env_get(\"partsize\"));\n+\t}\n+\n+\tenv_set(\"partsize\", NULL);\n+\tut_asserteq(1, run_commandf(\"part size mmc %u %s partsize\",\n+\t\t\t\t mmc_dev_num,\n+\t\t\t\t \"00000000-0000-0000-0000-000000000000\"));\n+\tut_assertnull(env_get(\"partsize\"));\n+\n \treturn 0;\n }\n DM_TEST(dm_test_cmd_part_size, UTF_SCAN_PDATA | UTF_SCAN_FDT);\n","prefixes":["v3","5/7"]}