On 27.1.2018 22:55, Lukasz Majewski wrote: > The goal of this patch is to clean up the code related to choosing spl > MMC boot mode. > > The spl_boot_mode() now is called only in spl_mmc_load_image() function, > which is only compiled in if CONFIG_SPL_MMC_SUPPORT is enabled. > > To achieve the goal, all per mach/arch implementations eligible for > unification has been replaced with one __weak implementation. > > Signed-off-by: Lukasz Majewski <lu...@denx.de> > > --- > Comments regarding cleanup (to help maintainers assessment if those changes > break anything): > > Implementations necessary to stay as is: > ---------------------------------------- > - SW fix for broken ROM boot code > arch/arm/mach-uniphier/mmc-boot-mode.c > > - Omap specific adjustments: > arch/arm/mach-omap2/boot-common.c > > Below archs had a bit off-standard approach, but were eligible for using > the __weak function: > -------------------- > - at91 -> check if proper configs flags in the weak function are set: > arch/arm/mach-at91/spl.c > > - zyncmp -> the BOOT_DEVICE_RAM return of spl_boot_mode seems like an error
zynqmp > (will hang in spl_mmc_load_image() anyway) > arch/arm/cpu/armv8/zynqmp/spl.c Changes look reasonable. Acked-by: Michal Simek <michal.si...@xilinx.com> (For ZynqMP) M _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot