On 12/04/2011 04:08 PM, Wolfgang Denk wrote:
Dear Shinya Kuribayashi,
In message4EDB8475.1080305@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