From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 2 Jun 2020 13:55:56 -0400 Subject: [PATCH] cmd: part: Add 'block' sub-command In-Reply-To: <20200601102026.51707-2-razvan.becheriu@gmail.com> References: <20200601102026.51707-1-razvan.becheriu@gmail.com> <20200601102026.51707-2-razvan.becheriu@gmail.com> Message-ID: <20200602175556.GP21630@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Jun 01, 2020 at 01:20:26PM +0300, razvan.becheriu at gmail.com wrote: > From: Razvan Becheriu > > 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 > --- > 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 \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 \n" > + " - set environment variable to the size of the partition block\n" > + " part can be either partition number or partition name\n" > "part number \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: