On 5/25/2017 3:09 PM, Arnd Bergmann wrote:
On Thu, May 25, 2017 at 4:51 PM, Babu Moger <babu.moger@xxxxxxxxxx> wrote:
On 5/24/2017 5:18 AM, Arnd Bergmann wrote:
I guess the time is ripe for adding (both) symbols to all architectures?
Good idea. I think we can do most of this by adding a few lines to
arch/Kconfig:
config CPU_BIG_ENDIAN
bool
config CPU_LITTLE_ENDIAN
def_bool !CPU_BIG_ENDIAN
This way, we only need to add 'select CPU_BIG_ENDIAN' to the
architectures that are always big-endian, and we don't need to
change anything for the ones that have a single 'CPU_BIG_ENDIAN'
option.
The three architectures that have a 'choice' statement (mips, ppc and
sh) will have to convert, and m32r will have to replace the
what to you mean by "(mips, ppc andsh) will have to convert"? Do you expect
any changes here?
Kconfig does not allow you to have the same symbol as both a regular
'bool' and also 'bool within choice', so those three have to replace the
choice with a user-visible 'config CPU_BIG_ENDIAN' option like the
other ones have.
Ok. I will address it in my next version. Thanks
I also notice that for arch/s390/Kconfig you now have both the
'select CPU_BIG_ENDIAN' and the 'config CPU_BIG_ENDIAN
def_bool y', I'd remove the second one in the same patch.
Sure. Will correct it.
Arnd
