Hi Dirk,

<snip>

> Third, I don't like the mixing of board and vendor name as directory 
> names. It seems to me that where you think you know the vendor, you 
> use the vendor name, and where you seem to not know it, you use the 
> board name. So, to be consistent, and to overcome vendor name issue 
> above, I'd like to use the board name everywhere. That is:

Jean-Christophe's patches seem to follow the current U-Boot convention - 
use board/<vendor> when a vendor has multiple boards, and use 
board/<boardname> when a vendor only has 1 board.  I can't speak to the 
accuracy of how he split boards up, but his idea seems correct.

> board/beagle/
> board/omap3evm/
> board/omap1510inn/
> ...
> board/omap5912osk/
> ...
> board/omap2420h4/
> ...
> board/zoom1/
> board/zoom2/
> board/overo/
> board/pandora/
> ...

This isn't in line with how U-Boot currently organizes the board/ directory.

> To improve this even more, something like this would be nice:
> 
> board/omap3beagle/
> board/omap3evm/
> board/omap1innovator/
> ...
> board/omap1osk/
> ...
> board/omap2h4/
> ...
> board/omap3zoom1/
> board/omap3zoom2/
> board/omap3overo/
> board/omap3pandora/
> ...

Neither is this.  Jean-Christophe seems to be doing the "right" thing 
with his patches and I'd ACK them assuming the boards are put in the 
proper board/<vendor> and board/<boardname> directory.

Best,
Peter
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to