[PATCH] cmd: part: Add 'block' sub-command

Tom Rini
2020-06-02T17:55:56Z

On Mon, Jun 01, 2020 at 01:20:26PM +0300, razvan.becheriu at gmail.com wrote:
> From: Razvan Becheriu <razvan.becheriu@gmail.com>
>
> Add part block sub-command which returns block size.
>
> e.g.:
> part block mmc $mmcdev system_a system_a_index
>
> Signed-off-by: Razvan Becheriu <razvan.becheriu@gmail.com>
> ---
> cmd/part.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/cmd/part.c b/cmd/part.c
> index 5e4e45ca6d..d78d914e7a 100644
> --- a/cmd/part.c
> +++ b/cmd/part.c
> @@ -25,6 +25,7 @@
> enum cmd_part_info {
> CMD_PART_INFO_START = 0,
> CMD_PART_INFO_SIZE,
> + CMD_PART_INFO_BLOCK,
> CMD_PART_INFO_NUMBER
> };
>
> @@ -151,6 +152,9 @@ static int do_part_info(int argc, char * const argv[], enum cmd_part_info param)
> case CMD_PART_INFO_SIZE:
> snprintf(buf, sizeof(buf), LBAF, info.size);
> break;
> + case CMD_PART_INFO_BLOCK:
> + snprintf(buf, sizeof(buf), LBAF, info.blksz);
> + break;
> case CMD_PART_INFO_NUMBER:
> snprintf(buf, sizeof(buf), "0x%x", part);
> break;
> @@ -177,6 +181,11 @@ static int do_part_size(int argc, char * const argv[])
> return do_part_info(argc, argv, CMD_PART_INFO_SIZE);
> }
>
> +static int do_part_block(int argc, char * const argv[])
> +{
> + return do_part_info(argc, argv, CMD_PART_INFO_BLOCK);
> +}
> +
> static int do_part_number(int argc, char * const argv[])
> {
> return do_part_info(argc, argv, CMD_PART_INFO_NUMBER);
> @@ -195,6 +204,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], "block"))
> + return do_part_block(argc - 2, argv + 2);
> else if (!strcmp(argv[1], "number"))
> return do_part_number(argc - 2, argv + 2);
>
> @@ -219,6 +230,9 @@ U_BOOT_CMD(
> "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 block <interface> <dev> <part> <varname>\n"
> + " - set environment variable to the size of the partition block\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"

I believe you can solve the problems mentioned in the cover letter with
setexpr to do the conversion so nak on this patch, thanks.

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200602/efed1ccb/attachment.sig>