mmc: omap_hsmmc: remove unused slot_id parameter
authorAndreas Fenkart <afenkart@gmail.com>
Sat, 8 Nov 2014 14:33:17 +0000 (15:33 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 26 Nov 2014 13:30:56 +0000 (14:30 +0100)
omap_hsmmc only supports one slot. So slot id is always zero, and
slot id was never used in the callbacks anyway

Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
arch/arm/mach-omap2/board-rx51-peripherals.c
arch/arm/mach-omap2/hsmmc.c
arch/arm/mach-omap2/hsmmc.h
drivers/mmc/host/omap_hsmmc.c
include/linux/platform_data/hsmmc-omap.h

index 0a8ac844b75b4e2276b6c24f3434aec6c2658f02..3d5040f82e90a40e53320a40202afa20f27b29d2 100644 (file)
@@ -484,7 +484,7 @@ static struct omap_mux_partition *partition;
  * Current flows to eMMC when eMMC is off and the data lines are pulled up,
  * so pull them down. N.B. we pull 8 lines because we are using 8 lines.
  */
-static void rx51_mmc2_remux(struct device *dev, int slot, int power_on)
+static void rx51_mmc2_remux(struct device *dev, int power_on)
 {
        if (power_on)
                omap_mux_write_array(partition, rx51_mmc2_on_mux);
index 4e2896ad01196d965417c2b646eaa690ffb5f83c..dc6e79c4484abbfb097abb322298c37a8b9dd5ca 100644 (file)
@@ -33,14 +33,14 @@ static u16 control_devconf1_offset;
 
 #define HSMMC_NAME_LEN 9
 
-static void omap_hsmmc1_before_set_reg(struct device *dev, int slot,
-                                 int power_on, int vdd)
+static void omap_hsmmc1_before_set_reg(struct device *dev,
+                                      int power_on, int vdd)
 {
        u32 reg, prog_io;
        struct omap_hsmmc_platform_data *mmc = dev->platform_data;
 
        if (mmc->remux)
-               mmc->remux(dev, slot, power_on);
+               mmc->remux(dev, power_on);
 
        /*
         * Assume we power both OMAP VMMC1 (for CMD, CLK, DAT0..3) and the
@@ -86,8 +86,7 @@ static void omap_hsmmc1_before_set_reg(struct device *dev, int slot,
        }
 }
 
-static void omap_hsmmc1_after_set_reg(struct device *dev, int slot,
-                                int power_on, int vdd)
+static void omap_hsmmc1_after_set_reg(struct device *dev, int power_on, int vdd)
 {
        u32 reg;
 
@@ -122,20 +121,18 @@ static void hsmmc2_select_input_clk_src(struct omap_hsmmc_platform_data *mmc)
        omap_ctrl_writel(reg, control_devconf1_offset);
 }
 
-static void hsmmc2_before_set_reg(struct device *dev, int slot,
-                                  int power_on, int vdd)
+static void hsmmc2_before_set_reg(struct device *dev, int power_on, int vdd)
 {
        struct omap_hsmmc_platform_data *mmc = dev->platform_data;
 
        if (mmc->remux)
-               mmc->remux(dev, slot, power_on);
+               mmc->remux(dev, power_on);
 
        if (power_on)
                hsmmc2_select_input_clk_src(mmc);
 }
 
-static int am35x_hsmmc2_set_power(struct device *dev, int slot,
-                                 int power_on, int vdd)
+static int am35x_hsmmc2_set_power(struct device *dev, int power_on, int vdd)
 {
        struct omap_hsmmc_platform_data *mmc = dev->platform_data;
 
@@ -145,8 +142,7 @@ static int am35x_hsmmc2_set_power(struct device *dev, int slot,
        return 0;
 }
 
-static int nop_mmc_set_power(struct device *dev, int slot, int power_on,
-                                                       int vdd)
+static int nop_mmc_set_power(struct device *dev, int power_on, int vdd)
 {
        return 0;
 }
@@ -250,7 +246,6 @@ static int __init omap_hsmmc_pdata_init(struct omap2_hsmmc_info *c,
                snprintf(hc_name, (HSMMC_NAME_LEN + 1), "mmc%islot%i",
                                                                c->mmc, 1);
        mmc->name = hc_name;
-       mmc->nr_slots = 1;
        mmc->caps = c->caps;
        mmc->internal_clock = !c->ext_clock;
        mmc->reg_offset = 0;
index 30c78c17eb7e114a62f35f871f4381a2fdaf3524..148cd9b15499a4b11a5eef326c183168c3e82457 100644 (file)
@@ -23,7 +23,7 @@ struct omap2_hsmmc_info {
        struct platform_device *pdev;   /* mmc controller instance */
        int     ocr_mask;       /* temporary HACK */
        /* Remux (pad configuration) when powering on/off */
-       void (*remux)(struct device *dev, int slot, int power_on);
+       void (*remux)(struct device *dev, int power_on);
        /* init some special card */
        void (*init_card)(struct mmc_card *card);
 };
index f4f1bcd632f3b90e6a38dc506f535a8d5008528e..82b40b85293fd2110ff04a6933a5044ac0f3c3e2 100644 (file)
@@ -207,7 +207,6 @@ struct omap_hsmmc_host {
        int                     use_dma, dma_ch;
        struct dma_chan         *tx_chan;
        struct dma_chan         *rx_chan;
-       int                     slot_id;
        int                     response_busy;
        int                     context_loss;
        int                     protect_card;
@@ -223,8 +222,8 @@ struct omap_hsmmc_host {
        struct  omap_hsmmc_platform_data        *pdata;
 
        /* To handle board related suspend/resume functionality for MMC */
-       int (*suspend)(struct device *dev, int slot);
-       int (*resume)(struct device *dev, int slot);
+       int (*suspend)(struct device *dev);
+       int (*resume)(struct device *dev);
 
        /* return MMC cover switch state, can be NULL if not supported.
         *
@@ -232,13 +231,13 @@ struct omap_hsmmc_host {
         *   0 - closed
         *   1 - open
         */
-       int (*get_cover_state)(struct device *dev, int slot);
+       int (*get_cover_state)(struct device *dev);
 
        /* Card detection IRQs */
        int card_detect_irq;
 
-       int (*card_detect)(struct device *dev, int slot);
-       int (*get_ro)(struct device *dev, int slot);
+       int (*card_detect)(struct device *dev);
+       int (*get_ro)(struct device *dev);
 
 };
 
@@ -249,7 +248,7 @@ struct omap_mmc_of_data {
 
 static void omap_hsmmc_start_dma_transfer(struct omap_hsmmc_host *host);
 
-static int omap_hsmmc_card_detect(struct device *dev, int slot)
+static int omap_hsmmc_card_detect(struct device *dev)
 {
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
        struct omap_hsmmc_platform_data *mmc = host->pdata;
@@ -258,7 +257,7 @@ static int omap_hsmmc_card_detect(struct device *dev, int slot)
        return !gpio_get_value_cansleep(mmc->switch_pin);
 }
 
-static int omap_hsmmc_get_wp(struct device *dev, int slot)
+static int omap_hsmmc_get_wp(struct device *dev)
 {
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
        struct omap_hsmmc_platform_data *mmc = host->pdata;
@@ -267,7 +266,7 @@ static int omap_hsmmc_get_wp(struct device *dev, int slot)
        return gpio_get_value_cansleep(mmc->gpio_wp);
 }
 
-static int omap_hsmmc_get_cover_state(struct device *dev, int slot)
+static int omap_hsmmc_get_cover_state(struct device *dev)
 {
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
        struct omap_hsmmc_platform_data *mmc = host->pdata;
@@ -278,7 +277,7 @@ static int omap_hsmmc_get_cover_state(struct device *dev, int slot)
 
 #ifdef CONFIG_PM
 
-static int omap_hsmmc_suspend_cdirq(struct device *dev, int slot)
+static int omap_hsmmc_suspend_cdirq(struct device *dev)
 {
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
 
@@ -286,7 +285,7 @@ static int omap_hsmmc_suspend_cdirq(struct device *dev, int slot)
        return 0;
 }
 
-static int omap_hsmmc_resume_cdirq(struct device *dev, int slot)
+static int omap_hsmmc_resume_cdirq(struct device *dev)
 {
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
 
@@ -303,8 +302,7 @@ static int omap_hsmmc_resume_cdirq(struct device *dev, int slot)
 
 #ifdef CONFIG_REGULATOR
 
-static int omap_hsmmc_set_power(struct device *dev, int slot, int power_on,
-                                  int vdd)
+static int omap_hsmmc_set_power(struct device *dev, int power_on, int vdd)
 {
        struct omap_hsmmc_host *host =
                platform_get_drvdata(to_platform_device(dev));
@@ -318,7 +316,7 @@ static int omap_hsmmc_set_power(struct device *dev, int slot, int power_on,
                return 0;
 
        if (mmc_pdata(host)->before_set_reg)
-               mmc_pdata(host)->before_set_reg(dev, slot, power_on, vdd);
+               mmc_pdata(host)->before_set_reg(dev, power_on, vdd);
 
        if (host->pbias) {
                if (host->pbias_enabled == 1) {
@@ -381,7 +379,7 @@ static int omap_hsmmc_set_power(struct device *dev, int slot, int power_on,
        }
 
        if (mmc_pdata(host)->after_set_reg)
-               mmc_pdata(host)->after_set_reg(dev, slot, power_on, vdd);
+               mmc_pdata(host)->after_set_reg(dev, power_on, vdd);
 
 error_set_power:
        return ret;
@@ -431,8 +429,8 @@ static int omap_hsmmc_reg_get(struct omap_hsmmc_host *host)
            (host->vcc_aux && regulator_is_enabled(host->vcc_aux))) {
                int vdd = ffs(mmc_pdata(host)->ocr_mask) - 1;
 
-               mmc_pdata(host)->set_power(host->dev, host->slot_id, 1, vdd);
-               mmc_pdata(host)->set_power(host->dev, host->slot_id, 0, 0);
+               mmc_pdata(host)->set_power(host->dev, 1, vdd);
+               mmc_pdata(host)->set_power(host->dev, 0, 0);
        }
 
        return 0;
@@ -813,7 +811,7 @@ int omap_hsmmc_cover_is_closed(struct omap_hsmmc_host *host)
        int r = 1;
 
        if (host->get_cover_state)
-               r = host->get_cover_state(host->dev, host->slot_id);
+               r = host->get_cover_state(host->dev);
        return r;
 }
 
@@ -1231,12 +1229,11 @@ static int omap_hsmmc_switch_opcond(struct omap_hsmmc_host *host, int vdd)
                clk_disable_unprepare(host->dbclk);
 
        /* Turn the power off */
-       ret = mmc_pdata(host)->set_power(host->dev, host->slot_id, 0, 0);
+       ret = mmc_pdata(host)->set_power(host->dev, 0, 0);
 
        /* Turn the power ON with given VDD 1.8 or 3.0v */
        if (!ret)
-               ret = mmc_pdata(host)->set_power(host->dev, host->slot_id, 1,
-                                                vdd);
+               ret = mmc_pdata(host)->set_power(host->dev, 1, vdd);
        pm_runtime_get_sync(host->dev);
        if (host->dbclk)
                clk_prepare_enable(host->dbclk);
@@ -1284,7 +1281,7 @@ static void omap_hsmmc_protect_card(struct omap_hsmmc_host *host)
                return;
 
        host->reqs_blocked = 0;
-       if (host->get_cover_state(host->dev, host->slot_id)) {
+       if (host->get_cover_state(host->dev)) {
                if (host->protect_card) {
                        dev_info(host->dev, "%s: cover is closed, "
                                         "card is now accessible\n",
@@ -1312,7 +1309,7 @@ static irqreturn_t omap_hsmmc_detect(int irq, void *dev_id)
        sysfs_notify(&host->mmc->class_dev.kobj, NULL, "cover_switch");
 
        if (host->card_detect)
-               carddetect = host->card_detect(host->dev, host->slot_id);
+               carddetect = host->card_detect(host->dev);
        else {
                omap_hsmmc_protect_card(host);
                carddetect = -ENOSYS;
@@ -1638,12 +1635,10 @@ static void omap_hsmmc_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
        if (ios->power_mode != host->power_mode) {
                switch (ios->power_mode) {
                case MMC_POWER_OFF:
-                       mmc_pdata(host)->set_power(host->dev, host->slot_id,
-                                                  0, 0);
+                       mmc_pdata(host)->set_power(host->dev, 0, 0);
                        break;
                case MMC_POWER_UP:
-                       mmc_pdata(host)->set_power(host->dev, host->slot_id,
-                                                  1, ios->vdd);
+                       mmc_pdata(host)->set_power(host->dev, 1, ios->vdd);
                        break;
                case MMC_POWER_ON:
                        do_send_init_stream = 1;
@@ -1690,7 +1685,7 @@ static int omap_hsmmc_get_cd(struct mmc_host *mmc)
 
        if (!host->card_detect)
                return -ENOSYS;
-       return host->card_detect(host->dev, host->slot_id);
+       return host->card_detect(host->dev);
 }
 
 static int omap_hsmmc_get_ro(struct mmc_host *mmc)
@@ -1699,7 +1694,7 @@ static int omap_hsmmc_get_ro(struct mmc_host *mmc)
 
        if (!host->get_ro)
                return -ENOSYS;
-       return host->get_ro(host->dev, 0);
+       return host->get_ro(host->dev);
 }
 
 static void omap_hsmmc_init_card(struct mmc_host *mmc, struct mmc_card *card)
@@ -1996,8 +1991,6 @@ static struct omap_hsmmc_platform_data *of_get_hsmmc_pdata(struct device *dev)
        if (of_find_property(np, "ti,dual-volt", NULL))
                pdata->controller_flags |= OMAP_HSMMC_SUPPORTS_DUAL_VOLT;
 
-       /* This driver only supports 1 slot */
-       pdata->nr_slots = 1;
        pdata->switch_pin = cd_gpio;
        pdata->gpio_wp = wp_gpio;
 
@@ -2068,11 +2061,6 @@ static int omap_hsmmc_probe(struct platform_device *pdev)
                return -ENXIO;
        }
 
-       if (pdata->nr_slots == 0) {
-               dev_err(&pdev->dev, "No Slots\n");
-               return -ENXIO;
-       }
-
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        irq = platform_get_irq(pdev, 0);
        if (res == NULL || irq < 0)
@@ -2095,7 +2083,6 @@ static int omap_hsmmc_probe(struct platform_device *pdev)
        host->use_dma   = 1;
        host->dma_ch    = -1;
        host->irq       = irq;
-       host->slot_id   = 0;
        host->mapbase   = res->start + pdata->reg_offset;
        host->base      = base + pdata->reg_offset;
        host->power_mode = MMC_POWER_OFF;
@@ -2338,7 +2325,7 @@ static int omap_hsmmc_prepare(struct device *dev)
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
 
        if (host->suspend)
-               return host->suspend(dev, host->slot_id);
+               return host->suspend(dev);
 
        return 0;
 }
@@ -2348,7 +2335,7 @@ static void omap_hsmmc_complete(struct device *dev)
        struct omap_hsmmc_host *host = dev_get_drvdata(dev);
 
        if (host->resume)
-               host->resume(dev, host->slot_id);
+               host->resume(dev);
 
 }
 
index 68ffec14b56afdd2bde49cd20fc4ab51018dcb1d..67bbcf0785f61da1d4a832e016de1cb99dd313a9 100644 (file)
@@ -38,9 +38,6 @@ struct omap_hsmmc_platform_data {
        /* back-link to device */
        struct device *dev;
 
-       /* number of slots per controller */
-       unsigned nr_slots:2;
-
        /* set if your board has components or wiring that limits the
         * maximum frequency on the MMC bus */
        unsigned int max_freq;
@@ -79,15 +76,12 @@ struct omap_hsmmc_platform_data {
        int switch_pin;                 /* gpio (card detect) */
        int gpio_wp;                    /* gpio (write protect) */
 
-       int (*set_power)(struct device *dev, int slot,
-                        int power_on, int vdd);
-       void (*remux)(struct device *dev, int slot, int power_on);
+       int (*set_power)(struct device *dev, int power_on, int vdd);
+       void (*remux)(struct device *dev, int power_on);
        /* Call back before enabling / disabling regulators */
-       void (*before_set_reg)(struct device *dev, int slot,
-                              int power_on, int vdd);
+       void (*before_set_reg)(struct device *dev, int power_on, int vdd);
        /* Call back after enabling / disabling regulators */
-       void (*after_set_reg)(struct device *dev, int slot,
-                             int power_on, int vdd);
+       void (*after_set_reg)(struct device *dev, int power_on, int vdd);
        /* if we have special card, init it using this callback */
        void (*init_card)(struct mmc_card *card);