On Fri, Nov 15, 2024 at 02:36:32PM +0100, Michal Simek wrote:

> There is no common symbol which mini configurations are using and recent
> get_mem_top() changes adding 1.3kB without having a way to remove it.
> That's why introduce new symbol which can be used for removing features
> which are not requested by these configurations.
> 
> Signed-off-by: Michal Simek <[email protected]>
> ---
> 
> Based on:
> https://lore.kernel.org/r/[email protected]
> ---
>  board/xilinx/Kconfig                             | 9 +++++++++
>  board/xilinx/common/board.c                      | 4 ++++
>  configs/amd_versal2_mini_defconfig               | 1 +
>  configs/amd_versal2_mini_emmc_defconfig          | 1 +
>  configs/amd_versal2_mini_ospi_defconfig          | 1 +
>  configs/amd_versal2_mini_qspi_defconfig          | 1 +
>  configs/xilinx_versal_mini_defconfig             | 1 +
>  configs/xilinx_versal_mini_emmc0_defconfig       | 1 +
>  configs/xilinx_versal_mini_emmc1_defconfig       | 1 +
>  configs/xilinx_versal_mini_ospi_defconfig        | 1 +
>  configs/xilinx_versal_mini_qspi_defconfig        | 1 +
>  configs/xilinx_versal_net_mini_defconfig         | 1 +
>  configs/xilinx_versal_net_mini_emmc_defconfig    | 1 +
>  configs/xilinx_versal_net_mini_ospi_defconfig    | 1 +
>  configs/xilinx_versal_net_mini_qspi_defconfig    | 1 +
>  configs/xilinx_zynqmp_mini_defconfig             | 1 +
>  configs/xilinx_zynqmp_mini_emmc0_defconfig       | 1 +
>  configs/xilinx_zynqmp_mini_emmc1_defconfig       | 1 +
>  configs/xilinx_zynqmp_mini_nand_defconfig        | 1 +
>  configs/xilinx_zynqmp_mini_nand_single_defconfig | 1 +
>  configs/xilinx_zynqmp_mini_qspi_defconfig        | 1 +
>  21 files changed, 32 insertions(+)
> 
> diff --git a/board/xilinx/Kconfig b/board/xilinx/Kconfig
> index 0ff8440e6e06..a7da97dbae8c 100644
> --- a/board/xilinx/Kconfig
> +++ b/board/xilinx/Kconfig
> @@ -40,6 +40,15 @@ config XILINX_PS_INIT_FILE
>  
>  endif
>  
> +config MINI
> +     bool "Mini configuration"
> +     depends on ARCH_ZYNQMP || ARCH_VERSAL || ARCH_VERSAL_NET || ARCH_VERSAL2
> +     help
> +       This option disables features which are not needed for Mini U-Boot
> +       configurations. Mini U-Boot is running in EL3 mostly on size 
> contrained
> +       systems. It's purpose is to program non volatile memories or running
> +       initial memory tests.

This is too generic of a symbol name. Please use XILINX_MINI or similar
here, thanks.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to