Re: [PATCH v3 1/4] riscv: dts: add OF_LIST handling to binman.dtsi

[E Shattow <e@freeshell.de>]

On 2/21/25 01:58, Heinrich Schuchardt wrote:
> Binman can automatically generate device-tree and configuration entries in
> the FIT image based on CONFIG_MULTI_DTB_FIT if the binman node includes the
> right sub-nodes.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
> v3:
> 	Use CONFIG_MULTI_DTB_FIT instead of CONFIG_OF_LIST to avoid
> 	build failure for QEMU defconfigs.
> v2:
> 	no change
> ---
>  arch/riscv/dts/binman.dtsi | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/riscv/dts/binman.dtsi b/arch/riscv/dts/binman.dtsi
> index 0405faca574..782ef037f7a 100644
> --- a/arch/riscv/dts/binman.dtsi
> +++ b/arch/riscv/dts/binman.dtsi
> @@ -82,8 +82,9 @@
>  					};
>  				};
>  
> -#ifndef CONFIG_OF_BOARD
> +#if !defined(CONFIG_OF_BOARD) || defined(CONFIG_MULTI_DTB_FIT)
>  				@fdt-SEQ {
> +					fit,operation = "gen-fdt-nodes";
>  					description = "NAME";
>  					type = "flat_dt";
>  					compression = "none";
> @@ -94,7 +95,7 @@
>  			configurations {
>  				default = "conf-1";
>  
> -#ifndef CONFIG_OF_BOARD
> +#if !defined(CONFIG_OF_BOARD) || defined(CONFIG_MULTI_DTB_FIT)
>  				@conf-SEQ {
>  #else
>  				conf-1 {
> @@ -115,7 +116,7 @@
>  #endif
>  #endif /* CONFIG_OPTEE */
>  
> -#ifndef CONFIG_OF_BOARD
> +#if !defined(CONFIG_OF_BOARD) || defined(CONFIG_MULTI_DTB_FIT)
>  					fdt = "fdt-SEQ";
>  #endif
>  				};

Reviewed-by: E Shattow <e@freeshell.de>