Re: [RESEND PATCH 1/2] config_distro_bootcmd: make possible to substitute 'part list' in distro_bootcmd

[Patrice CHOTARD <patrice.chotard@foss.st.com>]

On 4/11/25 09:15, Patrice CHOTARD wrote:
> 
> 
> On 3/27/25 15:38, Pascal Zimmermann wrote:
>> Make it possible to substitute the 'part list' command inside
>> 'scan_dev_for_boot_part' with a custom board specific implementation.
>>
>> For this the new define 'SCAN_DEV_FOR_BOOT_PARTS' is introduced.
>>
>> Signed-off-by: Pascal Zimmermann <pzimmermann@dh-electronics.com>
>> ---
>> Cc: Tom Rini <trini@konsulko.com>
>> Cc: Simon Glass <sjg@chromium.org>
>> Cc: u-boot@lists.denx.de
>> Cc: Marek Vasut <marex@denx.de>
>> Cc: u-boot@dh-electronics.com
>> ---
>>  include/config_distro_bootcmd.h | 7 ++++++-
>>  1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/include/config_distro_bootcmd.h b/include/config_distro_bootcmd.h
>> index 0a4e4b8ff85..df0dc8cfdee 100644
>> --- a/include/config_distro_bootcmd.h
>> +++ b/include/config_distro_bootcmd.h
>> @@ -194,6 +194,11 @@
>>  #define SCAN_DEV_FOR_EFI
>>  #endif
>>  
>> +#ifndef SCAN_DEV_FOR_BOOT_PARTS
>> +#define SCAN_DEV_FOR_BOOT_PARTS \
>> +	"part list ${devtype} ${devnum} -bootable devplist; "
>> +#endif
>> +
>>  #ifdef CONFIG_SATA
>>  #define BOOTENV_SHARED_SATA	BOOTENV_SHARED_BLKDEV(sata)
>>  #define BOOTENV_DEV_SATA	BOOTENV_DEV_BLKDEV
>> @@ -538,7 +543,7 @@
>>  		"\0"                                                      \
>>  	\
>>  	"scan_dev_for_boot_part="                                         \
>> -		"part list ${devtype} ${devnum} -bootable devplist; "     \
>> +		SCAN_DEV_FOR_BOOT_PARTS                                   \
>>  		"env exists devplist || setenv devplist 1; "              \
>>  		"for distro_bootpart in ${devplist}; do "                 \
>>  			"if fstype ${devtype} "                           \
> 
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Hi Pascal

Applied to u-boot-stm32/master

Thanks
Patrice


> 
> Thanks
> Patrice