From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@konsulko.com>
Date: Tue, 20 Oct 2015 14:02:03 -0400
Subject: [U-Boot] [PATCH 2/3] gpt: add optional parameter guid in gpt
 command
In-Reply-To: <CAKOfRZxobcuWy-co8u4=qUN1Yi1MQkUcPJ3ha0c-ZZ=3kCznqQ@mail.gmail.com>
References: <CAKOfRZw-o-NeR6CbBrjLt9a+VOp2VJXgXfPZRq3wD9wBTVaboQ@mail.gmail.com>
 <CAL_JsqK6t0qxKFdb7+B0o330u05rQ9avOJz8jJ5841PHr6MCkw@mail.gmail.com>
 <20151015214648.GE23893@bill-the-cat>
 <CAKOfRZxobcuWy-co8u4=qUN1Yi1MQkUcPJ3ha0c-ZZ=3kCznqQ@mail.gmail.com>
Message-ID: <20151020180203.GP23893@bill-the-cat>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Mon, Oct 19, 2015 at 04:59:28PM +0200, Patrick Delaunay wrote:
> 2015-10-15 23:46 GMT+02:00 Tom Rini <trini@konsulko.com>:
> 
> > On Thu, Oct 15, 2015 at 03:58:24PM -0500, Rob Herring wrote:
> > > On Tue, Oct 13, 2015 at 9:23 AM, Patrick Delaunay
> > > <patrick.delaunay73@gmail.com> wrote:
> > > > code under flag CONFIG_PARTITION_TYPE_GUID
> > > > add parameter guid to select partition type guid
> > > >
> > > > example of use with gpt command :
> > > >
> > > >   partitions = uuid_disk=${uuid_gpt_disk};name=boot,start=0x4400,
> > > >   size=0x6bc00,uuid=${uuid_gpt_boot};name=root,start=0x70000,
> > > >   size=0x7538ba00,uuid=${uuid_gpt_root},
> > > >   guid=0fc63daf-8483-4772-8e79-3d69d8477de4;
> > >
> > > The mixture of UUID and GUID is confusing. What we want are the
> > > GUIDs/UUIDs for unique ID and the partition type. I would just call
> > > the partition type "type". This would allow the same format to be used
> > > for MBR partitions if someone wanted to do support for that.
> >
> > ... and if someone wants to add MBR support, there's enough use cases
> > for it (mainly around flashing/factory stuff) that I wouldn't object.
> > So yes, lets do this with that kind of thing in mind.
> >
> 
> 
> Hi,
> 
> if you prefer, I can modify the parameter name to "type" as proposed in a
> version 2 of the patchset.
> 
> partitions =
>     uuid_disk=${uuid_gpt_disk}; \
>     name=boot,start=0x4400,size=0x6bc00,uuid=${uuid_gpt_boot}; \
> 
> name=root,start=0x70000,size=0x7538ba00,uuid=${uuid_gpt_root},type=0fc63daf-8483-4772-8e79-3d69d8477de4;
> 
> and to be coherent, I will also modify the field in the struct
> disk_partition_t to type_guid
> 
> +#ifdef CONFIG_PARTITION_TYPE_GUID
> +    char    type_guid[37];    /* type GUID as string, if exists    */
> +#endif
>  } disk_partition_t;

Sounds like a plan, thanks!

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