Re: [PATCH] configs: versal_net: Enable CONFIG_LTO for mini qspi/ospi

[Michal Simek <michal.simek@amd.com>]

On 1/26/24 09:11, Venkatesh Yadav Abbarapu wrote:
> Adding a tiny bit more code for mini u-boot leads to a OCM
> image overflow. Fix this by enabling LTO for this board, so that such
> changes still can be made to the common U-Boot code.
> 
> Enable building mini u-boot image with LTO, which results in about 8KB
> reduction in size.
> 
> Signed-off-by: Michal Simek <michal.simek@amd.com>
> Signed-off-by: Venkatesh Yadav Abbarapu <venkatesh.abbarapu@amd.com>
> ---
>   configs/xilinx_versal_net_mini_ospi_defconfig | 1 +
>   configs/xilinx_versal_net_mini_qspi_defconfig | 1 +
>   2 files changed, 2 insertions(+)
> 
> diff --git a/configs/xilinx_versal_net_mini_ospi_defconfig b/configs/xilinx_versal_net_mini_ospi_defconfig
> index 5f42243d22..d78c9f8059 100644
> --- a/configs/xilinx_versal_net_mini_ospi_defconfig
> +++ b/configs/xilinx_versal_net_mini_ospi_defconfig
> @@ -15,6 +15,7 @@ CONFIG_DEFAULT_DEVICE_TREE="versal-net-mini-ospi-single"
>   CONFIG_SYS_MEM_RSVD_FOR_MMU=y
>   # CONFIG_PSCI_RESET is not set
>   CONFIG_SYS_LOAD_ADDR=0xBBF80000
> +CONFIG_LTO=y
>   # CONFIG_EXPERT is not set
>   CONFIG_REMAKE_ELF=y
>   # CONFIG_AUTOBOOT is not set
> diff --git a/configs/xilinx_versal_net_mini_qspi_defconfig b/configs/xilinx_versal_net_mini_qspi_defconfig
> index 4fa83faa40..b0567f857a 100644
> --- a/configs/xilinx_versal_net_mini_qspi_defconfig
> +++ b/configs/xilinx_versal_net_mini_qspi_defconfig
> @@ -13,6 +13,7 @@ CONFIG_DEFAULT_DEVICE_TREE="versal-net-mini-qspi-single"
>   CONFIG_SYS_MEM_RSVD_FOR_MMU=y
>   # CONFIG_PSCI_RESET is not set
>   CONFIG_SYS_LOAD_ADDR=0xBBF80000
> +CONFIG_LTO=y
>   # CONFIG_EXPERT is not set
>   CONFIG_REMAKE_ELF=y
>   # CONFIG_AUTOBOOT is not set

Applied.
M