diff mbox series

[v2,4/7] cmd: part: support lookup by UUID

Message ID 20260404152640.3297713-5-dario.binacchi@amarulasolutions.com
State New
Headers
Series Support metadata-driven A/B boot for STM32MP25 |

Commit Message

Dario Binacchi April 4, 2026, 3:23 p.m. UTC 
The 'part' command currently allows looking up a partition only by its
number or name.

Extend the 'number', 'start', and 'size' subcommands to support looking
up the partition via its UUID. Unlike names, UUIDs guarantee unique
partition identification, avoiding ambiguity.

The logic is updated to check if the provided string is a valid UUID
before falling back to a name-based search. The help strings for these
subcommands are updated accordingly.

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>

---

Changes in v2:
- Update commit message.
- Update help for 'part start', 'part size' to mention UUID.

 cmd/part.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

Comments

Simon Glass April 10, 2026, 11 p.m. UTC | #1 
On 2026-04-04T15:23:27, Dario Binacchi
<dario.binacchi@amarulasolutions.com> wrote:
> cmd: part: support lookup by UUID
>
> The 'part' command currently allows looking up a partition only by its
> number or name.
>
> Extend the 'number', 'start', and 'size' subcommands to support looking
> up the partition via its UUID. Unlike names, UUIDs guarantee unique
> partition identification, avoiding ambiguity.
>
> The logic is updated to check if the provided string is a valid UUID
> before falling back to a name-based search. The help strings for these
> subcommands are updated accordingly.
>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>
> cmd/part.c | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

To unsubscribe from this group and stop receiving emails from it, send an email to linux-amarula+unsubscribe@amarulasolutions.com.
diff mbox series

Patch

diff --git a/cmd/part.c b/cmd/part.c
index 975a0a08a99d..c46ed25f37aa 100644
--- a/cmd/part.c
+++ b/cmd/part.c
@@ -139,6 +139,10 @@  static int do_part_info(int argc, char *const argv[], enum cmd_part_info param)
 		err = part_get_info(desc, part, &info);
 		if (err)
 			return 1;
+	} else if (uuid_str_valid(argv[2])) {
+		part = part_get_info_by_uuid(desc, argv[2], &info);
+		if (part < 0)
+			return 1;
 	} else {
 		part = part_get_info_by_name(desc, argv[2], &info);
 		if (part < 0)
@@ -309,13 +313,13 @@  U_BOOT_CMD(
 	"      flags can be -bootable (list only bootable partitions)\n"
 	"part start <interface> <dev> <part> <varname>\n"
 	"    - set environment variable to the start of the partition (in blocks)\n"
-	"      part can be either partition number or partition name\n"
+	"      part can be either partition number, UUID or name\n"
 	"part size <interface> <dev> <part> <varname>\n"
 	"    - set environment variable to the size of the partition (in blocks)\n"
-	"      part can be either partition number or partition name\n"
+	"      part can be either partition number, UUID or name\n"
 	"part number <interface> <dev> <part> <varname>\n"
-	"    - set environment variable to the partition number using the partition name\n"
-	"      part must be specified as partition name\n"
+	"    - set environment variable to the partition number using the partition UUID or name\n"
+	"      part must be specified as partition UUID or name\n"
 	"part name <interface> <dev> <part> <varname>\n"
 	"    - set environment variable to the partition name using the partition number\n"
 	"      part must be specified as partition number\n"