Re: [Buildroot] [PATCH 01/16] board/aarch64-efi/genimage-efi: convert partition-type-uuid to single letter notation

From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Giulio Benetti <giulio.benetti@benettiengineering.com>
Cc: Sergey Matyukevich <geomatsi@gmail.com>,
	Alistair Francis <alistair@alistair23.me>,
	Gerome Burlats <gerome.burlats@smile.fr>,
	Jagan Teki <jagan@amarulasolutions.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Suniel Mahesh <sunil@amarulasolutions.com>,
	Romain Naour <romain.naour@gmail.com>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 01/16] board/aarch64-efi/genimage-efi: convert partition-type-uuid to single letter notation
Date: Sun, 6 Feb 2022 22:36:57 +0100	[thread overview]
Message-ID: <20220206213657.GA457876@scaer> (raw)
In-Reply-To: <20220206200532.2758493-2-giulio.benetti@benettiengineering.com>

Giulio, All,

FTR, a little recap of what we discussed on IRC, see below

On 2022-02-06 21:05 +0100, Giulio Benetti spake thusly:
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>  board/aarch64-efi/genimage-efi.cfg | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/board/aarch64-efi/genimage-efi.cfg b/board/aarch64-efi/genimage-efi.cfg
> index 87e85816bc..7fbe13d390 100644
> --- a/board/aarch64-efi/genimage-efi.cfg
> +++ b/board/aarch64-efi/genimage-efi.cfg
> @@ -18,13 +18,13 @@ image disk.img {
>  
>  	partition boot {
>  		image = "efi-part.vfat"
> -		partition-type-uuid = c12a7328-f81f-11d2-ba4b-00a0c93ec93b
> +		partition-type-uuid = U
>  		offset = 32K
>  		bootable = true
>  	}
>  
>  	partition root {
> -		partition-type-uuid = b921b045-1df0-41c3-af44-4c6f280d3fae
> +		partition-type-uuid = L

As we discussed on IRC, this is not the same thing. genimage will extend
the 'L' shorthand to the full UUID 0fc63daf-8483-4772-8e79-3d69d8477de4,
which is different.

So I don't think uaing the shorthands are correct.

The UUID assignments are available on Wikipedia:
    https://en.wikipedia.org/wiki/GUID_Partition_Table#Partition_type_GUIDs

But the systemd docuemntation has extra UUID definitions:
    https://man.archlinux.org/man/systemd-gpt-auto-generator.8

And finally, if neither resources has a UUID for a specific
architecture, we can still use the 'generic' "Linux filesystem data"
UUID 0FC63DAF-8483-4772-8E79-3D69D8477DE4.

Regards,
Yann E. MORIN.

>  		image = "rootfs.ext2"
>  	}
>  }
> -- 
> 2.25.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot