On 08/16/2018 06:44 PM, Thomas Fitzsimmons wrote: > Add support for loading U-Boot on the Broadcom 7260 SoC. This port > assumes Broadcom's BOLT bootloader is acting as the second stage > bootloader, and U-Boot is acting as the third stage bootloader, loaded > as an ELF program by BOLT. > > Signed-off-by: Thomas Fitzsimmons <fitz...@fitzsim.org>
Reviewed-by: Florian Fainelli <f.faine...@gmail.com> Looks good to me given the structured you added earlier for 7445 support, just one nit below: [snip] > + > +#ifndef __CONFIG_H > +#define __CONFIG_H > + > +#define CONFIG_SYS_NS16550_COM3 0xf040c000 This is the physical address of UARTA, there are typically 3 UARTs on those SoCs, is there a reason to name this COM3 and not COM1? If this is an established u-boot convention, pardon my ignore here. -- Florian _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot