Hello Maintainers: Is this patch OK? If it pass basic checking, please let me know, and I shall try to make another related patch for KBuild (I can do nothing related with Kbuild, before get confirmation for this patch). Thanks. On 8/15/14 17:01, Chen Gang wrote: > > > On 8/15/14 6:14, Chen Gang wrote: >> On 08/15/2014 02:04 AM, Ralf Baechle wrote: >>> >> >> OK, thanks, I assumes when support both endian, the default choice is >> CPU_BIG_ENDIAN, although no default value for choice (originally, I did >> worry about it). >> >>> So I think you can just drop the MIPS segment from your patch. >>> >> >> If what I assumes is correct, what you said sounds reasonable to me. >> >> > > So for me, it is harmless to add CPU_*_ENDIAN explicitly, and can let > other members don't need think of. > > By the way, for sh, it is almost the same case, except it contents the > default value, for me, it is clear enough, so I skip sh architecture in > this patch. > > > Thanks > -- Chen Gang Open, share, and attitude like air, water, and life which God blessed