ARM: davinci: hawk: use gpio descriptor for mmc pins

Currently the mmc driver is polling the gpio to know if the
card was removed.

By using a gpio descriptor instead of the platform callbacks,
the driver will be able to register the gpio using the mmc core
APIs designed for this purpose.

This has the advantage that an irq will be registered, and
polling is no longer needed. Also, a dependency on platform
callbacks is removed for this board.

Signed-off-by: Axel Haslam <ahaslam@baylibre.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>

diff --git a/arch/arm/mach-davinci/board-omapl138-hawk.c b/arch/arm/mach-davinci/board-omapl138-hawk.c
index a4e87264ebd711476864008059ca80ba26361dc8..41d5500996b2d3f0da547c0725a5230f28c991f8 100644 (file)
--- a/arch/arm/mach-davinci/board-omapl138-hawk.c
+++ b/arch/arm/mach-davinci/board-omapl138-hawk.c
@@ -13,6 +13,7 @@
 #include <linux/init.h>
 #include <linux/console.h>
 #include <linux/gpio.h>
+#include <linux/gpio/machine.h>
 #include <linux/platform_data/gpio-davinci.h>
 #include <linux/regulator/machine.h>
 
@@ -25,8 +26,6 @@
 #include <mach/mux.h>
 
 #define HAWKBOARD_PHY_ID               "davinci_mdio-0:07"
-#define DA850_HAWK_MMCSD_CD_PIN                GPIO_TO_PIN(3, 12)
-#define DA850_HAWK_MMCSD_WP_PIN                GPIO_TO_PIN(3, 13)
 
 #define DA850_USB1_VBUS_PIN            GPIO_TO_PIN(2, 4)
 #define DA850_USB1_OC_PIN              GPIO_TO_PIN(6, 13)
@@ -123,19 +122,16 @@ static const short hawk_mmcsd0_pins[] = {
        -1
 };
 
-static int da850_hawk_mmc_get_ro(int index)
-{
-       return gpio_get_value(DA850_HAWK_MMCSD_WP_PIN);
-}
-
-static int da850_hawk_mmc_get_cd(int index)
-{
-       return !gpio_get_value(DA850_HAWK_MMCSD_CD_PIN);
-}
+static struct gpiod_lookup_table mmc_gpios_table = {
+       .dev_id = "da830-mmc.0",
+       .table = {
+               /* CD: gpio3_12: gpio60: chip 1 contains gpio range 32-63*/
+               GPIO_LOOKUP("davinci_gpio.1", 28, "cd", GPIO_ACTIVE_LOW),
+               GPIO_LOOKUP("davinci_gpio.1", 29, "wp", GPIO_ACTIVE_LOW),
+       },
+};
 
 static struct davinci_mmc_config da850_mmc_config = {
-       .get_ro         = da850_hawk_mmc_get_ro,
-       .get_cd         = da850_hawk_mmc_get_cd,
        .wires          = 4,
        .max_freq       = 50000000,
        .caps           = MMC_CAP_MMC_HIGHSPEED | MMC_CAP_SD_HIGHSPEED,
@@ -151,21 +147,7 @@ static __init void omapl138_hawk_mmc_init(void)
                return;
        }
 
-       ret = gpio_request_one(DA850_HAWK_MMCSD_CD_PIN,
-                       GPIOF_DIR_IN, "MMC CD");
-       if (ret < 0) {
-               pr_warn("%s: can not open GPIO %d\n",
-                       __func__, DA850_HAWK_MMCSD_CD_PIN);
-               return;
-       }
-
-       ret = gpio_request_one(DA850_HAWK_MMCSD_WP_PIN,
-                       GPIOF_DIR_IN, "MMC WP");
-       if (ret < 0) {
-               pr_warn("%s: can not open GPIO %d\n",
-                       __func__, DA850_HAWK_MMCSD_WP_PIN);
-               goto mmc_setup_wp_fail;
-       }
+       gpiod_add_lookup_table(&mmc_gpios_table);
 
        ret = da8xx_register_mmcsd0(&da850_mmc_config);
        if (ret) {
@@ -176,9 +158,7 @@ static __init void omapl138_hawk_mmc_init(void)
        return;
 
 mmc_setup_mmcsd_fail:
-       gpio_free(DA850_HAWK_MMCSD_WP_PIN);
-mmc_setup_wp_fail:
-       gpio_free(DA850_HAWK_MMCSD_CD_PIN);
+       gpiod_remove_lookup_table(&mmc_gpios_table);
 }
 
 static irqreturn_t omapl138_hawk_usb_ocic_irq(int irq, void *dev_id);