On Mon, Jun 15, 2015 at 08:59:47AM -0600, Stephen Warren wrote:
> On 06/13/2015 02:38 AM, Paul Kocialkowski wrote:
> >This introduces the part start and part size sub-commands. The purpose of 
> >these
> >is to store the start block and size of a partition in a variable, given the
> >device and partition number.
> >
> >This allows reading raw data that fits a single partition more easily.
> >For instance, this could be used to figure out the start block and size of a
> >kernel partition when a partition table is present, given the partition 
> >number.
> 
> Patches 1 and 2,
> Acked-by: Stephen Warren <swar...@nvidia.com>
> 
> One minor comment here though:
> 
> >diff --git a/common/cmd_part.c b/common/cmd_part.c
> 
> >+static int do_part_start(int argc, char * const argv[])
> 
> >+    if (argc < 4)
> >+            return CMD_RET_USAGE;
> ...
> >+    snprintf(buf, sizeof(buf), "0x%lx", info.start);
> >+    setenv(argv[3], buf);
> 
> It would be nice if the variable name parameter to this command (and
> part size) were optional, just like it is in "part uuid". This would
> make it simpler to run the command interactively while
> experimenting.

Yes.  Also numbers are hex by default in U-Boot so we don't need the 0x
here.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to