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

[Greentime Hu]

2017-11-16 18:25 GMT+08:00 Arnd Bergmann <arnd@xxxxxxxx>:
> 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.

Thanks
I will add these 2 configs in the next version patch.
Pass -EL or -EB to compiler based on different configs.