Hi 在 2025年11月3日週一 20:19，Mattijs Korpershoek <[email protected]> 寫道：
> > Hi George, > On Fri, Oct 31, 2025 at 16:10, Mattijs Korpershoek < > [email protected]> wrote: > > > Hi, > > > > On Tue, 21 Oct 2025 00:36:33 +0800, George Chan wrote: > >> If target u-boot img do not support androidboot v3 or greater, > >> abootimg might not be necessary. > >> > >> aarch64-linux-gnu-ld.bfd: boot/bootmeth_android.o: in function > `boot_android_normal': > >> > /home/user/sources/u-boot-next/boot/bootmeth_android.c:541:(.text.boot_android_normal+0xd0): > undefined reference to `set_avendor_bootimg_addr' > >> aarch64-linux-gnu-ld.bfd: > /home/user/sources/u-boot-next/boot/bootmeth_android.c:543:(.text.boot_android_normal+0xd8): > undefined reference to `set_abootimg_addr' > >> Segmentation fault (core dumped) > >> > >> [...] > > > > Thanks, Applied to https://source.denx.de/u-boot/custodians/u-boot-dfu > (u-boot-dfu) > > > > [1/1] boot: bootmeth_android: Conditionally dependent on abootimg > > > https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/4b493b9664390e16f61a6409ec60676df7759c7b > > This patch causes CI to fail: > https://source.denx.de/u-boot/custodians/u-boot-dfu/-/jobs/1283315 > > """ > building current source for 1 boards (1 thread, 8 jobs per thread) > sandbox: + sandbox > +cmd/abootimg.c:26:6: error: redefinition of 'set_abootimg_addr' > + 26 | void set_abootimg_addr(ulong addr) > + | ^ > +include/image.h:2042:13: note: previous definition is here > + 2042 | void __weak set_abootimg_addr(ulong addr) {} > + | ^ > """ > > This can also be reproduced locally by building sandbox: > > $ make sandbox_defconfig > $ make > > > Can you have a look please? I will have to drop this from my branch > until the build error is resolved. > > If you find a fix, please send a v2 of this patch with the fix included. > Thx for testing and report. I am sorry that not available for some time so please drop this patch or find volunteer to follow this up. Regards, George > Thanks > Mattijs > > > > > -- > > Mattijs >
