[U-Boot] [PATCH 1/3] GPT: add accessor function for disk GUID

Tom Rini trini at konsulko.com
Fri May 26 12:38:59 UTC 2017


On Sat, May 20, 2017 at 07:27:53PM -0700, alison at peloton-tech.com wrote:

> From: Alison Chaiken <alison at she-devel.com>
> 
> In order to read the GPT, modify the partition name strings, and then
> write out a new GPT, the disk GUID is needed.  While there is an
> existing accessor for the partition UUIDs, there is none yet for the
> disk GUID.
> 
> Signed-off-by: Alison Chaiken <alison at peloton-tech.com>

Overall looks good, a few style things:

> +	if (namestr != NULL)
> +		setenv(namestr, disk_guid);

You can just if (namestr) here

[snip]
> +	} else if (strcmp(argv[1], "guid") == 0) {
> +		if (argc == 5) strcpy(varname, argv[4]);

Go ahead and do this on multiple lines please.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170526/649a855f/attachment.sig>


More information about the U-Boot mailing list