From 3ba0a25c444a0197662367131ca2533a4bccf16a Mon Sep 17 00:00:00 2001
From: Samuel Holland <samuel@sholland.org>
Date: Sun, 10 Apr 2022 00:13:33 -0500
Subject: [PATCH] sunxi: Skip MMC0 init when its pinmux conflicts with UART0
Currently, selecting UART0_PORT_F entirely disables MMC support on sunxi
platforms. But this is a bigger hammer then needed. Muxing UART0 to the
pins on port F only causes a conflict with MMC0, so minimize the impact
by specifically skipping MMC0 init. We can continue to use MMC1/2 if
those are enabled.
Let's also remove the preprocessor check while refacting this function.
Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Andre Przywara <andre.przywara@arm.com>
Signed-off-by: Andre Przywara <andre.przywara@arm.com>
---
board/sunxi/board.c | 22 ++++++++++------------
1 file changed, 10 insertions(+), 12 deletions(-)
diff --git a/board/sunxi/board.c b/board/sunxi/board.c
index 371ed9eeba..806e3bcb69 100644
--- a/board/sunxi/board.c
+++ b/board/sunxi/board.c
@@ -516,19 +516,17 @@ static void mmc_pinmux_setup(int sdc)
int board_mmc_init(struct bd_info *bis)
{
- __maybe_unused struct mmc *mmc0, *mmc1;
-
- mmc_pinmux_setup(CONFIG_MMC_SUNXI_SLOT);
- mmc0 = sunxi_mmc_init(CONFIG_MMC_SUNXI_SLOT);
- if (!mmc0)
- return -1;
+ if (!IS_ENABLED(CONFIG_UART0_PORT_F)) {
+ mmc_pinmux_setup(CONFIG_MMC_SUNXI_SLOT);
+ if (!sunxi_mmc_init(CONFIG_MMC_SUNXI_SLOT))
+ return -1;
+ }
-#if CONFIG_MMC_SUNXI_SLOT_EXTRA != -1
- mmc_pinmux_setup(CONFIG_MMC_SUNXI_SLOT_EXTRA);
- mmc1 = sunxi_mmc_init(CONFIG_MMC_SUNXI_SLOT_EXTRA);
- if (!mmc1)
- return -1;
-#endif
+ if (CONFIG_MMC_SUNXI_SLOT_EXTRA != -1) {
+ mmc_pinmux_setup(CONFIG_MMC_SUNXI_SLOT_EXTRA);
+ if (!sunxi_mmc_init(CONFIG_MMC_SUNXI_SLOT_EXTRA))
+ return -1;
+ }
return 0;
}
--
2.39.5