Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Sergey Matyukevich <geomatsi@gmail.com>,
	Alistair Francis <alistair@alistair23.me>,
	Gerome Burlats <gerome.burlats@smile.fr>,
	Jagan Teki <jagan@amarulasolutions.com>,
	Suniel Mahesh <sunil@amarulasolutions.com>,
	Giulio Benetti <giulio.benetti@benettiengineering.com>,
	buildroot@buildroot.org, Romain Naour <romain.naour@gmail.com>
Subject: Re: [Buildroot] [PATCH 01/16] board/aarch64-efi/genimage-efi: convert partition-type-uuid to single letter notation
Date: Mon, 7 Feb 2022 08:55:17 +0100	[thread overview]
Message-ID: <20220207085517.5c13c097@windsurf> (raw)
In-Reply-To: <20220206213657.GA457876@scaer>

Hello,

On Sun, 6 Feb 2022 22:36:57 +0100
"Yann E. MORIN" <yann.morin.1998@free.fr> wrote:

> >  	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.

And? Is that really important?

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

Why?

> 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

Yes, but how useful are they?

Unless these systemd-defined UUID are really relevant, I would prefer
that we use the shortcuts provided by genimage wherever possible, and
only rely on using the full UUID for special cases, such as the
board/sifive/hifive-unleashed/genimage_sdcard.cfg (PATCH 15/16).

This way, it would be pretty clear that we have the "common cases" (boot
partition, rootfs partition) and the "odd/special cases".

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2022-02-07  7:55 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-06 20:05 [Buildroot] [PATCH 00/16] Use single letter notation in genimage.cfg files for partition-type-uuid Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 01/16] board/aarch64-efi/genimage-efi: convert partition-type-uuid to single letter notation Giulio Benetti
2022-02-06 21:36   ` Yann E. MORIN
2022-02-07  7:55     ` Thomas Petazzoni via buildroot [this message]
2022-02-07 17:37       ` Yann E. MORIN
2022-02-13 15:06         ` Arnout Vandecappelle
2022-02-13 22:24           ` Giulio Benetti
2022-02-13 22:53             ` [Buildroot] [PATCH v2 00/16] Use single letter notation in genimage.cfg files for partition-type-uuid Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 01/16] board/aarch64-efi/genimage-efi: convert partition-type-uuid to single letter notation Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 02/16] board/beaglev/genimage-efi: remove partition-type-uuid since its default value is L(rootfs) Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 03/16] board/friendlyarm/nanopc-t4/genimage: convert partition-type-uuid to single letter notation Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 04/16] board/friendlyarm/nanopi-m4/genimage: " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 05/16] board/orangepi/orangepi-pc-plus/genimage: keep partition-type-uuid without "" for consistency Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 06/16] board/orangepi/orangepi-r1/genimage: fix partition-type-uuid for boot partition and remove " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 07/16] board/orangepi/orangepi-zero-plus2/genimage: keep partition-type-uuid without " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 08/16] board/pc/genimage-efi: convert partition-type-uuid to single letter notation Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 09/16] board/pine64/rockpro64/genimage: " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 10/16] board/qemu/aarch64-sbsa/genimage: " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 11/16] board/radxa/rockpi-n4/genimage: " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 12/16] board/radxa/rockpi-n10/genimage: " Giulio Benetti
2022-02-13 22:53               ` [Buildroot] [PATCH v2 13/16] board/radxa/rockpi-n8/genimage: " Giulio Benetti
2022-02-13 22:54             ` [Buildroot] [PATCH v2 14/16] board/sifive/hifive-unleashed/genimage_sdcard: remove partition-type-uuid since its default value is L(rootfs) Giulio Benetti
2022-02-13 22:54               ` [Buildroot] [PATCH v2 15/16] board/sifive/hifive-unleashed/genimage_sdcard: add comments to particular partition-type-uuid parameters Giulio Benetti
2022-02-13 22:54               ` [Buildroot] [PATCH v2 16/16] board/sifive/hifive-unleashed/genimage_spi-nor: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 02/16] board/beaglev/genimage-efi: convert partition-type-uuid to single letter notation Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 03/16] board/friendlyarm/nanopc-t4/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 04/16] board/friendlyarm/nanopi-m4/genimage: keep partition-type-uuid without "" for consistency Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 05/16] board/orangepi/orangepi-pc-plus/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 06/16] board/orangepi/orangepi-r1/genimage: fix partition-type-uuid for boot partition and remove " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 07/16] board/orangepi/orangepi-zero-plus2/genimage: keep partition-type-uuid without " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 08/16] board/pc/genimage-efi: convert partition-type-uuid to single letter notation Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 09/16] board/pine64/rockpro64/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 10/16] board/qemu/aarch64-sbsa/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 11/16] board/radxa/rockpi-n4/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 12/16] board/radxa/rockpi-n10/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 13/16] board/radxa/rockpi-n8/genimage: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 14/16] board/sifive/hifive-unleashed/genimage_sdcard: " Giulio Benetti
2022-02-06 20:05 ` [Buildroot] [PATCH 15/16] board/sifive/hifive-unleashed/genimage_sdcard: add comments to particular partition-type-uuid parameters Giulio Benetti
2022-07-22 20:41 ` [Buildroot] [PATCH 00/16] Use single letter notation in genimage.cfg files for partition-type-uuid Thomas Petazzoni via buildroot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220207085517.5c13c097@windsurf \
    --to=buildroot@buildroot.org \
    --cc=alistair@alistair23.me \
    --cc=geomatsi@gmail.com \
    --cc=gerome.burlats@smile.fr \
    --cc=giulio.benetti@benettiengineering.com \
    --cc=jagan@amarulasolutions.com \
    --cc=romain.naour@gmail.com \
    --cc=sunil@amarulasolutions.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=yann.morin.1998@free.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox