2017-11-17 19:02:44

by Greentime Hu

[permalink] [raw]
Subject: Re: [PATCH 26/31] nds32: Build infrastructure

2017-11-16 18:25 GMT+08:00 Arnd Bergmann <[email protected]>:
> On Thu, Nov 16, 2017 at 11:03 AM, Greentime Hu <[email protected]> wrote:
>> 2017-11-13 18:45 GMT+08:00 Geert Uytterhoeven <[email protected]>:
>>> 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.

From 1584339498157434858@xxx Fri Nov 17 18:38:07 +0000 2017
X-GM-THRID: 1583483358934427114
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread