[PATCH v2 1/2] xilinx: mbv: Remove redundancy in MB-V defconfigs

[Michal Simek]

On 10/2/25 10:19, Michal Simek wrote:
> From: Padmarao Begari <padmarao.begari at amd.com>
> 
> Removes redundant configuration options from MB-V platform defconfigs
> targeting both modes. Common settings are now placed in
> xilinx_mbv32_defconfig, which is included via the #include <configs/...>.
> This approach centralizes configuration management, reduces duplication,
> and makes future maintenance easier.
> 
> Signed-off-by: Padmarao Begari <padmarao.begari at amd.com>
> Signed-off-by: Michal Simek <michal.simek at amd.com>
> ---
> 
> Changes in v2:
> - new patch in series
> 
>   configs/xilinx_mbv32_smode_defconfig | 49 ++-------------------------
>   configs/xilinx_mbv64_defconfig       | 47 ++------------------------
>   configs/xilinx_mbv64_smode_defconfig | 50 ++--------------------------
>   3 files changed, 6 insertions(+), 140 deletions(-)
> 
> diff --git a/configs/xilinx_mbv32_smode_defconfig b/configs/xilinx_mbv32_smode_defconfig
> index c95b44973952..dd1221a6904f 100644
> --- a/configs/xilinx_mbv32_smode_defconfig
> +++ b/configs/xilinx_mbv32_smode_defconfig
> @@ -1,50 +1,5 @@
> -CONFIG_RISCV=y
> -CONFIG_SYS_MALLOC_LEN=0xe00000
> -CONFIG_NR_DRAM_BANKS=1
> -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
> -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x81200000
> -CONFIG_ENV_SIZE=0x20000
> -CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv32"
> -CONFIG_SPL_STACK=0x80200000
> -CONFIG_SPL_BSS_START_ADDR=0x84000000
> -CONFIG_SPL_BSS_MAX_SIZE=0x80000
> -CONFIG_SYS_BOOTM_LEN=0x800000
> -CONFIG_SYS_LOAD_ADDR=0x80200000
> -CONFIG_SPL_SIZE_LIMIT=0x40000
> -CONFIG_SPL=y
> -CONFIG_DEBUG_UART_BASE=0x40600000
> -CONFIG_DEBUG_UART_CLOCK=1000000
> -CONFIG_SYS_CLK_FREQ=100000000
> -CONFIG_BOOT_SCRIPT_OFFSET=0x0
> -CONFIG_TARGET_XILINX_MBV=y
> +#include <configs/xilinx_mbv32_defconfig>
> +
>   CONFIG_SPL_OPENSBI_LOAD_ADDR=0x80100000
>   CONFIG_RISCV_SMODE=y
> -# CONFIG_SPL_SMP is not set
> -CONFIG_REMAKE_ELF=y
> -CONFIG_FIT=y
> -CONFIG_SPL_LOAD_FIT_ADDRESS=0x80200000
> -CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_DISPLAY_CPUINFO=y
> -CONFIG_DISPLAY_BOARDINFO=y
> -# CONFIG_BOARD_INIT is not set
> -# CONFIG_BOARD_LATE_INIT is not set
> -CONFIG_SPL_MAX_SIZE=0x40000
> -# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> -CONFIG_SPL_HAVE_INIT_STACK=y
> -CONFIG_SPL_SYS_MALLOC=y
>   CONFIG_SPL_OPENSBI_SCRATCH_OPTIONS=0x2
> -# CONFIG_CMD_MII is not set
> -CONFIG_CMD_TIMER=y
> -CONFIG_ENV_RELOC_GD_ENV_ADDR=y
> -CONFIG_SPL_DM_SEQ_ALIAS=y
> -CONFIG_DM_MTD=y
> -CONFIG_DEBUG_UART_UARTLITE=y
> -CONFIG_DEBUG_UART_ANNOUNCE=y
> -CONFIG_DEBUG_UART_SKIP_INIT=y
> -CONFIG_XILINX_UARTLITE=y
> -# CONFIG_RISCV_TIMER is not set
> -CONFIG_XILINX_TIMER=y
> -# CONFIG_BINMAN_FDT is not set
> -CONFIG_BINMAN_DTB="./arch/riscv/dts/xilinx-binman.dtb"
> -CONFIG_PANIC_HANG=y
> -CONFIG_SPL_GZIP=y
> diff --git a/configs/xilinx_mbv64_defconfig b/configs/xilinx_mbv64_defconfig
> index a3cc1a5669e5..27d674fc34fb 100644
> --- a/configs/xilinx_mbv64_defconfig
> +++ b/configs/xilinx_mbv64_defconfig
> @@ -1,47 +1,4 @@
> -CONFIG_RISCV=y
> -CONFIG_SYS_MALLOC_LEN=0xe00000
> -CONFIG_NR_DRAM_BANKS=1
> -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
> -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x81200000
> -CONFIG_ENV_SIZE=0x20000
> +#include <configs/xilinx_mbv32_defconfig>
> +
>   CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv64"
> -CONFIG_SPL_STACK=0x80200000
> -CONFIG_SPL_BSS_START_ADDR=0x84000000
> -CONFIG_SPL_BSS_MAX_SIZE=0x80000
> -CONFIG_SYS_BOOTM_LEN=0x800000
> -CONFIG_SYS_LOAD_ADDR=0x80200000
> -CONFIG_SPL_SIZE_LIMIT=0x40000
> -CONFIG_SPL=y
> -CONFIG_DEBUG_UART_BASE=0x40600000
> -CONFIG_DEBUG_UART_CLOCK=100000000
> -CONFIG_SYS_CLK_FREQ=100000000
> -CONFIG_BOOT_SCRIPT_OFFSET=0x0
> -CONFIG_DEBUG_UART=y
> -CONFIG_TARGET_XILINX_MBV=y
>   CONFIG_ARCH_RV64I=y
> -# CONFIG_SPL_SMP is not set
> -CONFIG_REMAKE_ELF=y
> -CONFIG_FIT=y
> -CONFIG_SPL_LOAD_FIT_ADDRESS=0x80200000
> -CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_DISPLAY_CPUINFO=y
> -CONFIG_DISPLAY_BOARDINFO=y
> -# CONFIG_BOARD_INIT is not set
> -# CONFIG_BOARD_LATE_INIT is not set
> -CONFIG_SPL_MAX_SIZE=0x40000
> -# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> -CONFIG_SPL_HAVE_INIT_STACK=y
> -CONFIG_SPL_SYS_MALLOC=y
> -# CONFIG_CMD_MII is not set
> -CONFIG_CMD_TIMER=y
> -CONFIG_ENV_RELOC_GD_ENV_ADDR=y
> -CONFIG_SPL_DM_SEQ_ALIAS=y
> -CONFIG_DM_MTD=y
> -CONFIG_DEBUG_UART_ANNOUNCE=y
> -CONFIG_DEBUG_UART_SKIP_INIT=y
> -CONFIG_XILINX_UARTLITE=y
> -CONFIG_XILINX_TIMER=y
> -# CONFIG_BINMAN_FDT is not set
> -CONFIG_BINMAN_DTB="./arch/riscv/dts/xilinx-binman.dtb"
> -CONFIG_PANIC_HANG=y
> -CONFIG_SPL_GZIP=y
> diff --git a/configs/xilinx_mbv64_smode_defconfig b/configs/xilinx_mbv64_smode_defconfig
> index 3d49670c60c4..56bf0bdc7e4f 100644
> --- a/configs/xilinx_mbv64_smode_defconfig
> +++ b/configs/xilinx_mbv64_smode_defconfig
> @@ -1,51 +1,5 @@
> -CONFIG_RISCV=y
> -CONFIG_SYS_MALLOC_LEN=0xe00000
> -CONFIG_NR_DRAM_BANKS=1
> -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
> -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x81200000
> -CONFIG_ENV_SIZE=0x20000
> -CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv64"
> -CONFIG_SPL_STACK=0x80200000
> -CONFIG_SPL_BSS_START_ADDR=0x84000000
> -CONFIG_SPL_BSS_MAX_SIZE=0x80000
> -CONFIG_SYS_BOOTM_LEN=0x800000
> -CONFIG_SYS_LOAD_ADDR=0x80200000
> -CONFIG_SPL_SIZE_LIMIT=0x40000
> -CONFIG_SPL=y
> -CONFIG_DEBUG_UART_BASE=0x40600000
> -CONFIG_DEBUG_UART_CLOCK=1000000
> -CONFIG_SYS_CLK_FREQ=100000000
> -CONFIG_BOOT_SCRIPT_OFFSET=0x0
> -CONFIG_TARGET_XILINX_MBV=y
> +#include <configs/xilinx_mbv64_defconfig>
> +
>   CONFIG_SPL_OPENSBI_LOAD_ADDR=0x80100000
> -CONFIG_ARCH_RV64I=y
>   CONFIG_RISCV_SMODE=y
> -# CONFIG_SPL_SMP is not set
> -CONFIG_REMAKE_ELF=y
> -CONFIG_FIT=y
> -CONFIG_SPL_LOAD_FIT_ADDRESS=0x80200000
> -CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_DISPLAY_CPUINFO=y
> -CONFIG_DISPLAY_BOARDINFO=y
> -# CONFIG_BOARD_INIT is not set
> -# CONFIG_BOARD_LATE_INIT is not set
> -CONFIG_SPL_MAX_SIZE=0x40000
> -# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> -CONFIG_SPL_HAVE_INIT_STACK=y
> -CONFIG_SPL_SYS_MALLOC=y
>   CONFIG_SPL_OPENSBI_SCRATCH_OPTIONS=0x2
> -# CONFIG_CMD_MII is not set
> -CONFIG_CMD_TIMER=y
> -CONFIG_ENV_RELOC_GD_ENV_ADDR=y
> -CONFIG_SPL_DM_SEQ_ALIAS=y
> -CONFIG_DM_MTD=y
> -CONFIG_DEBUG_UART_UARTLITE=y
> -CONFIG_DEBUG_UART_ANNOUNCE=y
> -CONFIG_DEBUG_UART_SKIP_INIT=y
> -CONFIG_XILINX_UARTLITE=y
> -# CONFIG_RISCV_TIMER is not set
> -CONFIG_XILINX_TIMER=y
> -# CONFIG_BINMAN_FDT is not set
> -CONFIG_BINMAN_DTB="./arch/riscv/dts/xilinx-binman.dtb"
> -CONFIG_PANIC_HANG=y
> -CONFIG_SPL_GZIP=y

Applied.
M