dm: mmc: Move mmc_switch_part() above its callers
authorSimon Glass <sjg@chromium.org>
Sun, 1 May 2016 19:52:25 +0000 (13:52 -0600)
committerSimon Glass <sjg@chromium.org>
Tue, 17 May 2016 15:54:43 +0000 (09:54 -0600)
This function is defined after it is used. In preparation for making it
static, move it up a little. Also drop the printf() which should not appear
in a driver.

Signed-off-by: Simon Glass <sjg@chromium.org>
drivers/mmc/mmc.c

index 185d7b2a8e3a6a58a97b9d5c8628ef2db40a9c10..2211ac6d9948ea900262c50bd8ac50ad1f503d19 100644 (file)
@@ -582,30 +582,6 @@ static int mmc_set_capacity(struct mmc *mmc, int part_num)
        return 0;
 }
 
-int mmc_select_hwpart(int dev_num, int hwpart)
-{
-       struct mmc *mmc = find_mmc_device(dev_num);
-       int ret;
-
-       if (!mmc)
-               return -ENODEV;
-
-       if (mmc->block_dev.hwpart == hwpart)
-               return 0;
-
-       if (mmc->part_config == MMCPART_NOAVAILABLE) {
-               printf("Card doesn't support part_switch\n");
-               return -EMEDIUMTYPE;
-       }
-
-       ret = mmc_switch_part(dev_num, hwpart);
-       if (ret)
-               return ret;
-
-       return 0;
-}
-
-
 int mmc_switch_part(int dev_num, unsigned int part_num)
 {
        struct mmc *mmc = find_mmc_device(dev_num);
@@ -630,6 +606,27 @@ int mmc_switch_part(int dev_num, unsigned int part_num)
        return ret;
 }
 
+int mmc_select_hwpart(int dev_num, int hwpart)
+{
+       struct mmc *mmc = find_mmc_device(dev_num);
+       int ret;
+
+       if (!mmc)
+               return -ENODEV;
+
+       if (mmc->block_dev.hwpart == hwpart)
+               return 0;
+
+       if (mmc->part_config == MMCPART_NOAVAILABLE)
+               return -EMEDIUMTYPE;
+
+       ret = mmc_switch_part(dev_num, hwpart);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
 int mmc_hwpart_config(struct mmc *mmc,
                      const struct mmc_hwpart_conf *conf,
                      enum mmc_hwpart_conf_mode mode)