[U-Boot] [PATCH v1] spl: eMMC/SD: Provide one __weak spl_boot_mode() function

[Lukasz Majewski <lukma@denx.de>]

Hi Marek,

> On 01/27/2018 10:55 PM, 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.  
> 
> It always was ;-)

But some way forgotten :-)

> 
> > To achieve the goal, all per mach/arch implementations eligible for
> > unification has been replaced with one __weak implementation.
> > 
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > 
> > ---
> > Comments regarding cleanup (to help maintainers assessment if those
> > changes break anything):  
> 
> You should run it through Travis at least, to detect if something
> stopped building.

https://travis-ci.org/lmajewski/u-boot-dfu

All "green".

> 
> > 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 (will hang in spl_mmc_load_image() anyway)
> >         arch/arm/cpu/armv8/zynqmp/spl.c
> > 
> > - fsl -> here we had a return of BOOT_DEVICE_NAND, but the
> > spl_boot_mode is eMMC/SD card specific
> >         (will hang in spl_mmc_load_image() anyway)
> > 
> >         arch/arm/cpu/armv8/fsl-layerscape/spl.c
> >         arch/arm/cpu/armv7/ls102xa/spl.c
> >         arch/arm/cpu/arm1136/mx35/generic.c
> > 
> > Test HW:
> > 
> > iMX6Q - display5 board  
> Reviewed-by: Marek Vasut <marex@denx.de>
> 




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180127/4490b3b1/attachment.sig>