ARM: OMAP2: HSMMC: explicit fields to declare cover/card detect pin

board-rx51 has no card detect pin in the mmc slot, but can detect that
the (cell-phone) cover has been removed and the card is accessible.
The semantics between cover/card detect differ, the gpio on the slot
informs you after the card has been removed, cover removal does not
necessarily mean that the card has been removed.
This means different code paths are necessary. To complete this we
also want different fields in the platform data for cover and card
detect. This separation is not pushed all the way down into struct
omap2_hsmmc_info which is used to initialize the platform data.
If we did that we had to go over all board files and set the new
gpio_cod pin to -EINVAL. If we forget one board or some out-of-tree
archicture forgets that the default '0' is used which is a valid pin
number.

Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

1 files changed, 25 insertions, 8 deletions

diff --git a/arch/arm/mach-omap2/hsmmc.c b/arch/arm/mach-omap2/hsmmc.c
index dc6e79c4484a..9a8611ab5dfa 100644
--- a/arch/arm/mach-omap2/hsmmc.c
+++ b/arch/arm/mach-omap2/hsmmc.c
@@ -150,9 +150,13 @@ static int nop_mmc_set_power(struct device *dev, int power_on, int vdd)
 static inline void omap_hsmmc_mux(struct omap_hsmmc_platform_data
                                   *mmc_controller, int controller_nr)
 {
-        if (gpio_is_valid(mmc_controller->switch_pin) &&
-            (mmc_controller->switch_pin < OMAP_MAX_GPIO_LINES))
-                omap_mux_init_gpio(mmc_controller->switch_pin,
+        if (gpio_is_valid(mmc_controller->gpio_cd) &&
+            (mmc_controller->gpio_cd < OMAP_MAX_GPIO_LINES))
+                omap_mux_init_gpio(mmc_controller->gpio_cd,
+                                   OMAP_PIN_INPUT_PULLUP);
+        if (gpio_is_valid(mmc_controller->gpio_cod) &&
+            (mmc_controller->gpio_cod < OMAP_MAX_GPIO_LINES))
+                omap_mux_init_gpio(mmc_controller->gpio_cod,
                                    OMAP_PIN_INPUT_PULLUP);
         if (gpio_is_valid(mmc_controller->gpio_wp) &&
             (mmc_controller->gpio_wp < OMAP_MAX_GPIO_LINES))
@@ -250,15 +254,20 @@ static int __init omap_hsmmc_pdata_init(struct omap2_hsmmc_info *c,
         mmc->internal_clock = !c->ext_clock;
         mmc->reg_offset = 0;
 
-        mmc->switch_pin = c->gpio_cd;
+        if (c->cover_only) {
+                /* detect if mobile phone cover removed */
+                mmc->gpio_cd = -EINVAL;
+                mmc->gpio_cod = c->gpio_cd;
+        } else {
+                /* card detect pin on the mmc socket itself */
+                mmc->gpio_cd = c->gpio_cd;
+                mmc->gpio_cod = -EINVAL;
+        }
         mmc->gpio_wp = c->gpio_wp;
 
         mmc->remux = c->remux;
         mmc->init_card = c->init_card;
 
-        if (c->cover_only)
-                mmc->cover = 1;
-
         if (c->nonremovable)
                 mmc->nonremovable = 1;
 
@@ -358,7 +367,15 @@ void omap_hsmmc_late_init(struct omap2_hsmmc_info *c)
                 if (!mmc_pdata)
                         continue;
 
-                mmc_pdata->switch_pin = c->gpio_cd;
+                if (c->cover_only) {
+                        /* detect if mobile phone cover removed */
+                        mmc_pdata->gpio_cd = -EINVAL;
+                        mmc_pdata->gpio_cod = c->gpio_cd;
+                } else {
+                        /* card detect pin on the mmc socket itself */
+                        mmc_pdata->gpio_cd = c->gpio_cd;
+                        mmc_pdata->gpio_cod = -EINVAL;
+                }
                 mmc_pdata->gpio_wp = c->gpio_wp;
 
                 res = omap_device_register(pdev);