Re: [PATCH v3 0/5] Add support of Android Boot Image version 2 and non-AB image

Hi Guillaume,

On mer., nov. 20, 2024 at 18:54, Guillaume LA ROQUE <[email protected]> wrote:

> Hi,
>
> unfortunately i don't find for now how i can fix issue in test.
>
> i don't really understand with when we launch test individually it's 
> working but if we launch test_ut test not working ,
> I see on android test for image version 4 to mmc7 in bootflow list detected.
>
> => ut bootstd bootflow_android_image_v4
> Test: bootflow_android_image_v4: bootflow.c
> scan_mmc_android_bootdev: mmc7
> order 0: mmc2
> order 1: mmc1
> order 2: mmc7
> order 3: <NULL>
> Enabled mmc mmc7 bootdev
>        ab_select_slot() ANDROID: Attempting slot a, tries remaining 7
>        ab_select_slot() ANDROID: Attempting slot a, tries remaining 7
> Older order: mmc2
> Showing all bootflows
> Seq  Method       State   Uclass    Part Name Filename
> ---  -----------  ------  --------  ---- ------------------------ 
> ----------------
>    0  extlinux     ready   mmc          1 mmc1.bootdev.part_1 
> /extlinux/extlinux.conf
>    1  android      ready   mmc          0 mmc7.bootdev.whole
>    2  android      ready   mmc          0 mmc7.bootdev.whole
> ---  -----------  ------  --------  ---- ------------------------ 
> ----------------
> (3 bootflows, 3 valid)
>
> if we just launch bootflow_android_image_v4 we have only a 
> mmc7.bootdev.whole and so test is ok.
>
> if someone have an idea or can give some idea of what i can try or check .
>
> thanks for helping.

I'll have a look

>
> Guillaume
> Le 20/11/2024 à 13:56, Guillaume LA ROQUE a écrit :
>> Hi,
>>
>> Le 19/11/2024 à 18:21, Mattijs Korpershoek a écrit :
>>> Hi Guillaume,
>>>
>>> On mar., nov. 19, 2024 at 15:16, Mattijs Korpershoek 
>>> <[email protected]> wrote:
>>>
>>>> Hi,
>>>>
>>>> On Tue, 19 Nov 2024 12:37:37 +0100, Guillaume La Roque wrote:
>>>>> Actually bootmethod android only support android boot image version 4
>>>>> and with AB image, some old platform wtill use android boot image
>>>>> version 2 with AB or without AB slot.
>>>>>
>>>>> This patchset add support of both version 2 and non-AB slot images.
>>>>> It's fixed in same time a boot issue seen on khadas vim3{l} board 
>>>>> with 16GB eMMC
>>>>>
>>>>> [...]
>>>> Thanks, Applied to 
>>>> https://source.denx.de/u-boot/custodians/u-boot-dfu (u-boot-dfu-next)
>>>>
>>>> [1/5] bootstd: android: add support of bootimage v2
>>>> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/dbe7eee992e90bbe84278db20d29159e3eedfe0d
>>>> [2/5] bootstd: android: add non-A/B image support
>>>> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/a85f2e0e7bd3665366ca2704d1142d3f16f22c36
>>>> [3/5] configs: khadas-vim3l_android{_ab}: move on bootmeth android
>>>> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/88547773b2df5e4cabbaec57a0f2e7f212dbf823
>>>> [4/5] configs: khadas-vim3_android{_ab}: move on bootmeth android
>>>> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/4a2e58717f2f33ef0dbb422026ef71938f2265aa
>>>> [5/5] bootstd: Add test for Android boot image v2
>>>> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/acaa7f35a33146f887948d34130229388280844a
>>> CI has found some issues when applying these series:
>>> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/pipelines/23452
>>>
>>> Could you have a look please?
>> i will check
>>>
>>> Thanks!
>>>
>>>> -- 
>>>> Mattijs
>>
>>