]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
mmc: sunxi: Cleanup "new timing mode" selection
authorAndre Przywara <andre.przywara@arm.com>
Wed, 5 May 2021 08:57:47 +0000 (09:57 +0100)
committerAndre Przywara <andre.przywara@arm.com>
Sat, 10 Jul 2021 00:22:09 +0000 (01:22 +0100)
Among the SoCs using the "new timing mode", only the A83T needs to
explicitly switch to that mode.

By just defining the symbol for that one odd A83T bit to 0 for any other
SoCs, we can always OR that in, and save the confusing nested #ifdefs.

Clean up the also confusing new_mode setting on the way.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
drivers/mmc/sunxi_mmc.c

index bc68debdad6559af72469e9bc0995f565771e9f2..33cedb4edbaf392ea1d00e50542b26d64370f21b 100644 (file)
 #include <asm-generic/gpio.h>
 #include <linux/delay.h>
 
+#ifndef CCM_MMC_CTRL_MODE_SEL_NEW
+#define CCM_MMC_CTRL_MODE_SEL_NEW      0
+#endif
+
 struct sunxi_mmc_plat {
        struct mmc_config cfg;
        struct mmc mmc;
@@ -102,13 +106,10 @@ static int mmc_resource_init(int sdc_no)
 static int mmc_set_mod_clk(struct sunxi_mmc_priv *priv, unsigned int hz)
 {
        unsigned int pll, pll_hz, div, n, oclk_dly, sclk_dly;
-       bool new_mode = true;
+       bool new_mode = IS_ENABLED(CONFIG_MMC_SUNXI_HAS_NEW_MODE);
        bool calibrate = false;
        u32 val = 0;
 
-       if (!IS_ENABLED(CONFIG_MMC_SUNXI_HAS_NEW_MODE))
-               new_mode = false;
-
        /* A83T support new mode only on eMMC */
        if (IS_ENABLED(CONFIG_MACH_SUN8I_A83T) && priv->mmc_no != 2)
                new_mode = false;
@@ -176,12 +177,8 @@ static int mmc_set_mod_clk(struct sunxi_mmc_priv *priv, unsigned int hz)
        }
 
        if (new_mode) {
-#ifdef CONFIG_MMC_SUNXI_HAS_NEW_MODE
-#ifdef CONFIG_MMC_SUNXI_HAS_MODE_SWITCH
-               val = CCM_MMC_CTRL_MODE_SEL_NEW;
-#endif
+               val |= CCM_MMC_CTRL_MODE_SEL_NEW;
                setbits_le32(&priv->reg->ntsr, SUNXI_MMC_NTSR_MODE_SEL_NEW);
-#endif
        } else if (!calibrate) {
                /*
                 * Use hardcoded delay values if controller doesn't support