mmc: host: tegra: get rid of tegra_gpio_enable()
Laxman Dewangan [Tue, 31 Jul 2012 09:48:54 +0000 (14:48 +0530)]
The gpio mode of pin is configured when setting
direction and hence this call is no more required.

Change-Id: I2c9bce83a283f6ef45754075cf3b7b1bc5c1d9ac
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
Reviewed-on: http://git-master/r/119612
Reviewed-by: Simone Willett <swillett@nvidia.com>
Tested-by: Simone Willett <swillett@nvidia.com>

drivers/mmc/host/sdhci-tegra.c

index 59c025a..8ff35e4 100644 (file)
@@ -992,7 +992,6 @@ static int __devinit sdhci_tegra_probe(struct platform_device *pdev)
                                "failed to allocate power gpio\n");
                        goto err_power_req;
                }
-               tegra_gpio_enable(plat->power_gpio);
                gpio_direction_output(plat->power_gpio, 1);
        }
 
@@ -1003,7 +1002,6 @@ static int __devinit sdhci_tegra_probe(struct platform_device *pdev)
                                "failed to allocate cd gpio\n");
                        goto err_cd_req;
                }
-               tegra_gpio_enable(plat->cd_gpio);
                gpio_direction_input(plat->cd_gpio);
 
                tegra_host->card_present = (gpio_get_value(plat->cd_gpio) == 0);
@@ -1038,7 +1036,6 @@ static int __devinit sdhci_tegra_probe(struct platform_device *pdev)
                                "failed to allocate wp gpio\n");
                        goto err_wp_req;
                }
-               tegra_gpio_enable(plat->wp_gpio);
                gpio_direction_input(plat->wp_gpio);
        }
 
@@ -1165,23 +1162,17 @@ err_add_host:
 err_clk_put:
        clk_put(pltfm_host->clk);
 err_clk_get:
-       if (gpio_is_valid(plat->wp_gpio)) {
-               tegra_gpio_disable(plat->wp_gpio);
+       if (gpio_is_valid(plat->wp_gpio))
                gpio_free(plat->wp_gpio);
-       }
 err_wp_req:
        if (gpio_is_valid(plat->cd_gpio))
                free_irq(gpio_to_irq(plat->cd_gpio), host);
 err_cd_irq_req:
-       if (gpio_is_valid(plat->cd_gpio)) {
-               tegra_gpio_disable(plat->cd_gpio);
+       if (gpio_is_valid(plat->cd_gpio))
                gpio_free(plat->cd_gpio);
-       }
 err_cd_req:
-       if (gpio_is_valid(plat->power_gpio)) {
-               tegra_gpio_disable(plat->power_gpio);
+       if (gpio_is_valid(plat->power_gpio))
                gpio_free(plat->power_gpio);
-       }
 err_power_req:
 err_no_mem:
        kfree(tegra_host);
@@ -1214,21 +1205,16 @@ static int __devexit sdhci_tegra_remove(struct platform_device *pdev)
                regulator_put(tegra_host->vdd_io_reg);
        }
 
-       if (gpio_is_valid(plat->wp_gpio)) {
-               tegra_gpio_disable(plat->wp_gpio);
+       if (gpio_is_valid(plat->wp_gpio))
                gpio_free(plat->wp_gpio);
-       }
 
        if (gpio_is_valid(plat->cd_gpio)) {
                free_irq(gpio_to_irq(plat->cd_gpio), host);
-               tegra_gpio_disable(plat->cd_gpio);
                gpio_free(plat->cd_gpio);
        }
 
-       if (gpio_is_valid(plat->power_gpio)) {
-               tegra_gpio_disable(plat->power_gpio);
+       if (gpio_is_valid(plat->power_gpio))
                gpio_free(plat->power_gpio);
-       }
 
        if (tegra_host->clk_enabled)
                clk_disable(pltfm_host->clk);