Re: [PATCH 26/31] nds32: Build infrastructure

From: Arnd Bergmann
Date: Thu Nov 16 2017 - 05:26:26 EST


On Thu, Nov 16, 2017 at 11:03 AM, Greentime Hu <green.hu@xxxxxxxxx> wrote:
> 2017-11-13 18:45 GMT+08:00 Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>:
>> Given the checks for __NDS32_EB__, NDS32 can be either big or little endian,
>> so you should have (excatly one of) CPU_BIG_ENDIAN or CPU_LITTLE_ENDIAN set.
>
> Thanks.
> I will check if we need this config or not and update in the next version patch.

I think we have one architecture in the kernel that determines endianess from
the way that the toolchain is built. What all the others do it to have a Kconfig
option, at least CONFIG_CPU_BIG_ENDIAN that is used to pass -mbig-endian
or -mlittle-endian to the compiler. You should do it that way so you can use
any toolchain with any kernel configuration.

Arnd