[PATCH V2 3/3] dts: andes: add SPL config option for itb layout

Tom Rini trini at konsulko.com
Mon Sep 18 17:04:26 CEST 2023


On Mon, Sep 18, 2023 at 08:11:36PM +0800, Randolph wrote:

> Add SPL_LOAD_FIT_CONFIG for binman itb layout selection
> 
> Signed-off-by: Randolph <randolph at andestech.com>
> ---
>  arch/riscv/dts/ae350-u-boot.dtsi     | 1 +
>  arch/riscv/dts/ae350_32.dts          | 1 -
>  arch/riscv/dts/ae350_64.dts          | 1 -
>  board/AndesTech/ae350/Kconfig        | 6 ++++++
>  configs/ae350_rv32_defconfig         | 1 +
>  configs/ae350_rv32_spl_defconfig     | 1 +
>  configs/ae350_rv32_spl_xip_defconfig | 1 +
>  configs/ae350_rv32_xip_defconfig     | 1 +
>  configs/ae350_rv64_defconfig         | 1 +
>  configs/ae350_rv64_spl_defconfig     | 1 +
>  configs/ae350_rv64_spl_xip_defconfig | 1 +
>  configs/ae350_rv64_xip_defconfig     | 1 +
>  12 files changed, 15 insertions(+), 2 deletions(-)

So this is where I have a problem.  It's not clear why you need to
change anything here, as there's no example given for doing something
other than what you do today, and there's no documentation about it
either.  From other contexts, it seems like you're trying to re-implement
CONFIG_SPL_OS_BOOT but not explaining why you can't use the existing
framework.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230918/d08e6ca7/attachment.sig>


More information about the U-Boot mailing list