On 8/15/14 6:14, Chen Gang wrote:

On 08/15/2014 02:04 AM, Ralf Baechle wrote:

quoted

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).

quoted

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