[U-Boot] [PATCH] boards.cfg: keep it sorted filling the board field

[Masahiro Yamada]

Hi Tom,


On Wed, 9 Jul 2014 09:12:48 -0400
Tom Rini <trini at ti.com> wrote:

> On Wed, Jul 09, 2014 at 05:34:28PM +0900, Masahiro Yamada wrote:
> > The boards.cfg file has allowed to use "-" for the board (= 6th) field
> > if the board name is the same as the 7th field.
> > 
> > But I notice one problem.
> > Because tools/reformat.py sorts the lines in the simple alphabetical
> > order (= the order of character code), some entries for the same board
> > are not lined up together.
> > 
> > For example, "bf527-ezkit" and "bf527-ezkit-v2" share the same board.
> > But they are located separately because "bf527-ezkit" fills the board
> > field with "-" whereas "bf527-ezkit-v2" specifies it explicitely.
> > 
> > The similar things can be seen:
> >  - between "trizepsive" and "polaris"
> >  - between "RRvision" and "RRvision_LCD"
> >  - between "korat" and "korat_perm"
> >  - between "lwmon5" and "lcd4_lwmon5"
> > 
> > This commit was generated by the following command:
> > 
> > awk '$6 == "-" { $6 = $7 } { print }' boards.cfg \
> >   | tools/reformat.py -i -d '-' -s 8  > boards0.cfg; \
> >   mv boards0.cfg boards.cfg
> > 
> > Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
> 
> Tricky!  So I'm fine with correcting this problem but we also need to
> update the top part of baords.cfg where it says how to re-sort things so
> that we don't lose this in the future.
> 
> -- 



This patch had gone to Change Requested
so I resent this + follow-up patch:

http://patchwork.ozlabs.org/patch/369047/
http://patchwork.ozlabs.org/patch/369023/



We won't lose this because using '-' is prohibited
for that the board field and the target field are the same.


Best Regards
Masahiro Yamada