[U-Boot] [PATCH] MIPS: fix endianess handling

Daniel Schwierzeck daniel.schwierzeck at googlemail.com
Mon Dec 5 01:10:35 CET 2011


On 12/04/2011 04:08 PM, Wolfgang Denk wrote:
> Dear Shinya Kuribayashi,
>
> In message<4EDB8475.1080305 at pobox.com>  you wrote:
>>
>>>> -pb1000                       mips        mips32      pb1x00              -              au1x00      pb1x00:PB1000
>>>> +pb1000                       mips        mips32      pb1x00              -              au1x00      pb1x00:PB1000,SYS_LITTLE_ENDIAN
> ...
>> But I don't see any reason CONFIG_SYS_LITTLE_ENDIAN is specified in
>> boards.cfg.  Just putting it in configs/pb1000.h is enough, isn't it?
>
> Agreed.  In almost all cases this should be done in the board config
> file only.  Only if there are cases where a board suppors both LE and
> BE configurations (is there any such board??) then we should need a
> SYS_*_ENDIAN setting in boards.cfg (but even then, only one - not
> two).

I did this because of dbau1550 and dbau1550_el. For the pb1000 I'll
move it to the board config.

>
> Best regards,
>
> Wolfgang Denk
>


-- 
Best regards,
Daniel


More information about the U-Boot mailing list