[PATCH] xilinx: Introduce MINI configuration

[Michal Simek]

On 11/15/24 15:19, Tom Rini wrote:
> 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.

No issue.

Thanks,
Michal