On 12/04/2011 04:08 PM, Wolfgang Denk wrote:
Dear Shinya Kuribayashi,

In message<4edb8475.1080...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to