Re: [PATCH-next v3] arm32: enable HAVE_LD_DEAD_CODE_DATA_ELIMINATION

[Arnd Bergmann]

On Fri, Mar 15, 2024, at 12:48, liuyuntao (F) wrote:
> On 2024/3/15 18:46, Ard Biesheuvel wrote:
>> @@ -28,6 +28,7 @@
>>                                     -DEFI_HAVE_MEMCHR -DEFI_HAVE_STRRCHR \
>>                                     -DEFI_HAVE_STRCMP -fno-builtin -fpic \
>>                                     $(call cc-option,-mno-single-pic-base)
>> +cflags-$(CONFIG_ARM)           := $(filter-out -fdata-sections, $(CFLAGS-y))
>>   cflags-$(CONFIG_RISCV)         += -fpic -DNO_ALTERNATIVE -mno-relax
>>   cflags-$(CONFIG_LOONGARCH)     += -fpie
>
> Another error:
> cannot initialize array of ‘short unsigned int’ from a string literal 
> with type array of ‘unsigned int’
>   17 | static const efi_char16_t shim_MokSBState_name[] = L"MokSBStateRT";

The line sets cflags-y to an empty string.

It probably should have been

cflags-$(CONFIG_ARM)           := $(filter-out -fdata-sections, $(cflags-y))

(small cflags instead of CFLAGS).

     Arnd