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 */
>  };

Reply via email to