[PATCH v2 0/7] mkimage: Detect FIT image load address overlaps and fix related test/DTS issues

[E Shattow]

Hi Aristo,

On 9/24/25 04:43, Nishanth Menon wrote:
> On 06:37-20250924, Nishanth Menon wrote:
>> On 10:59-20250914, Aristo Chen wrote:
>>> This patch series enhances FIT image robustness by adding **memory
>>> region overlap detection** to `mkimage` and fixing existing overlaps
>>> in DTS files and `binman` tests.
> 
> [...]
>>>
>>
>> Looks like i see a build regression in linux-next after this series.
> 
> I fat fingered that one.. sorry, I meant u-boot next.
> 
> 
> Fails at commit 4d84fa1261eb, last pass was on commit d81c1118580f
> 
>>
>> https://gist.github.com/nmenon/b2fc9e7680cc296062c7dced94105f76
> 
> I  believe there are outstanding comments on V1 that have'nt been
> addressed either. Can we revert/drop this series for now while the
> comments are addressed?
> 

Similar to Nishanth, I am seeing a build regression, itb.map:

ImagePos    Offset      Size  Name
<none>      00000000  <none>    itb
<none>      00000000  <none>    fit
<none>      00000000  000c1698  uboot
<none>      00000000  000c1698  u-boot-nodtb
<none>      000c1698  00202128  opensbi
<none>      00000000  00202128  opensbi

Yields this error:
...
 MKIMAGE fit-dtb.blob
Warning: not able to get `load` of node 'fdt-1'
Warning: not able to get `load` of node 'fdt-2'
Warning: not able to get `load` of node 'fdt-3'
Warning: not able to get `load` of node 'fdt-4'
Warning: not able to get `load` of node 'fdt-5'
Warning: not able to get `load` of node 'fdt-6'
Warning: not able to get `load` of node 'fdt-7'
  CAT     u-boot-fit-dtb.bin
...
  MKIMAGE u-boot.img
Warning: not able to get `load` of node 'fdt-1'
Warning: not able to get `load` of node 'fdt-2'
Warning: not able to get `load` of node 'fdt-3'
Warning: not able to get `load` of node 'fdt-4'
Warning: not able to get `load` of node 'fdt-5'
Warning: not able to get `load` of node 'fdt-6'
Warning: not able to get `load` of node 'fdt-7'
  COPY    u-boot.dtb
  MKIMAGE u-boot-dtb.img
Warning: not able to get `load` of node 'fdt-1'
Warning: not able to get `load` of node 'fdt-2'
Warning: not able to get `load` of node 'fdt-3'
Warning: not able to get `load` of node 'fdt-4'
Warning: not able to get `load` of node 'fdt-5'
Warning: not able to get `load` of node 'fdt-6'
Warning: not able to get `load` of node 'fdt-7'
  BINMAN  .binman_stamp
Wrote map file './itb.map' to show errors
binman: Error 1 running 'mkimage -t -F ./itb.fit.fit': Warning: not able
to get `load` of node 'fdt-1'
[Config: conf-1] Error: Overlap detected:
  - uboot: [0x40200000 - 0x402c1698]
  - opensbi: [0x40000000 - 0x40202128]

make[1]: *** [/tmp/u-boot.2.git/Makefile:1339: .binman_stamp] Error 1
make[1]: Leaving directory '/home/es/build/u-boot'
make: *** [Makefile:198: sub-make] Error 2
make: Leaving directory '/tmp/u-boot.2.git'

If you would like to reproduce the issue, use config
starfive_visionfive2_defconfig and pass to OPENSBI env variable the path
of opensbi object 'fw_dynamic.bin' built using PLATFORM=generic, with
the ordinary gcc riscv64 toolchain. The starfive visionfive2 board
target in U-Boot supports multiple vendors and boards selected at
runtime and is an example of a more complex FIT usage that you may be
interested in for comparison and testing.

Best regards,

-E