Re: [PATCH v3 0/5] Add partition type GUID support for environment

[Sumit Garg <sumit.garg@kernel.org>]

On Mon, Apr 27, 2026 at 10:19:30PM +0530, Balaji Selvanathan wrote:
> 
> On 4/20/2026 7:07 PM, Sumit Garg wrote:
> > On Sun, Apr 19, 2026 at 03:54:02PM +0530, Balaji Selvanathan wrote:
> > > This series adds support for locating partitions using GPT partition
> > > type GUID instead of unique partition UUID. This enables the saveenv
> > > command to work with partitions identified by their type rather than
> > > unique identifiers, providing flexibility for systems where partition
> > > UUIDs may vary across devices but types remain constant.
> > Why don't we rather rely on partition name/label instead? That can be
> > common across platforms too. I would rather prefer a config fragment for
> > eg. UFS based environment or eMMC based environment and so on.
> > 
> > -Sumit
> 
> Hi Sumit,
> 
> Using type GUIDs as they offer following advantages: they are standardized
> across vendors (e.g., ChromeOS kernel GUID, Linux filesystem GUID) unlike
> arbitrary partition names that vary by OEM or firmware version, they are
> immutable and set at partition creation, making them more robust than
> easily-modified partition names.
> 

That's fair, let's at least document the partition name used for
environment in the defconfig such that it's easier for users to know
that.

-Sumit

> 
> > 
> > > Patch 1 adds part_get_info_by_type_guid() function support
> > > Patch 2 adds scsi_get_blk_by_type_guid() function support
> > > Patch 3 adds support for partition type GUID based environment
> > > Patch 4 enables partition type GUID configs for QCM6490 and QCS615
> > > Patch 5 adds unit test for the partition type GUID lookup functionality
> > > 
> > > Signed-off-by: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
> > > ---
> > > Changes in v3:
> > > - Addressed minor corrections in Patch 1 in part_get_info_by_type_guid function
> > > - Refactor env_scsi_get_part and env_scsi_load functions based
> > >    on the choice configs in Patch 3
> > > - Add unit test for the partition type GUID lookup functionality in Patch 5
> > > - Link to v2: https://lore.kernel.org/u-boot/20260109070912.4106466-1-balaji.selvanathan@oss.qualcomm.com/
> > > 
> > > Changes in v2:
> > > - Compute blk_find_max_devnum(UCLASS_SCSI) only once in scsi_get_blk_by_type_guid()
> > > - Introduce a Kconfig choice config to select between UUID-based
> > > and type GUID-based partition lookup methods.
> > > - Enable above new config in qcom_qcs615_defconfig and qcm6490_defconfig
> > > - Link to v1: https://lore.kernel.org/u-boot/20260108064947.3237361-1-balaji.selvanathan@oss.qualcomm.com/
> > > 
> > > ---
> > > Balaji Selvanathan (5):
> > >        disk: Add partition lookup by type GUID functionality
> > >        scsi: Add partition lookup by type GUID for SCSI devices
> > >        env: scsi: Add support for partition type GUID based environment
> > >        configs: Enable partition type GUID for QCM6490 and QCS615 boards
> > >        test: dm: Add partition type GUID lookup test
> > > 
> > >   configs/qcm6490_defconfig     |  5 +++++
> > >   configs/qcom_qcs615_defconfig |  5 +++++
> > >   disk/part.c                   | 37 +++++++++++++++++++++++++++++++
> > >   drivers/scsi/scsi-uclass.c    | 28 ++++++++++++++++++++++--
> > >   env/Kconfig                   | 51 ++++++++++++++++++++++++++++++++++++-------
> > >   env/scsi.c                    | 45 +++++++++++++++++++++++---------------
> > >   include/part.h                | 21 ++++++++++++++++++
> > >   include/scsi.h                | 11 ++++++++++
> > >   test/dm/part.c                | 48 ++++++++++++++++++++++++++++++++++++++++
> > >   9 files changed, 223 insertions(+), 28 deletions(-)
> > > ---
> > > base-commit: e3405917a1806971d9e72a94186b299f05581e1a
> > > change-id: 20260419-type-c6a48a1c273d
> > > 
> > > Best regards,
> > > -- 
> > > Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
> > >