[PATCH v2 0/3] Restore support of short name for type UUID parameter

[Tom Rini]

On Fri, May 23, 2025 at 11:11:37AM +0200, Patrick Delaunay wrote:

> Fix and add documentation/tests for selection by string for known
> partition type GUID introduced by bcb41dcaefac ("uuid: add
> selection by string for known partition type GUID"):
> 
> - split list_guid for short name (used also for partiton
>   description with type parameter) and full name to display
>   information
> 
> - as the function are uuid_str_to_bin() / uuid_guid_get_str()
>   are no more under CONFIG_PARTITION_TYPE_GUID,  since commit
>   31ce367cd100 ("lib/uuid.c: change prototype of uuid_guid_get_str()")
>   and commit c1528f324c60 ("lib: compile uuid_guid_get_str if
>   CONFIG_LIB_UUID=y") move the content of array under EFI_PARTITION
>   and linker will remove it is not used it (in SPL)
> 
> - Add and fix documentation for gpt command
> 
> - Add test test_gpt_write_part_type to test "type=" parameters
> 
> This first patch solves an issue for the "system" shortcut for ESP,
> removed by commit d54e1004b8b1 ("lib/uuid.c: use unique name
> for PARTITION_SYSTEM_GUID") but used in 2 location (at least):
> 
> 1- board/samsung/e850-96/e850-96.env:10:
> 
> partitions=name=esp,start=512K,size=128M,bootable,type=system;
> partitions+=name=rootfs,size=-,bootable,type=linux
> 
> 2- arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c:1151
> 
> 			case PART_ESP:
> 				/* EFI System Partition */
> 				type_str = "system"
> ....
> 			offset += snprintf(buf + offset,
> 					   buflen - offset,
> 					   ",type=%s", type_str);

This leads to failure in CI:
https://source.denx.de/u-boot/u-boot/-/jobs/1151511

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250530/883c205d/attachment.sig>