On Fri, Feb 17, 2023 at 09:37:32PM +0100, Ahmad Fatoum wrote: > We currently require IMD_OF_USED to get the DT compatible into the > barebox image data meta section. As the PBL is already referencing the > DT to pass it along to barebox proper, let's add a reference from the DT > to the image data section to make IMD_OF_USED unnecessary. > > Suggested-by: Antony Pavlov <antonynpavlov@xxxxxxxxx> > Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> > --- > scripts/gen-dtb-s | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/scripts/gen-dtb-s b/scripts/gen-dtb-s > index c5c46a4d8675..9cc2e7a72cc2 100755 > --- a/scripts/gen-dtb-s > +++ b/scripts/gen-dtb-s > @@ -49,6 +49,9 @@ echo "__dtb_${name}_start:" > echo ".incbin \"$dtb\"" > echo "__dtb_${name}_end:" > echo ".global __dtb_${name}_end" > +if [ "$imd" = "y" ]; then > + echo ".word __barebox_imd_OF_${name}" > +fi > echo ".balign STRUCT_ALIGNMENT" > > compressed=$(${CONFIG_SHELL} "${srctree}/scripts/file-size.sh" $dtb.z) > @@ -66,4 +69,7 @@ echo ".incbin \"$dtb.z\"" > echo "__dtb_z_${name}_end:" > echo ".global __dtb_z_${name}_end" > echo ".balign STRUCT_ALIGNMENT" > +if [ "$imd" = "y" ]; then > + echo ".word __barebox_imd_OF_${name}" > +fi > echo "#endif" This results in several linker errors here when compiling imx_v8_defconfig: /opt/OSELAS.Toolchain-2022.10.0/aarch64-v8a-linux-gnu/gcc-12.2.1-clang-15.0.2-glibc-2.36-binutils-2.39-kernel-6.0.5-sanitized/bin/aarch64-v8a-linux-gnu-ld: arch/arm/dts/imx8mm-innocomm-wb15-evk.dtb.pbl.o: relocation R_AARCH64_ABS32 against `__barebox_imd_OF_imx8mm_innocomm_wb15_evk' can not be used when making a shared object Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |