Re: [PATCH v3 2/4] riscv: dts: no default configuration for MULTI_DTB_FIT

[E Shattow <e@freeshell.de>]

On 2/21/25 01:58, Heinrich Schuchardt wrote:
> JH7110 boards are currently the only use case for multi DTB FIT images
> on RISC-V.
> 
> Booting JH7110 systems with a VisionFive 2 device-tree used to kind of
> work without causing harm to the hardware. But there is no guarantee
> that this will hold true in future. So we should not rely on it.
> 
> Before the current patch series booting failed on unsupported boards due
> to the lack of a device-tree in the binman generated default configuration
> when reaching main U-Boot.
> 
> By not setting a default configuration booting will now fail on
> unsupported boards already in SPL. This allows SPL to
> continue with the next boot source for a possible recovery.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
> v3:
> 	new patch
> ---
>  arch/riscv/dts/binman.dtsi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/arch/riscv/dts/binman.dtsi b/arch/riscv/dts/binman.dtsi
> index 782ef037f7a..ceb916b74a7 100644
> --- a/arch/riscv/dts/binman.dtsi
> +++ b/arch/riscv/dts/binman.dtsi
> @@ -93,7 +93,10 @@
>  			};
>  
>  			configurations {
> +
> +#ifndef CONFIG_MULTI_DTB_FIT
>  				default = "conf-1";
> +#endif
>  
>  #if !defined(CONFIG_OF_BOARD) || defined(CONFIG_MULTI_DTB_FIT)
>  				@conf-SEQ {

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