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.
