Hi Pavel,

> Altera Cyclone 5 board is very different board (big, rectangular,
> expensive) than EBV Socrates (small, circular, cheap) board. Different
> parts are used there, too, but same configuration of u-boot works on
> both. Nevertheless, printing wrong name confuses users. Virtual target
> is completely different, and board configured for it will not boot on
> physical targets.

Thanks for the initiative again.

> Therefore this splits the configuration so that u-boot knows they are
> different. So far it is only used for correcting the puts, but there
> may be other uses in future.
>
> Signed-off-by: Pavel Machek <pa...@denx.de>
>
> ---
>
> Diff from v1: separate virtual target, too, and make it apply to
> recent u-boot.
>
> diff --git a/board/altera/socfpga/socfpga_cyclone5.c 
> b/board/altera/socfpga/socfpga_cyclone5.c
> index a960eb6..33946b6 100644
> --- a/board/altera/socfpga/socfpga_cyclone5.c
> +++ b/board/altera/socfpga/socfpga_cyclone5.c
> @@ -28,7 +28,7 @@ int print_cpuinfo(void)
>   */
>  int checkboard(void)
>  {
> -     puts("BOARD : Altera SOCFPGA Cyclone5 Board\n");
> +     puts("BOARD : " ALTERA_BOARD_NAME "\n");
>       return 0;
>  }
>  
> diff --git a/boards.cfg b/boards.cfg
> index 221b7f8..6eebbf5 100644
> --- a/boards.cfg
> +++ b/boards.cfg
> @@ -379,6 +379,8 @@ Active  arm         armv7          rmobile     renesas    
>      lager
>  Active  arm         armv7          s5pc1xx     samsung         goni          
>       s5p_goni                              -                                 
>                                                                               
>                   Przemyslaw Marczak <p.marc...@samsung.com>
>  Active  arm         armv7          s5pc1xx     samsung         smdkc100      
>       smdkc100                              -                                 
>                                                                               
>                   Minkyu Kang <mk7.k...@samsung.com>
>  Active  arm         armv7          socfpga     altera          socfpga       
>       socfpga_cyclone5                      -                                 
>                                                                               
>                   -
> +Active  arm         armv7          socfpga     altera          socfpga       
>       socfpga_virtual                      -                                  
>                                                                               
>                  -
> +Active  arm         armv7          socfpga     altera          socfpga       
>       socfpga_socrates

As you correctly note, the socrates is sold by EBV and not Altera so I
guess the 5th column needs to be changed.  Will this still work?

Other than that:

Acked-by: Detlev Zundel <d...@denx.de>

Cheers
  Detlev
  
-- 
Those who would give up essential liberty to purchase a little
temporary safety, deserve neither liberty nor safety.
                                     -- Benjamin Franklin
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to