On Mon, Jul 04, 2011 at 01:45:41PM -0400, Christopher Harvey wrote:
> Signed-off-by: Christopher Harvey <char...@matrox.com>
> ---
>  arch/arm/lib/board.c |    4 ++++
>  arch/arm/lib/bootm.c |    6 ++++++
>  2 files changed, 10 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
> index 169dfeb..dbb835a 100644
> --- a/arch/arm/lib/board.c
> +++ b/arch/arm/lib/board.c
> @@ -404,6 +404,10 @@ void board_init_f (ulong bootflag)
>       post_bootmode_init();
>       post_run (NULL, POST_ROM | post_bootmode_get(0));
>  #endif
> +     /* 0xffffffff is used to mark is value as "unset".

s/mark is/mark a/

> +        Hopefully there will never be this many machines. 
> +        Can't use 0 since 0 is already used as a mach-type. */
> +     gd->bd->bi_arch_number = 0xffffffff; 
>  
>       gd->bd->bi_baudrate = gd->baudrate;
>       /* Ram ist board specific, so move it to board code ... */
> diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
> index 802e833..70b3b76 100644
> --- a/arch/arm/lib/bootm.c
> +++ b/arch/arm/lib/bootm.c
> @@ -113,6 +113,12 @@ int do_bootm_linux(int flag, int argc, char *argv[], 
> bootm_headers_t *images)
>               printf ("Using machid 0x%x from environment\n", machid);
>       }
>  
> +#ifdef DEBUG
> +     if(machid==0xffffffff) {
> +             debug("\nWarning: machid not set! Linux will not finish 
> booting.\n\n");

s/finish/start/ ;-)

Also, shouldn't the compile fail in this case (#error)?  Or, at least #warn?

> +     }
> +#endif
> +
>       show_boot_progress (15);
>  
>  #ifdef CONFIG_OF_LIBFDT
> -- 
> 1.7.3.4

Please take comments with a grain of salt, I'm asking, not telling.  I'm
fairly new to this as well.

thx,

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

Reply via email to