mmc: extend and rename cd-gpio helpers to handle more slot GPIO functions
authorGuennadi Liakhovetski <g.liakhovetski@gmx.de>
Mon, 30 Apr 2012 21:31:57 +0000 (23:31 +0200)
committerChris Ball <cjb@laptop.org>
Wed, 11 Jul 2012 03:04:04 +0000 (23:04 -0400)
GPIOs can be used in MMC/SD-card slots not only for hotplug detection, but
also to implement the write-protection pin. Rename cd-gpio helpers to
slot-gpio to make addition of further slot GPIO functions possible.

Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Chris Ball <cjb@laptop.org>
drivers/mmc/core/Makefile
drivers/mmc/core/slot-gpio.c [moved from drivers/mmc/core/cd-gpio.c with 53% similarity]
drivers/mmc/host/tmio_mmc_pio.c
include/linux/mmc/slot-gpio.h [moved from include/linux/mmc/cd-gpio.h with 68% similarity]

index dca4428..38ed210 100644 (file)
@@ -7,6 +7,6 @@ mmc_core-y                      := core.o bus.o host.o \
                                   mmc.o mmc_ops.o sd.o sd_ops.o \
                                   sdio.o sdio_ops.o sdio_bus.o \
                                   sdio_cis.o sdio_io.o sdio_irq.o \
-                                  quirks.o cd-gpio.o
+                                  quirks.o slot-gpio.o
 
 mmc_core-$(CONFIG_DEBUG_FS)    += debugfs.o
similarity index 53%
rename from drivers/mmc/core/cd-gpio.c
rename to drivers/mmc/core/slot-gpio.c
index 8f5dc08..9796710 100644 (file)
 #include <linux/gpio.h>
 #include <linux/interrupt.h>
 #include <linux/jiffies.h>
-#include <linux/mmc/cd-gpio.h>
 #include <linux/mmc/host.h>
+#include <linux/mmc/slot-gpio.h>
 #include <linux/module.h>
 #include <linux/slab.h>
 
-struct mmc_cd_gpio {
-       unsigned int gpio;
-       char label[0];
+struct mmc_gpio {
+       unsigned int cd_gpio;
+       char cd_label[0];
 };
 
-static irqreturn_t mmc_cd_gpio_irqt(int irq, void *dev_id)
+static irqreturn_t mmc_gpio_cd_irqt(int irq, void *dev_id)
 {
        /* Schedule a card detection after a debounce timeout */
        mmc_detect_change(dev_id, msecs_to_jiffies(100));
        return IRQ_HANDLED;
 }
 
-int mmc_cd_gpio_request(struct mmc_host *host, unsigned int gpio)
+int mmc_gpio_request_cd(struct mmc_host *host, unsigned int gpio)
 {
        size_t len = strlen(dev_name(host->parent)) + 4;
-       struct mmc_cd_gpio *cd;
+       struct mmc_gpio *ctx;
        int irq = gpio_to_irq(gpio);
        int ret;
 
        if (irq < 0)
                return irq;
 
-       cd = kmalloc(sizeof(*cd) + len, GFP_KERNEL);
-       if (!cd)
+       ctx = kmalloc(sizeof(*ctx) + len, GFP_KERNEL);
+       if (!ctx)
                return -ENOMEM;
 
-       snprintf(cd->label, len, "%s cd", dev_name(host->parent));
+       snprintf(ctx->cd_label, len, "%s cd", dev_name(host->parent));
 
-       ret = gpio_request_one(gpio, GPIOF_DIR_IN, cd->label);
+       ret = gpio_request_one(gpio, GPIOF_DIR_IN, ctx->cd_label);
        if (ret < 0)
                goto egpioreq;
 
-       ret = request_threaded_irq(irq, NULL, mmc_cd_gpio_irqt,
-                                  IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING |
-                                  IRQF_ONESHOT, cd->label, host);
+       ret = request_threaded_irq(irq, NULL, mmc_gpio_cd_irqt,
+                       IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
+                       ctx->cd_label, host);
        if (ret < 0)
                goto eirqreq;
 
-       cd->gpio = gpio;
+       ctx->cd_gpio = gpio;
        host->hotplug.irq = irq;
-       host->hotplug.handler_priv = cd;
+       host->hotplug.handler_priv = ctx;
 
        return 0;
 
 eirqreq:
        gpio_free(gpio);
 egpioreq:
-       kfree(cd);
+       kfree(ctx);
        return ret;
 }
-EXPORT_SYMBOL(mmc_cd_gpio_request);
+EXPORT_SYMBOL(mmc_gpio_request_cd);
 
-void mmc_cd_gpio_free(struct mmc_host *host)
+void mmc_gpio_free_cd(struct mmc_host *host)
 {
-       struct mmc_cd_gpio *cd = host->hotplug.handler_priv;
+       struct mmc_gpio *ctx = host->hotplug.handler_priv;
 
-       if (!cd)
+       if (!ctx)
                return;
 
        free_irq(host->hotplug.irq, host);
-       gpio_free(cd->gpio);
-       kfree(cd);
+       gpio_free(ctx->cd_gpio);
+       kfree(ctx);
 }
-EXPORT_SYMBOL(mmc_cd_gpio_free);
+EXPORT_SYMBOL(mmc_gpio_free_cd);
index 0ad3917..7ffc489 100644 (file)
@@ -34,9 +34,9 @@
 #include <linux/io.h>
 #include <linux/irq.h>
 #include <linux/mfd/tmio.h>
-#include <linux/mmc/cd-gpio.h>
 #include <linux/mmc/host.h>
 #include <linux/mmc/mmc.h>
+#include <linux/mmc/slot-gpio.h>
 #include <linux/mmc/tmio.h>
 #include <linux/module.h>
 #include <linux/pagemap.h>
@@ -977,7 +977,7 @@ int __devinit tmio_mmc_host_probe(struct tmio_mmc_host **host,
        tmio_mmc_enable_mmc_irqs(_host, irq_mask);
 
        if (pdata->flags & TMIO_MMC_USE_GPIO_CD) {
-               ret = mmc_cd_gpio_request(mmc, pdata->cd_gpio);
+               ret = mmc_gpio_request_cd(mmc, pdata->cd_gpio);
                if (ret < 0) {
                        tmio_mmc_host_remove(_host);
                        return ret;
@@ -1009,7 +1009,7 @@ void tmio_mmc_host_remove(struct tmio_mmc_host *host)
                 * This means we can miss a card-eject, but this is anyway
                 * possible, because of delayed processing of hotplug events.
                 */
-               mmc_cd_gpio_free(mmc);
+               mmc_gpio_free_cd(mmc);
 
        if (!host->native_hotplug)
                pm_runtime_get_sync(&pdev->dev);
similarity index 68%
rename from include/linux/mmc/cd-gpio.h
rename to include/linux/mmc/slot-gpio.h
index cefaba0..edfaa32 100644 (file)
@@ -8,11 +8,11 @@
  * published by the Free Software Foundation.
  */
 
-#ifndef MMC_CD_GPIO_H
-#define MMC_CD_GPIO_H
+#ifndef MMC_SLOT_GPIO_H
+#define MMC_SLOT_GPIO_H
 
 struct mmc_host;
-int mmc_cd_gpio_request(struct mmc_host *host, unsigned int gpio);
-void mmc_cd_gpio_free(struct mmc_host *host);
+int mmc_gpio_request_cd(struct mmc_host *host, unsigned int gpio);
+void mmc_gpio_free_cd(struct mmc_host *host);
 
 #endif