mmc: uniphier-sd: move uniphier_sd_init() below
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Thu, 25 Aug 2016 05:52:36 +0000 (14:52 +0900)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Sun, 28 Aug 2016 03:39:48 +0000 (12:39 +0900)
No more reason to define this function above the ops structure.
Move it near the caller.  Also, change its return type to void
because it never fails.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
drivers/mmc/uniphier-sd.c

index b8e784f59c99af241112d683d4c68d976377514d..b254c708577b26dca7c9404d0887e9797f34216d 100644 (file)
@@ -600,7 +600,24 @@ static int uniphier_sd_set_ios(struct udevice *dev)
        return 0;
 }
 
-static int uniphier_sd_init(struct uniphier_sd_priv *priv)
+static int uniphier_sd_get_cd(struct udevice *dev)
+{
+       struct uniphier_sd_priv *priv = dev_get_priv(dev);
+
+       if (priv->caps & UNIPHIER_SD_CAP_NONREMOVABLE)
+               return 1;
+
+       return !!(readl(priv->regbase + UNIPHIER_SD_INFO1) &
+                 UNIPHIER_SD_INFO1_CD);
+}
+
+static const struct dm_mmc_ops uniphier_sd_ops = {
+       .send_cmd = uniphier_sd_send_cmd,
+       .set_ios = uniphier_sd_set_ios,
+       .get_cd = uniphier_sd_get_cd,
+};
+
+static void uniphier_sd_host_init(struct uniphier_sd_priv *priv)
 {
        u32 tmp;
 
@@ -628,27 +645,8 @@ static int uniphier_sd_init(struct uniphier_sd_priv *priv)
                tmp |= UNIPHIER_SD_DMA_MODE_ADDR_INC;
                writel(tmp, priv->regbase + UNIPHIER_SD_DMA_MODE);
        }
-
-       return 0;
-}
-
-static int uniphier_sd_get_cd(struct udevice *dev)
-{
-       struct uniphier_sd_priv *priv = dev_get_priv(dev);
-
-       if (priv->caps & UNIPHIER_SD_CAP_NONREMOVABLE)
-               return 1;
-
-       return !!(readl(priv->regbase + UNIPHIER_SD_INFO1) &
-                 UNIPHIER_SD_INFO1_CD);
 }
 
-static const struct dm_mmc_ops uniphier_sd_ops = {
-       .send_cmd = uniphier_sd_send_cmd,
-       .set_ios = uniphier_sd_set_ios,
-       .get_cd = uniphier_sd_get_cd,
-};
-
 static int uniphier_sd_probe(struct udevice *dev)
 {
        struct uniphier_sd_priv *priv = dev_get_priv(dev);
@@ -715,7 +713,7 @@ static int uniphier_sd_probe(struct udevice *dev)
                priv->caps |= UNIPHIER_SD_CAP_DIV1024;
        }
 
-       uniphier_sd_init(priv);
+       uniphier_sd_host_init(priv);
 
        priv->cfg.voltages = MMC_VDD_165_195 | MMC_VDD_32_33 | MMC_VDD_33_34;
        priv->cfg.f_min = priv->mclk /