gpio: tegra: Hide tegra_gpio_enable/disable()

Recent pinctrl discussions concluded that gpiolib APIs should in fact do
whatever is required to mux a GPIO onto pins, by calling pinctrl APIs if
required. This change implements this for the Tegra GPIO driver, and removes
calls to the Tegra-specific APIs from drivers and board files.

Cc: Chris Ball <cjb@laptop.org>
Cc: linux-mmc@vger.kernel.org
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Acked-by: Chris Ball <cjb@laptop.org> # for sdhci-tegra.c
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Olof Johansson <olof@lixom.net>

10 files changed, 4 insertions, 121 deletions

diff --git a/arch/arm/mach-tegra/board-dt-tegra20.c b/arch/arm/mach-tegra/board-dt-tegra20.c
index 0952494f481a..65a5249b3718 100644
--- a/arch/arm/mach-tegra/board-dt-tegra20.c
+++ b/arch/arm/mach-tegra/board-dt-tegra20.c
@@ -55,7 +55,6 @@ void ventana_pinmux_init(void);
 
 struct of_dev_auxdata tegra20_auxdata_lookup[] __initdata = {
         OF_DEV_AUXDATA("nvidia,tegra20-pinmux", TEGRA_APB_MISC_BASE + 0x14, "tegra-pinmux", NULL),
-        OF_DEV_AUXDATA("nvidia,tegra20-gpio", TEGRA_GPIO_BASE, "tegra-gpio", NULL),
         OF_DEV_AUXDATA("nvidia,tegra20-sdhci", TEGRA_SDMMC1_BASE, "sdhci-tegra.0", NULL),
         OF_DEV_AUXDATA("nvidia,tegra20-sdhci", TEGRA_SDMMC2_BASE, "sdhci-tegra.1", NULL),
         OF_DEV_AUXDATA("nvidia,tegra20-sdhci", TEGRA_SDMMC3_BASE, "sdhci-tegra.2", NULL),
diff --git a/arch/arm/mach-tegra/board-harmony-pinmux.c b/arch/arm/mach-tegra/board-harmony-pinmux.c
index 1af85bccc0f1..3ebe2c859a3b 100644
--- a/arch/arm/mach-tegra/board-harmony-pinmux.c
+++ b/arch/arm/mach-tegra/board-harmony-pinmux.c
@@ -15,13 +15,11 @@
  */
 
 #include <linux/kernel.h>
-#include <linux/gpio.h>
 #include <linux/of.h>
 
 #include <mach/pinmux.h>
 #include <mach/pinmux-tegra20.h>
 
-#include "gpio-names.h"
 #include "board-harmony.h"
 #include "board-pinmux.h"
 
@@ -144,24 +142,9 @@ static struct tegra_pingroup_config harmony_pinmux[] = {
         {TEGRA_PINGROUP_XM2D,  TEGRA_MUX_NONE,          TEGRA_PUPD_NORMAL,    TEGRA_TRI_NORMAL},
 };
 
-static struct tegra_gpio_table gpio_table[] = {
-        { .gpio = TEGRA_GPIO_SD2_CD,            .enable = true  },
-        { .gpio = TEGRA_GPIO_SD2_WP,            .enable = true  },
-        { .gpio = TEGRA_GPIO_SD2_POWER,         .enable = true  },
-        { .gpio = TEGRA_GPIO_SD4_CD,            .enable = true  },
-        { .gpio = TEGRA_GPIO_SD4_WP,            .enable = true  },
-        { .gpio = TEGRA_GPIO_SD4_POWER,         .enable = true  },
-        { .gpio = TEGRA_GPIO_CDC_IRQ,           .enable = true  },
-        { .gpio = TEGRA_GPIO_HP_DET,            .enable = true  },
-        { .gpio = TEGRA_GPIO_INT_MIC_EN,        .enable = true  },
-        { .gpio = TEGRA_GPIO_EXT_MIC_EN,        .enable = true  },
-};
-
 static struct tegra_board_pinmux_conf conf = {
         .pgs = harmony_pinmux,
         .pg_count = ARRAY_SIZE(harmony_pinmux),
-        .gpios = gpio_table,
-        .gpio_count = ARRAY_SIZE(gpio_table),
 };
 
 void harmony_pinmux_init(void)
diff --git a/arch/arm/mach-tegra/board-paz00-pinmux.c b/arch/arm/mach-tegra/board-paz00-pinmux.c
index c775572dcea4..f0ec46612f52 100644
--- a/arch/arm/mach-tegra/board-paz00-pinmux.c
+++ b/arch/arm/mach-tegra/board-paz00-pinmux.c
@@ -15,13 +15,11 @@
  */
 
 #include <linux/kernel.h>
-#include <linux/gpio.h>
 #include <linux/of.h>
 
 #include <mach/pinmux.h>
 #include <mach/pinmux-tegra20.h>
 
-#include "gpio-names.h"
 #include "board-paz00.h"
 #include "board-pinmux.h"
 
@@ -144,21 +142,9 @@ static struct tegra_pingroup_config paz00_pinmux[] = {
         {TEGRA_PINGROUP_XM2D,  TEGRA_MUX_NONE,          TEGRA_PUPD_NORMAL,    TEGRA_TRI_NORMAL},
 };
 
-static struct tegra_gpio_table gpio_table[] = {
-        { .gpio = TEGRA_GPIO_SD1_CD,    .enable = true },
-        { .gpio = TEGRA_GPIO_SD1_WP,    .enable = true },
-        { .gpio = TEGRA_GPIO_SD1_POWER, .enable = true },
-        { .gpio = TEGRA_ULPI_RST,       .enable = true },
-        { .gpio = TEGRA_WIFI_PWRN,      .enable = true },
-        { .gpio = TEGRA_WIFI_RST,       .enable = true },
-        { .gpio = TEGRA_WIFI_LED,       .enable = true },
-};
-
 static struct tegra_board_pinmux_conf conf = {
         .pgs = paz00_pinmux,
         .pg_count = ARRAY_SIZE(paz00_pinmux),
-        .gpios = gpio_table,
-        .gpio_count = ARRAY_SIZE(gpio_table),
 };
 
 void paz00_pinmux_init(void)
diff --git a/arch/arm/mach-tegra/board-pinmux.c b/arch/arm/mach-tegra/board-pinmux.c
index adc3efe979b3..3015b5a38936 100644
--- a/arch/arm/mach-tegra/board-pinmux.c
+++ b/arch/arm/mach-tegra/board-pinmux.c
@@ -18,7 +18,6 @@
 #include <linux/of.h>
 #include <linux/string.h>
 
-#include <mach/gpio-tegra.h>
 #include <mach/pinmux.h>
 
 #include "board-pinmux.h"
@@ -26,18 +25,6 @@
 
 struct tegra_board_pinmux_conf *confs[2];
 
-static void tegra_board_pinmux_setup_gpios(void)
-{
-        int i;
-
-        for (i = 0; i < ARRAY_SIZE(confs); i++) {
-                if (!confs[i])
-                        continue;
-
-                tegra_gpio_config(confs[i]->gpios, confs[i]->gpio_count);
-        }
-}
-
 static void tegra_board_pinmux_setup_pinmux(void)
 {
         int i;
@@ -57,29 +44,17 @@ static void tegra_board_pinmux_setup_pinmux(void)
 static int tegra_board_pinmux_bus_notify(struct notifier_block *nb,
                                          unsigned long event, void *vdev)
 {
-        static bool had_gpio;
-        static bool had_pinmux;
-
         struct device *dev = vdev;
-        const char *devname;
 
         if (event != BUS_NOTIFY_BOUND_DRIVER)
                 return NOTIFY_DONE;
 
-        devname = dev_name(dev);
+        if (strcmp(dev_name(dev), PINMUX_DEV))
+                return NOTIFY_DONE;
 
-        if (!had_gpio && !strcmp(devname, GPIO_DEV)) {
-                tegra_board_pinmux_setup_gpios();
-                had_gpio = true;
-        } else if (!had_pinmux && !strcmp(devname, PINMUX_DEV)) {
-                tegra_board_pinmux_setup_pinmux();
-                had_pinmux = true;
-        }
+        tegra_board_pinmux_setup_pinmux();
 
-        if (had_gpio && had_pinmux)
-                return NOTIFY_STOP_MASK;
-        else
-                return NOTIFY_DONE;
+        return NOTIFY_STOP_MASK;
 }
 
 static struct notifier_block nb = {
diff --git a/arch/arm/mach-tegra/board-pinmux.h b/arch/arm/mach-tegra/board-pinmux.h
index 4aac73546f54..e08214d84bd9 100644
--- a/arch/arm/mach-tegra/board-pinmux.h
+++ b/arch/arm/mach-tegra/board-pinmux.h
@@ -15,11 +15,9 @@
 #ifndef __MACH_TEGRA_BOARD_PINMUX_H
 #define __MACH_TEGRA_BOARD_PINMUX_H
 
-#define GPIO_DEV "tegra-gpio"
 #define PINMUX_DEV "tegra-pinmux"
 
 struct tegra_pingroup_config;
-struct tegra_gpio_table;
 
 struct tegra_board_pinmux_conf {
         struct tegra_pingroup_config *pgs;
@@ -27,9 +25,6 @@ struct tegra_board_pinmux_conf {
 
         struct tegra_drive_pingroup_config *drives;
         int drive_count;
-
-        struct tegra_gpio_table *gpios;
-        int gpio_count;
 };
 
 void tegra_board_pinmux_init(struct tegra_board_pinmux_conf *conf_a,
diff --git a/arch/arm/mach-tegra/board-seaboard-pinmux.c b/arch/arm/mach-tegra/board-seaboard-pinmux.c
index 55e7e43a14ad..3bf7e9705b6a 100644
--- a/arch/arm/mach-tegra/board-seaboard-pinmux.c
+++ b/arch/arm/mach-tegra/board-seaboard-pinmux.c
@@ -15,13 +15,11 @@
 
 #include <linux/kernel.h>
 #include <linux/init.h>
-#include <linux/gpio.h>
 #include <linux/of.h>
 
 #include <mach/pinmux.h>
 #include <mach/pinmux-tegra20.h>
 
-#include "gpio-names.h"
 #include "board-pinmux.h"
 #include "board-seaboard.h"
 
@@ -179,35 +177,9 @@ static struct tegra_pingroup_config ventana_pinmux[] = {
         {TEGRA_PINGROUP_SPIG, TEGRA_MUX_SPI2_ALT, TEGRA_PUPD_NORMAL,    TEGRA_TRI_TRISTATE},
 };
 
-static struct tegra_gpio_table common_gpio_table[] = {
-        { .gpio = TEGRA_GPIO_SD2_CD,            .enable = true },
-        { .gpio = TEGRA_GPIO_SD2_WP,            .enable = true },
-        { .gpio = TEGRA_GPIO_SD2_POWER,         .enable = true },
-        { .gpio = TEGRA_GPIO_CDC_IRQ,           .enable = true },
-};
-
-static struct tegra_gpio_table seaboard_gpio_table[] = {
-        { .gpio = TEGRA_GPIO_LIDSWITCH,         .enable = true },
-        { .gpio = TEGRA_GPIO_POWERKEY,          .enable = true },
-        { .gpio = TEGRA_GPIO_HP_DET,            .enable = true },
-        { .gpio = TEGRA_GPIO_ISL29018_IRQ,      .enable = true },
-        { .gpio = TEGRA_GPIO_USB1,              .enable = true },
-};
-
-static struct tegra_gpio_table ventana_gpio_table[] = {
-        /* hp_det */
-        { .gpio = TEGRA_GPIO_PW2,               .enable = true },
-        /* int_mic_en */
-        { .gpio = TEGRA_GPIO_PX0,               .enable = true },
-        /* ext_mic_en */
-        { .gpio = TEGRA_GPIO_PX1,               .enable = true },
-};
-
 static struct tegra_board_pinmux_conf common_conf = {
         .pgs = common_pinmux,
         .pg_count = ARRAY_SIZE(common_pinmux),
-        .gpios = common_gpio_table,
-        .gpio_count = ARRAY_SIZE(common_gpio_table),
 };
 
 static struct tegra_board_pinmux_conf seaboard_conf = {
@@ -215,15 +187,11 @@ static struct tegra_board_pinmux_conf seaboard_conf = {
         .pg_count = ARRAY_SIZE(seaboard_pinmux),
         .drives = seaboard_drive_pinmux,
         .drive_count = ARRAY_SIZE(seaboard_drive_pinmux),
-        .gpios = seaboard_gpio_table,
-        .gpio_count = ARRAY_SIZE(seaboard_gpio_table),
 };
 
 static struct tegra_board_pinmux_conf ventana_conf = {
         .pgs = ventana_pinmux,
         .pg_count = ARRAY_SIZE(ventana_pinmux),
-        .gpios = ventana_gpio_table,
-        .gpio_count = ARRAY_SIZE(ventana_gpio_table),
 };
 
 void seaboard_pinmux_init(void)
diff --git a/arch/arm/mach-tegra/board-seaboard.c b/arch/arm/mach-tegra/board-seaboard.c
index 59987540df19..a0184fb44222 100644
--- a/arch/arm/mach-tegra/board-seaboard.c
+++ b/arch/arm/mach-tegra/board-seaboard.c
@@ -249,7 +249,6 @@ static void __init tegra_kaen_init(void)
         debug_uart_platform_data[0].irq = INT_UARTB;
 
         seaboard_audio_pdata.gpio_hp_mute = TEGRA_GPIO_KAEN_HP_MUTE;
-        tegra_gpio_enable(TEGRA_GPIO_KAEN_HP_MUTE);
 
         seaboard_common_init();
 
diff --git a/arch/arm/mach-tegra/board-trimslice-pinmux.c b/arch/arm/mach-tegra/board-trimslice-pinmux.c
index a21a2be57cb6..a1902d4e8e5d 100644
--- a/arch/arm/mach-tegra/board-trimslice-pinmux.c
+++ b/arch/arm/mach-tegra/board-trimslice-pinmux.c
@@ -13,7 +13,6 @@
  * GNU General Public License for more details.
  *
  */
-#include <linux/gpio.h>
 #include <linux/kernel.h>
 #include <linux/init.h>
 #include <linux/of.h>
@@ -21,7 +20,6 @@
 #include <mach/pinmux.h>
 #include <mach/pinmux-tegra20.h>
 
-#include "gpio-names.h"
 #include "board-pinmux.h"
 #include "board-trimslice.h"
 
@@ -144,19 +142,9 @@ static struct tegra_pingroup_config trimslice_pinmux[] = {
         {TEGRA_PINGROUP_XM2D,  TEGRA_MUX_NONE,          TEGRA_PUPD_NORMAL,      TEGRA_TRI_NORMAL},
 };
 
-static struct tegra_gpio_table gpio_table[] = {
-        { .gpio = TRIMSLICE_GPIO_SD4_CD, .enable = true }, /* mmc4 cd */
-        { .gpio = TRIMSLICE_GPIO_SD4_WP, .enable = true }, /* mmc4 wp */
-
-        { .gpio = TRIMSLICE_GPIO_USB1_MODE, .enable = true }, /* USB1 mode */
-        { .gpio = TRIMSLICE_GPIO_USB2_RST,  .enable = true }, /* USB2 PHY rst */
-};
-
 static struct tegra_board_pinmux_conf conf = {
         .pgs = trimslice_pinmux,
         .pg_count = ARRAY_SIZE(trimslice_pinmux),
-        .gpios = gpio_table,
-        .gpio_count = ARRAY_SIZE(gpio_table),
 };
 
 void trimslice_pinmux_init(void)
diff --git a/arch/arm/mach-tegra/include/mach/gpio-tegra.h b/arch/arm/mach-tegra/include/mach/gpio-tegra.h
index 6140820555e1..a978b3cc3a8d 100644
--- a/arch/arm/mach-tegra/include/mach/gpio-tegra.h
+++ b/arch/arm/mach-tegra/include/mach/gpio-tegra.h
@@ -25,13 +25,4 @@
 
 #define TEGRA_NR_GPIOS          INT_GPIO_NR
 
-struct tegra_gpio_table {
-        int     gpio;   /* GPIO number */
-        bool    enable; /* Enable for GPIO at init? */
-};
-
-void tegra_gpio_config(struct tegra_gpio_table *table, int num);
-void tegra_gpio_enable(int gpio);
-void tegra_gpio_disable(int gpio);
-
 #endif
diff --git a/arch/arm/mach-tegra/usb_phy.c b/arch/arm/mach-tegra/usb_phy.c
index c5b2ac04e2a0..d71d2fed6721 100644
--- a/arch/arm/mach-tegra/usb_phy.c
+++ b/arch/arm/mach-tegra/usb_phy.c
@@ -711,7 +711,6 @@ struct tegra_usb_phy *tegra_usb_phy_open(int instance, void __iomem *regs,
                         err = -ENXIO;
                         goto err1;
                 }
-                tegra_gpio_enable(ulpi_config->reset_gpio);
                 gpio_request(ulpi_config->reset_gpio, "ulpi_phy_reset_b");
                 gpio_direction_output(ulpi_config->reset_gpio, 0);
                 phy->ulpi = otg_ulpi_create(&ulpi_viewport_access_ops, 0);