Message ID | 20220727133534.126824-1-eballetbo@redhat.com |
---|---|
State | Changes Requested, archived |
Delegated to: | Tom Rini |
Headers | show |
Series | cmd: part: Add partition-related type command | expand |
On Wed, Jul 27, 2022 at 03:35:34PM +0200, Enric Balletbo i Serra wrote: > This implements the following command: > > part type mmc 0:1 > -> print partittion type UUID > 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 > 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(+) Seems reasonable. Please update some of the existing tests (I don't know if test/py/tests/test_gpt.py or test/py/tests/test_part.py or perhaps both would be easier) for v2.
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" > );
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" "part types\n" " - list supported partition table types" );
This implements the following command: part type mmc 0:1 -> print partittion type UUID 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 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(+)