Re: [PATCH RFC 8/9] board: amlogic: add kconfig fragments for SPL

On 07/09/2025 16:36, Ferass El Hafidi wrote:
> Add kconfig fragments for building SPL binaries for:
>   · ODROID-C2
>   · Videostrong KII Pro
>   · Libre Computer LePotato (1 GB and 2 GB variants)
> 
> Signed-off-by: Ferass El Hafidi <[email protected]>
> ---
>   board/amlogic/p200/spl-odroid-c2.config           | 37 ++++++++++++++++++++++
>   board/amlogic/p200/spl-videostrong-kii-pro.config | 38 +++++++++++++++++++++++
>   board/amlogic/p212/spl-libretech-cc-1gb.config    | 36 +++++++++++++++++++++
>   board/amlogic/p212/spl-libretech-cc-2gb.config    | 38 +++++++++++++++++++++++
>   4 files changed, 149 insertions(+)
> 
> diff --git a/board/amlogic/p200/spl-odroid-c2.config b/board/amlogic/p200/spl-odroid-c2.config
> new file mode 100644
> index 0000000000000000000000000000000000000000..d5ea295bc6d281f3d7770cc841ccbae5aee468a4
> --- /dev/null
> +++ b/board/amlogic/p200/spl-odroid-c2.config
> @@ -0,0 +1,37 @@
> +#
> +# U-Boot SPL fragment for odroid-c2
> +#
> +
> +CONFIG_SPL=y
> +CONFIG_SPL_LIBCOMMON_SUPPORT=y
> +CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_SPL_TEXT_BASE=0xd9001000
> +CONFIG_SPL_STACK=0xd9013000
> +CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
> +CONFIG_SPL_BSS_START_ADDR=0xd900c000
> +CONFIG_SPL_HAVE_INIT_STACK=y
> +CONFIG_SPL_BSS_MAX_SIZE=0x1000
> +CONFIG_SPL_SYS_MALLOC_SIMPLE=y
> +# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> +# CONFIG_SPL_SERIAL_PRESENT is not set
> +# CONFIG_SPL_SHA1 is not set
> +
> +# Enable MMC and serial drivers
> +CONFIG_SPL_SERIAL=y
> +CONFIG_SPL_DRIVERS_MISC=y
> +CONFIG_SPL_MMC=y
> +CONFIG_SPL_SYS_MMCSD_RAW_MODE=y
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x100
> +# CONFIG_SYS_MMCSD_FS_BOOT is not set
> +
> +# Image things
> +CONFIG_SPL_LOAD_FIT=y
> +CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
> +CONFIG_SPL_MAX_SIZE=0xb000
> +CONFIG_SPL_PAD_TO=0x1fe00
> +CONFIG_SPL_ATF=y
> +CONFIG_SPL_OF_CONTROL=y
> +# DRAM configuration
> +CONFIG_DRAM_SIZE=2048
> +CONFIG_DRAM_2T_MODE=y
> +CONFIG_DRAM_TWO_IDENTICAL_RANKS=y
> diff --git a/board/amlogic/p200/spl-videostrong-kii-pro.config b/board/amlogic/p200/spl-videostrong-kii-pro.config
> new file mode 100644
> index 0000000000000000000000000000000000000000..5b3ba9d990840411ef3cef3db3c6aa9d7085b3ef
> --- /dev/null
> +++ b/board/amlogic/p200/spl-videostrong-kii-pro.config
> @@ -0,0 +1,38 @@
> +#
> +# U-Boot SPL fragment for videostrong-kii-pro
> +#
> +
> +CONFIG_SPL=y
> +CONFIG_SPL_LIBCOMMON_SUPPORT=y
> +CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_SPL_TEXT_BASE=0xd9001000
> +CONFIG_SPL_STACK=0xd9013000
> +CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
> +CONFIG_SPL_BSS_START_ADDR=0xd900c000
> +CONFIG_SPL_HAVE_INIT_STACK=y
> +CONFIG_SPL_BSS_MAX_SIZE=0x1000
> +CONFIG_SPL_SYS_MALLOC_SIMPLE=y
> +# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> +# CONFIG_SPL_SERIAL_PRESENT is not set
> +# CONFIG_SPL_SHA1 is not set
> +
> +# Enable MMC and serial drivers
> +CONFIG_SPL_SERIAL=y
> +CONFIG_SPL_DRIVERS_MISC=y
> +CONFIG_SPL_MMC=y
> +CONFIG_SPL_SYS_MMCSD_RAW_MODE=y
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x100
> +# CONFIG_SYS_MMCSD_FS_BOOT is not set
> +
> +# Image things
> +CONFIG_SPL_LOAD_FIT=y
> +CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
> +CONFIG_SPL_MAX_SIZE=0xb000
> +CONFIG_SPL_PAD_TO=0x1fe00
> +CONFIG_SPL_ATF=y
> +CONFIG_SPL_OF_CONTROL=y
> +# DRAM configuration
> +CONFIG_DRAM_SIZE=2048
> +CONFIG_DRAM_2T_MODE=y
> +CONFIG_DRAM_TWO_DIFF_RANKS=y
> +
> diff --git a/board/amlogic/p212/spl-libretech-cc-1gb.config b/board/amlogic/p212/spl-libretech-cc-1gb.config
> new file mode 100644
> index 0000000000000000000000000000000000000000..f58bb69325980d582d2d826ebbf28e4700965dfb
> --- /dev/null
> +++ b/board/amlogic/p212/spl-libretech-cc-1gb.config
> @@ -0,0 +1,36 @@
> +#
> +# U-Boot SPL fragment for libretech-cc (1 GB variant)
> +#
> +
> +CONFIG_SPL=y
> +CONFIG_SPL_LIBCOMMON_SUPPORT=y
> +CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_SPL_TEXT_BASE=0xd9001000
> +CONFIG_SPL_STACK=0xd9013000
> +CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
> +CONFIG_SPL_BSS_START_ADDR=0xd900c000
> +CONFIG_SPL_HAVE_INIT_STACK=y
> +CONFIG_SPL_BSS_MAX_SIZE=0x1000
> +CONFIG_SPL_SYS_MALLOC_SIMPLE=y
> +# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> +# CONFIG_SPL_SERIAL_PRESENT is not set
> +# CONFIG_SPL_SHA1 is not set
> +
> +# Enable MMC and serial drivers
> +CONFIG_SPL_SERIAL=y
> +CONFIG_SPL_DRIVERS_MISC=y
> +CONFIG_SPL_MMC=y
> +CONFIG_SPL_SYS_MMCSD_RAW_MODE=y
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x100
> +# CONFIG_SYS_MMCSD_FS_BOOT is not set
> +
> +# Image things
> +CONFIG_SPL_LOAD_FIT=y
> +CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
> +CONFIG_SPL_MAX_SIZE=0xb000
> +CONFIG_SPL_PAD_TO=0x1fe00
> +CONFIG_SPL_ATF=y
> +CONFIG_SPL_OF_CONTROL=y
> +# DRAM configuration
> +CONFIG_DRAM_2T_MODE=y
> +CONFIG_DRAM_DQS_CORR=y
> diff --git a/board/amlogic/p212/spl-libretech-cc-2gb.config b/board/amlogic/p212/spl-libretech-cc-2gb.config
> new file mode 100644
> index 0000000000000000000000000000000000000000..65d700558e65ca8d819f397a0153be8bf07ae702
> --- /dev/null
> +++ b/board/amlogic/p212/spl-libretech-cc-2gb.config
> @@ -0,0 +1,38 @@
> +#
> +# U-Boot SPL fragment for libretech-cc (2 GB variant)
> +#
> +
> +CONFIG_SPL=y
> +CONFIG_SPL_LIBCOMMON_SUPPORT=y
> +CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_SPL_TEXT_BASE=0xd9001000
> +CONFIG_SPL_STACK=0xd9013000
> +CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
> +CONFIG_SPL_BSS_START_ADDR=0xd900c000
> +CONFIG_SPL_HAVE_INIT_STACK=y
> +CONFIG_SPL_BSS_MAX_SIZE=0x1000
> +CONFIG_SPL_SYS_MALLOC_SIMPLE=y
> +# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> +# CONFIG_SPL_SERIAL_PRESENT is not set
> +# CONFIG_SPL_SHA1 is not set
> +
> +# Enable MMC and serial drivers
> +CONFIG_SPL_SERIAL=y
> +CONFIG_SPL_DRIVERS_MISC=y
> +CONFIG_SPL_MMC=y
> +CONFIG_SPL_SYS_MMCSD_RAW_MODE=y
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x100
> +# CONFIG_SYS_MMCSD_FS_BOOT is not set
> +
> +# Image things
> +CONFIG_SPL_LOAD_FIT=y
> +CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
> +CONFIG_SPL_MAX_SIZE=0xb000
> +CONFIG_SPL_PAD_TO=0x1fe00
> +CONFIG_SPL_ATF=y
> +CONFIG_SPL_OF_CONTROL=y
> +# DRAM configuration
> +CONFIG_DRAM_SIZE=2048
> +CONFIG_DRAM_2T_MODE=y
> +CONFIG_DRAM_DQS_CORR=y
> +CONFIG_DRAM_TWO_IDENTICAL_RANKS=y
> 

Reviewed-by: Neil Armstrong <[email protected]>