From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Sat, 27 Jan 2018 23:12:03 +0100 Subject: [U-Boot] [PATCH v1] spl: eMMC/SD: Provide one __weak spl_boot_mode() function In-Reply-To: <656397bc-b43e-4f76-b743-712a3a92daf5@denx.de> References: <20180127215547.19111-1-lukma@denx.de> <656397bc-b43e-4f76-b743-712a3a92daf5@denx.de> Message-ID: <20180127231203.2d2e4fff@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.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 > > > > --- > > 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 > 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: