[PATCH] xilinx: Introduce MINI configuration

[Tom Rini]

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 <michal.simek at amd.com>
> ---
> 
> Based on:
> https://lore.kernel.org/r/20241025172724.195093-3-sughosh.ganu@linaro.org
> ---
>  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
-------------- 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/20241115/661c2f10/attachment.sig>