[PATCH 2/5] spl: mmc: extend spl_mmc_boot_mode() to take mmc argument

[Tan, Ley Foon]

> -----Original Message-----
> From: Andre Przywara <andre.przywara at arm.com>
> Sent: Sunday, November 8, 2020 9:14 PM
> To: Jagan Teki <jagan at amarulasolutions.com>
> Cc: Petr Štetiar <ynezz at true.cz>; Sunil Mohan Adapa <sunil at medhas.org>;
> Samuel Holland <samuel at sholland.org>; Aleksandr Aleksandrov
> <aleksandr.aleksandrov at emlid.com>; Icenowy Zheng <icenowy at aosc.io>;
> Simon Glass <sjg at chromium.org>; Tom Rini <trini at konsulko.com>; linux-
> sunxi at googlegroups.com; u-boot at lists.denx.de; Stefano Babic
> <sbabic at denx.de>; Fabio Estevam <festevam at gmail.com>; NXP i . MX U-
> Boot Team <uboot-imx at nxp.com>; Lokesh Vutla <lokeshvutla at ti.com>;
> Philipp Tomsich <philipp.tomsich at theobroma-systems.com>; Kever Yang
> <kever.yang at rock-chips.com>; Marek Vasut <marex at denx.de>; Simon
> Goldschmidt <simon.k.r.goldschmidt at gmail.com>; Tan, Ley Foon
> <ley.foon.tan at intel.com>; Patrick Delaunay <patrick.delaunay at st.com>;
> Patrice Chotard <patrice.chotard at st.com>; Andre Przywara
> <andre.przywara at arm.com>
> Subject: [PATCH 2/5] spl: mmc: extend spl_mmc_boot_mode() to take mmc
> argument
> 
> Platforms can overwrite the weak definition of spl_mmc_boot_mode() to
> determine where to load U-Boot proper from.
> For most of them this is a trivial decision based on Kconfig variables, but it
> might be desirable the probe the actual device to answer this question.
> 
> Pass the pointer to the mmc struct to that function, so implementations can
> make use of them.
> 
> Compile-tested for all users changed.
> 
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Fabio Estevam <festevam at gmail.com>
> Cc: NXP i.MX U-Boot Team <uboot-imx at nxp.com>
> Cc: Lokesh Vutla <lokeshvutla at ti.com>
> Cc: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
> Cc: Kever Yang <kever.yang at rock-chips.com>
> Cc: Marek Vasut <marex at denx.de>
> Cc: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> Cc: Ley Foon Tan <ley.foon.tan at intel.com>
> Cc: Patrick Delaunay <patrick.delaunay at st.com>
> Cc: Patrice Chotard <patrice.chotard at st.com>
> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
> ---
>  arch/arm/mach-imx/spl.c                | 2 +-
>  arch/arm/mach-k3/am6_init.c            | 2 +-
>  arch/arm/mach-k3/j721e_init.c          | 2 +-
>  arch/arm/mach-omap2/boot-common.c      | 2 +-
>  arch/arm/mach-rockchip/spl.c           | 2 +-
>  arch/arm/mach-socfpga/spl_a10.c        | 2 +-
>  arch/arm/mach-socfpga/spl_agilex.c     | 2 +-
>  arch/arm/mach-socfpga/spl_gen5.c       | 2 +-
>  arch/arm/mach-socfpga/spl_s10.c        | 2 +-
>  arch/arm/mach-stm32mp/spl.c            | 2 +-
>  arch/arm/mach-uniphier/mmc-boot-mode.c | 5 +----
>  common/spl/spl_mmc.c                   | 4 ++--
>  include/spl.h                          | 3 ++-
>  13 files changed, 15 insertions(+), 17 deletions(-)
> 

For SoCFPGA,
Reviewed-by: Ley Foon Tan <ley.foon.tan at inte.com>

Regards
Ley Foon