Hi Simon, On 2025-05-25 12:47, Simon Glass wrote: > Add an fdtmap so it is possible to look at the image with 'binman ls'. > > Signed-off-by: Simon Glass <s...@chromium.org> > Reviewed-by: Jonas Karlman <jo...@kwiboo.se> > --- > > (no changes since v3) > > Changes in v3: > - Add blank lines before the node > > arch/arm/dts/rockchip-u-boot.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/dts/rockchip-u-boot.dtsi > b/arch/arm/dts/rockchip-u-boot.dtsi > index cc2feed6464..8a37a9d8e7f 100644 > --- a/arch/arm/dts/rockchip-u-boot.dtsi > +++ b/arch/arm/dts/rockchip-u-boot.dtsi > @@ -192,6 +192,10 @@ > #endif > offset = <CONFIG_SPL_PAD_TO>; > }; > +#endif
This should not be added here as it breaks the build. It is reverted in a later patch in this series so your CI has not noticed the build error. Regards, Jonas > + > + fdtmap { > + }; > }; > > #ifdef CONFIG_ROCKCHIP_SPI_IMAGE > @@ -224,6 +228,9 @@ > /* Sync with u-boot,spl-payload-offset if present */ > offset = <CONFIG_SYS_SPI_U_BOOT_OFFS>; > }; > + > + fdtmap { > + }; > }; > #endif /* CONFIG_ROCKCHIP_SPI_IMAGE */ > };