Dear Linus Walleij, > When U-Boot is configured for Versatile AB, it will still pass > the machine ID of Versatile PB to the kernel. After this simple > fix the system boots correctly. > > Cc: Stefano Babic <[email protected]> > Cc: Marek Vasut <[email protected]> > Signed-off-by: Linus Walleij <[email protected]> > --- > board/armltd/versatile/versatile.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/board/armltd/versatile/versatile.c > b/board/armltd/versatile/versatile.c index 30a3b904db4c..4e2d3421d8b1 > 100644 > --- a/board/armltd/versatile/versatile.c > +++ b/board/armltd/versatile/versatile.c > @@ -52,7 +52,11 @@ int board_early_init_f (void) > int board_init (void) > { > /* arch number of Versatile Board */ > +#ifdef CONFIG_ARCH_VERSATILE_AB > + gd->bd->bi_arch_number = MACH_TYPE_VERSATILE_AB; > +#else > gd->bd->bi_arch_number = MACH_TYPE_VERSATILE_PB; > +#endif > > /* adress of boot parameters */ > gd->bd->bi_boot_params = 0x00000100;
We should just switch to DT, but I do understand the motivation here. Can you not detect which board it is dynamically to avoid the ifdef? Otherwise: Acked-by: Marek Vasut <[email protected]> Best regards, Marek Vasut _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
