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 <[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.
No issue.
Thanks,
Michal
