[PATCH 1/1] xilinx: disable Unicode capitalization

[Heinrich Schuchardt]

On 17.05.21 08:39, Michal Simek wrote:
>
>
> 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 at 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

Hello Michal,

this job failed:
https://source.denx.de/u-boot/custodians/u-boot-efi/-/jobs/269614

After deactivating CONFIG_EFI_UNICODE_CAPITALIZATION I get success:
https://source.denx.de/u-boot/custodians/u-boot-efi/-/jobs/269620

> we are enabling more and more features for our generic targets that's
> why fixing that problem is better than disabling some options.

I do not understand what you mean here.

CONFIG_EFI_UNICODE_CAPITALIZATION is enabled since years.

Best regards

Heinrich