<div dir="ltr"><div dir="ltr">Hello Patrice,<br><br></div><div dir="ltr">Sorry, but I can't log in. My Custodian account has expired, and that might be the reason. I need to ask Tom Rini to reactivate it. <br>In the meantime, can you please give me some information about the error?</div><br>Thanks and regards,<br>Dario<div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Apr 29, 2026 at 11:14 AM Patrice CHOTARD <<a href="mailto:patrice.chotard@foss.st.com">patrice.chotard@foss.st.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On 4/26/26 17:46, Dario Binacchi wrote:<br>
> This series, the mainline version of [1], provides the necessary changes<br>
> in U-Boot to support the metadata-driven A/B update flow for STM32MP25<br>
> boards. It implements the logic required to dynamically select the boot<br>
> bank based on FWU metadata.<br>
> <br>
> These changes are designed to be backward compatible while providing<br>
> the necessary infrastructure for the A/B scheme.<br>
> <br>
> The series has been tested on the STM32MP257F-EV1 board.<br>
> <br>
> Additionally, some patches in the series propose changes with<br>
> general-purpose utility:<br>
> <br>
> 1/7 lib: uuid: add partition type GUID for extended bootloader<br>
> 4/7 cmd: part: support lookup by UUID in 'part number'<br>
> 6/7 fwu: add helper to get image GUID by type and bank index<br>
> <br>
> [1] <a href="https://github.com/STMicroelectronics/u-boot/pull/6" rel="noreferrer" target="_blank">https://github.com/STMicroelectronics/u-boot/pull/6</a><br>
> <br>
> Changes in v7:<br>
> - Add Reviewed-by of Simon Glass for patc 7/8 of "test: dm:<br>
> fwu_mdata: add test for fwu_mdata_get_image_guid".<br>
> <br>
> Changes in v6:<br>
> - Add Reviewed-by of Simon Glass to patch 3/8 "test: cmd: add unit<br>
> tests for part command"<br>
> <br>
> Changes in v5:<br>
> - Update doc/usage/cmd/part.rst to reflect the changes related to the<br>
> part command.<br>
> - Add Acked-by of Ilias Apalodimas to patch 6/8 "fwu: add helper to get<br>
> image GUID by type and bank index"<br>
> <br>
> Changes in v4:<br>
> - Place the part command tests in the cmd suite rather than the dm<br>
> suite.<br>
> - Order the include files according the U-Boot coding style.<br>
> - Add patch 7/8 test: dm: fwu_mdata: add test for fwu_mdata_get_image_guid<br>
> <br>
> Changes in v3:<br>
> - Wrap lines exceeding 80 columns in test/cmd/part.c file.<br>
> - Combine run_command() and ut_asserteq() -> ut_asserteq(1, run_command(<br>
> - Add kerneldoc comment for fwu_mdata_get_image_guid() above its declaration<br>
> in fwu.h.<br>
> - Add log_warning() messages to fwu_platform_hook() to catch inconsistent<br>
> FWU metadata (boot GUID found but root GUID missing or viceversa), as<br>
> suggested by Simon Glass.<br>
> <br>
> Changes in v2:<br>
> - Add links to the XBOOTLDR specification in the commit message of patch<br>
> 1/7 lib: uuid: add partition type GUID for extended bootloader.<br>
> - Update help for 'part start', 'part size' to mention UUID.<br>
> - Add kerneldoc comment for fwu_mdata_get_image_guid().<br>
> - Pass efi_guid_t by pointer in fwu_mdata_get_image_guid().<br>
> - Add patches:<br>
> 5/7 test: cmd: part: add UUID lookup tests<br>
> 3/7 test: cmd: add unit tests for part command<br>
> 2/7 test: dm: part: add test for part_get_info_by_uuid<br>
> <br>
> Dario Binacchi (8):<br>
> lib: uuid: add partition type GUID for extended bootloader<br>
> test: dm: part: add test for part_get_info_by_uuid<br>
> test: cmd: add unit tests for part command<br>
> cmd: part: support lookup by UUID<br>
> test: cmd: part: add UUID lookup tests<br>
> fwu: add helper to get image GUID by type and bank index<br>
> test: dm: fwu_mdata: add test for fwu_mdata_get_image_guid<br>
> board: st: stm32mp25: support dynamic A/B bank bootup<br>
> <br>
> board/st/stm32mp2/stm32mp2.c | 32 ++++<br>
> cmd/part.c | 12 +-<br>
> doc/README.gpt | 2 +<br>
> doc/usage/cmd/part.rst | 6 +-<br>
> include/configs/stm32mp25_st_common.h | 15 ++<br>
> include/fwu.h | 11 ++<br>
> include/part_efi.h | 3 +<br>
> lib/fwu_updates/fwu.c | 33 ++++<br>
> lib/uuid.c | 1 +<br>
> test/cmd/Makefile | 1 +<br>
> test/cmd/part.c | 209 ++++++++++++++++++++++++++<br>
> test/dm/fwu_mdata.c | 48 ++++++<br>
> test/dm/part.c | 53 +++++++<br>
> 13 files changed, 419 insertions(+), 7 deletions(-)<br>
> create mode 100644 test/cmd/part.c<br>
> <br>
<br>
<br>
<br>
Hi Dario<br>
<br>
I am preparing a pull request for STM32 including this series.<br>
Unfortunately, there is an error linked to tests test/cmd/part.c <br>
<br>
You can use my temporary pull request tag u-boot-stm32-20260429<br>
<br>
You will find the error logs here encountered with qemu test : <a href="https://source.denx.de/u-boot/custodians/u-boot-stm/-/pipelines/29932" rel="noreferrer" target="_blank">https://source.denx.de/u-boot/custodians/u-boot-stm/-/pipelines/29932</a><br>
<br>
Thanks<br>
Patrice<br>
<br>
</blockquote></div><div><br clear="all"></div><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><font color="#888888"><font color="#000000" face="Times New Roman"><span style="font-size:14px"><b>Dario Binacchi</b></span></font><br style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:medium"><span style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:12px">Senior Embedded Software Engineer<br></span></font><span style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:12px">M. +39 328 0625246</span><font color="#888888"><span style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:12px"><br><a href="mailto:dario.binacchi@amarulasolutions.com" style="color:rgb(17,85,204)" target="_blank">dario.binacchi@amarulasolutions.com</a></span><br style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:medium"><span style="font-family:"Times New Roman";font-size:12px;color:rgb(241,194,50)">―――――――――――――――</span><br style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:medium"><span style="color:rgb(0,0,0);font-family:"Times New Roman";font-size:12px"><span style="font-weight:700">Amarula Solutions SRL</span><br> Via Felice Cavallotti 25D, 41012 Carpi, MO, IT<br><a href="mailto:info@amarulasolutions.com" style="color:rgb(17,85,204)" target="_blank">info@amarulasolutions.com</a><br><a href="http://www.amarulasolutions.com/" style="color:rgb(17,85,204)" target="_blank">www.amarulasolutions.com</a></span></font></div></div></div></div>