From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <michal.simek@xilinx.com>
Date: Mon, 17 May 2021 08:39:12 +0200
Subject: [PATCH 1/1] xilinx: disable Unicode capitalization
In-Reply-To: <20210516091731.26845-1-xypron.glpk@gmx.de>
References: <20210516091731.26845-1-xypron.glpk@gmx.de>
Message-ID: <5784507a-58e5-d6e8-3555-36c033131977@xilinx.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de



On 5/16/21 11:17 AM, Heinrich Schuchardt wrote:
> Save some KiB when building  xilinx_versal_virt_defconfig by disabling
> Unicode capitalization support. This avoids build failures when adding new
> features for the UEFI sub-system.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> It would be preferable to have a size check for the u-boot binary.
> This would avoid seeing problems only during execution.
> ---
>  configs/xilinx_versal_virt_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/configs/xilinx_versal_virt_defconfig b/configs/xilinx_versal_virt_defconfig
> index 707693713a..8bc2ff4a4c 100644
> --- a/configs/xilinx_versal_virt_defconfig
> +++ b/configs/xilinx_versal_virt_defconfig
> @@ -111,3 +111,4 @@ CONFIG_USB_GADGET_PRODUCT_NUM=0x0300
>  CONFIG_USB_GADGET_DOWNLOAD=y
>  CONFIG_USB_FUNCTION_THOR=y
>  CONFIG_OF_LIBFDT_OVERLAY=y
> +# CONFIG_EFI_UNICODE_CAPITALIZATION is not set
> --
> 2.30.2
> 

What's the build issue? Any log? U-Boot is getting bigger and bigger and
we are enabling more and more features for our generic targets that's
why fixing that problem is better than disabling some options.

Thanks,
Michal