Hi Randoph, On Fri, 6 Oct 2023 at 05:26, Randolph <rando...@andestech.com> wrote: > > Introduce common Kconfig symbol for riscv architecture > This symbol SPL_LOAD_FIT_CONFIG for binman itb layout selection > Default is using binman.dtsi > > Signed-off-by: Randolph <rando...@andestech.com> > --- > arch/riscv/Kconfig | 7 +++++++ > arch/riscv/dts/ae350-u-boot.dtsi | 1 + > arch/riscv/dts/ae350_32.dts | 1 - > arch/riscv/dts/ae350_64.dts | 1 - > 4 files changed, 8 insertions(+), 2 deletions(-) > > diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig > index aff1f33665..ec1cfcaaa7 100644 > --- a/arch/riscv/Kconfig > +++ b/arch/riscv/Kconfig > @@ -422,4 +422,11 @@ config TPL_USE_ARCH_MEMSET > > endmenu > > +config SPL_LOAD_FIT_CONFIG > + string "Default FIT configuration for SPL" > + default "binman.dtsi" > + depends on SPL_LOAD_FIT > + help > + Specify corresponding FIT configuration for SPL modes. > + > endmenu > diff --git a/arch/riscv/dts/ae350-u-boot.dtsi > b/arch/riscv/dts/ae350-u-boot.dtsi > index aef9159b7a..ff5725501f 100644 > --- a/arch/riscv/dts/ae350-u-boot.dtsi > +++ b/arch/riscv/dts/ae350-u-boot.dtsi > @@ -1,4 +1,5 @@ > // SPDX-License-Identifier: (GPL-2.0 OR MIT) > +#include CONFIG_SPL_LOAD_FIT_CONFIG
That doesn't seem great to me. I will check the next patch. > / { > cpus { > diff --git a/arch/riscv/dts/ae350_32.dts b/arch/riscv/dts/ae350_32.dts > index 61af6d5465..2caabad888 100644 > --- a/arch/riscv/dts/ae350_32.dts > +++ b/arch/riscv/dts/ae350_32.dts > @@ -2,7 +2,6 @@ > > /dts-v1/; > > -#include "binman.dtsi" > #include "ae350-u-boot.dtsi" > > / { > diff --git a/arch/riscv/dts/ae350_64.dts b/arch/riscv/dts/ae350_64.dts > index 8c7db29b4f..9d5f6c743c 100644 > --- a/arch/riscv/dts/ae350_64.dts > +++ b/arch/riscv/dts/ae350_64.dts > @@ -2,7 +2,6 @@ > > /dts-v1/; > > -#include "binman.dtsi" > #include "ae350-u-boot.dtsi" > > / { > -- > 2.34.1 > Regards, Simon