On 7/27/22 15:35, Enric Balletbo i Serra wrote:

This implements the following command:

 part type mmc 0:1
   -> print partittion type UUID

%s/partittion/partition/

 part type mmc 0:1 uuid
   -> set environment variable to partition type UUID

"part type" can be useful when writing a bootcmd which searches for a

Does such a command exist? Do you plan to implement such a command?

specific partition type to enable automatic discovery of partitions and their intended usage or mount point.

Signed-off-by: Enric Balletbo i Serra eballetbo@redhat.com

cmd/part.c | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+)

diff --git a/cmd/part.c b/cmd/part.c index 9d419c967c..26b0cc84fe 100644 --- a/cmd/part.c +++ b/cmd/part.c @@ -182,6 +182,29 @@ static int do_part_number(int argc, char *const argv[]) return do_part_info(argc, argv, CMD_PART_INFO_NUMBER); }

+static int do_part_type(int argc, char *const argv[]) +{

• int part;
• struct blk_desc *dev_desc;
• struct disk_partition info;
• if (argc < 2)

• return CMD_RET_USAGE;
  

• if (argc > 3)

• return CMD_RET_USAGE;
  

• part = blk_get_device_part_str(argv[0], argv[1], &dev_desc, &info, 0);
• if (part < 0)

• return 1;
  

• if (argc > 2)

• env_set(argv[2], info.type_guid);
  

• else

• printf("%s\n", info.type_guid);
  

• return 0;

+}

• static int do_part_types(int argc, char * const argv[]) { struct part_driver *drv = ll_entry_start(struct part_driver,

@@ -218,6 +241,8 @@ static int do_part(struct cmd_tbl *cmdtp, int flag, int argc, return do_part_size(argc - 2, argv + 2); else if (!strcmp(argv[1], "number")) return do_part_number(argc - 2, argv + 2);

• else if (!strcmp(argv[1], "type"))

• return do_part_type(argc - 2, argv + 2);
  

  else if (!strcmp(argv[1], "types")) return do_part_types(argc - 2, argv + 2); return CMD_RET_USAGE;

@@ -244,6 +269,10 @@ U_BOOT_CMD( "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"

• "part type <interface> <dev>:<part>\n"
• " - print partition type\n"
• "part type <interface> <dev>:<part> <varname>\n"
• " - set environment variable to partition type\n"

For each command we want a man-page in doc/usage/cmd. I miss a corresponding change in /doc.

Best regards

Heinrich

"part types\n" " - list supported partition table types" );