bdinfo: Drop the option to not use the generic 'bd' command

[u-boot.git] / cmd / part.c

diff --git a/cmd/part.c b/cmd/part.c
index 746bf40b2d0a8ad50c78b7d3e49d96ab7a074990..216f14bf5d7844cb5b247f7b87cebc165c351dba 100644 (file)
--- a/cmd/part.c
+++ b/cmd/part.c
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * Copyright (c) 2012, NVIDIA CORPORATION.  All rights reserved.
  *
@@ -12,21 +13,26 @@
  * (C) Copyright 2003 - 2004
  * Sysgo Real-Time Solutions, AG <www.elinos.com>
  * Pavel Bartusek <[email protected]>
- *
- * SPDX-License-Identifier:    GPL-2.0+
  */
 
 #include <common.h>
 #include <config.h>
 #include <command.h>
+#include <env.h>
 #include <part.h>
 #include <vsprintf.h>
 
-static int do_part_uuid(int argc, char * const argv[])
+enum cmd_part_info {
+       CMD_PART_INFO_START = 0,
+       CMD_PART_INFO_SIZE,
+       CMD_PART_INFO_NUMBER
+};
+
+static int do_part_uuid(int argc, char *const argv[])
 {
        int part;
        struct blk_desc *dev_desc;
-       disk_partition_t info;
+       struct disk_partition info;
 
        if (argc < 2)
                return CMD_RET_USAGE;
@@ -45,7 +51,7 @@ static int do_part_uuid(int argc, char * const argv[])
        return 0;
 }
 
-static int do_part_list(int argc, char * const argv[])
+static int do_part_list(int argc, char *const argv[])
 {
        int ret;
        struct blk_desc *desc;
@@ -84,9 +90,9 @@ static int do_part_list(int argc, char * const argv[])
        if (var != NULL) {
                int p;
                char str[512] = { '\0', };
-               disk_partition_t info;
+               struct disk_partition info;
 
-               for (p = 1; p < 128; p++) {
+               for (p = 1; p < MAX_SEARCH_PARTITIONS; p++) {
                        char t[5];
                        int r = part_get_info(desc, p, &info);
 
@@ -108,11 +114,12 @@ static int do_part_list(int argc, char * const argv[])
        return 0;
 }
 
-static int do_part_start(int argc, char * const argv[])
+static int do_part_info(int argc, char *const argv[], enum cmd_part_info param)
 {
        struct blk_desc *desc;
-       disk_partition_t info;
+       struct disk_partition info;
        char buf[512] = { 0 };
+       char *endp;
        int part;
        int err;
        int ret;
@@ -122,17 +129,35 @@ static int do_part_start(int argc, char * const argv[])
        if (argc > 4)
                return CMD_RET_USAGE;
 
-       part = simple_strtoul(argv[2], NULL, 0);
-
        ret = blk_get_device_by_str(argv[0], argv[1], &desc);
        if (ret < 0)
                return 1;
 
-       err = part_get_info(desc, part, &info);
-       if (err)
-               return 1;
+       part = simple_strtoul(argv[2], &endp, 0);
+       if (*endp == '\0') {
+               err = part_get_info(desc, part, &info);
+               if (err)
+                       return 1;
+       } else {
+               part = part_get_info_by_name(desc, argv[2], &info);
+               if (part == -1)
+                       return 1;
+       }
 
-       snprintf(buf, sizeof(buf), LBAF, info.start);
+       switch (param) {
+       case CMD_PART_INFO_START:
+               snprintf(buf, sizeof(buf), LBAF, info.start);
+               break;
+       case CMD_PART_INFO_SIZE:
+               snprintf(buf, sizeof(buf), LBAF, info.size);
+               break;
+       case CMD_PART_INFO_NUMBER:
+               snprintf(buf, sizeof(buf), "0x%x", part);
+               break;
+       default:
+               printf("** Unknown cmd_part_info value: %d\n", param);
+               return 1;
+       }
 
        if (argc > 3)
                env_set(argv[3], buf);
@@ -142,41 +167,23 @@ static int do_part_start(int argc, char * const argv[])
        return 0;
 }
 
-static int do_part_size(int argc, char * const argv[])
+static int do_part_start(int argc, char *const argv[])
 {
-       struct blk_desc *desc;
-       disk_partition_t info;
-       char buf[512] = { 0 };
-       int part;
-       int err;
-       int ret;
-
-       if (argc < 3)
-               return CMD_RET_USAGE;
-       if (argc > 4)
-               return CMD_RET_USAGE;
-
-       part = simple_strtoul(argv[2], NULL, 0);
-
-       ret = blk_get_device_by_str(argv[0], argv[1], &desc);
-       if (ret < 0)
-               return 1;
-
-       err = part_get_info(desc, part, &info);
-       if (err)
-               return 1;
-
-       snprintf(buf, sizeof(buf), LBAF, info.size);
+       return do_part_info(argc, argv, CMD_PART_INFO_START);
+}
 
-       if (argc > 3)
-               env_set(argv[3], buf);
-       else
-               printf("%s\n", buf);
+static int do_part_size(int argc, char *const argv[])
+{
+       return do_part_info(argc, argv, CMD_PART_INFO_SIZE);
+}
 
-       return 0;
+static int do_part_number(int argc, char *const argv[])
+{
+       return do_part_info(argc, argv, CMD_PART_INFO_NUMBER);
 }
 
-static int do_part(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_part(struct cmd_tbl *cmdtp, int flag, int argc,
+                  char *const argv[])
 {
        if (argc < 2)
                return CMD_RET_USAGE;
@@ -189,6 +196,8 @@ static int do_part(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
                return do_part_start(argc - 2, argv + 2);
        else if (!strcmp(argv[1], "size"))
                return do_part_size(argc - 2, argv + 2);
+       else if (!strcmp(argv[1], "number"))
+               return do_part_number(argc - 2, argv + 2);
 
        return CMD_RET_USAGE;
 }
@@ -207,6 +216,11 @@ 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 size <interface> <dev> <part> <varname>\n"
-       "    - set environment variable to the size of the partition (in blocks)"
+       "    - set environment variable to the size of the partition (in blocks)\n"
+       "      part can be either partition number or partition 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"
 );