On Wed, Sep 11, 2019 at 07:14:59PM +0800, mingming lee wrote: > This adds a general board file based on MT8518 SoCs from MediaTek. > > Apart from the generic parts (cpu) we add some low level init codes > and initialize the early clocks. > > This commit is adding the basic boot support for the MT8518 eMMC board. > > Signed-off-by: mingming lee <mingming....@mediatek.com>
OK, there's a few problems here: [snip] > +int board_init(void) > +{ > + /* address of boot parameters */ > + gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100; > + > + printf("gd->fdt_blob is %p\n", gd->fdt_blob); Please remove this (and audit the series for other) bring-up debug printfs that we shouldn't need anymore. > + return 0; > +} > + > +int board_late_init(void) > +{ > + /*to load environment variable from persistent store*/ > + gd->env_valid = 1; > + env_relocate(); > + > + return 0; > +} Er, do you really need this? [snip] > +/* Machine ID */ > +#define CONFIG_MACH_TYPE 8518 > +#define CONFIG_SYS_NONCACHED_MEMORY BIT(20) CONFIG_MACH_TYPE is not relevant to aarch64 platforms (it's for pre-device tree boards) and 'BIT(20)' isn't a valid value for CONFIG_SYS_NONCACHED_MEMORY (and leads to a warning a build). Thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot