summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRussell King <rmk+kernel@armlinux.org.uk>2016-09-03 23:01:43 +0100
committerRussell King <rmk+kernel@armlinux.org.uk>2020-10-12 21:55:24 +0100
commitc48c142be69a3891cbd9bcc9c3ab5816f8152723 (patch)
treee549c149d08c88f0f62caea027fa48fa5d2677c9
parent64ba939dd2beab5c66a5c16db9e5dfabb11a2a15 (diff)
ARM: pxa/colibri: switch pcmcia to gpiod APIs
Switch the Colibri family of devices to use the gpiod APIs rather than listing the GPIOs and interrupts in the board specific PCMCIA driver. Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
-rw-r--r--arch/arm/mach-pxa/colibri-pxa270.c24
-rw-r--r--arch/arm/mach-pxa/colibri-pxa320.c23
-rw-r--r--drivers/pcmcia/pxa2xx_colibri.c101
3 files changed, 62 insertions, 86 deletions
diff --git a/arch/arm/mach-pxa/colibri-pxa270.c b/arch/arm/mach-pxa/colibri-pxa270.c
index 2f2cd2ae4187..3dcc122c1021 100644
--- a/arch/arm/mach-pxa/colibri-pxa270.c
+++ b/arch/arm/mach-pxa/colibri-pxa270.c
@@ -6,7 +6,7 @@
* Daniel Mack <daniel@caiaq.de>
* Marek Vasut <marek.vasut@gmail.com>
*/
-
+#include <linux/gpio/machine.h>
#include <linux/init.h>
#include <linux/interrupt.h>
#include <linux/moduleparam.h>
@@ -76,8 +76,29 @@ static mfp_cfg_t colibri_pxa270_evalboard_pin_config[] __initdata = {
GPIO117_I2C_SCL,
GPIO118_I2C_SDA,
};
+
+static struct gpiod_lookup_table colibri_pxa270_pcmcia_gpio_table = {
+ .dev_id = "pxa2xx-pcmcia",
+ .table = {
+ GPIO_LOOKUP("GPIO", 1, "ready", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 53, "reset", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 82, "bvd2", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 83, "bvd1", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 84, "detect", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 107, "ppen", GPIO_ACTIVE_HIGH),
+ { },
+ },
+};
+
+static void colibri_pxa270_init_pcmcia(void)
+{
+ gpiod_add_lookup_table(&colibri_pxa270_pcmcia_gpio_table);
+}
#else
static mfp_cfg_t colibri_pxa270_evalboard_pin_config[] __initdata = {};
+static void colibri_pxa270_init_pcmcia(void)
+{
+}
#endif
#ifdef CONFIG_MACH_COLIBRI_PXA270_INCOME
@@ -282,6 +303,7 @@ static void __init colibri_pxa270_init(void)
case COLIBRI_EVALBOARD:
pxa2xx_mfp_config(ARRAY_AND_SIZE(
colibri_pxa270_evalboard_pin_config));
+ colibri_pxa270_init_pcmcia();
colibri_evalboard_init();
break;
case COLIBRI_PXA270_INCOME:
diff --git a/arch/arm/mach-pxa/colibri-pxa320.c b/arch/arm/mach-pxa/colibri-pxa320.c
index 35dd3adb7712..1109e10f7022 100644
--- a/arch/arm/mach-pxa/colibri-pxa320.c
+++ b/arch/arm/mach-pxa/colibri-pxa320.c
@@ -13,6 +13,7 @@
#include <linux/platform_device.h>
#include <linux/gpio/machine.h>
#include <linux/gpio.h>
+#include <linux/gpio/machine.h>
#include <linux/interrupt.h>
#include <asm/mach-types.h>
@@ -91,8 +92,29 @@ static mfp_cfg_t colibri_pxa320_evalboard_pin_config[] __initdata = {
GPIO79_GPIO, /* PBVD2 */
GPIO54_GPIO, /* POE */
};
+
+static struct gpiod_lookup_table colibri_pxa320_pcmcia_gpio_table = {
+ .dev_id = "pxa2xx-pcmcia",
+ .table = {
+ GPIO_LOOKUP("GPIO", 29, "ready", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 53, "bvd1", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 57, "ppen", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 77, "reset", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 79, "bvd2", GPIO_ACTIVE_HIGH),
+ GPIO_LOOKUP("GPIO", 81, "detect", GPIO_ACTIVE_HIGH),
+ { },
+ },
+};
+
+static void colibri_pxa320_init_pcmcia(void)
+{
+ gpiod_add_lookup_table(&colibri_pxa320_pcmcia_gpio_table);
+}
#else
static mfp_cfg_t colibri_pxa320_evalboard_pin_config[] __initdata = {};
+static void colibri_pxa320_init_pcmcia(void)
+{
+}
#endif
#if defined(CONFIG_AX88796)
@@ -245,6 +267,7 @@ void __init colibri_pxa320_init(void)
colibri_pxa3xx_init_lcd(mfp_to_gpio(GPIO49_GPIO));
colibri_pxa320_init_ac97();
colibri_pxa320_init_udc();
+ colibri_pxa320_init_pcmcia();
/* Evalboard init */
pxa3xx_mfp_config(ARRAY_AND_SIZE(colibri_pxa320_evalboard_pin_config));
diff --git a/drivers/pcmcia/pxa2xx_colibri.c b/drivers/pcmcia/pxa2xx_colibri.c
index f0f725e99604..824c43c12e88 100644
--- a/drivers/pcmcia/pxa2xx_colibri.c
+++ b/drivers/pcmcia/pxa2xx_colibri.c
@@ -7,91 +7,41 @@
* Copyright (C) 2010 Marek Vasut <marek.vasut@gmail.com>
*/
+#include <linux/gpio/consumer.h>
#include <linux/module.h>
#include <linux/platform_device.h>
-#include <linux/delay.h>
-#include <linux/gpio.h>
#include <asm/mach-types.h>
#include "soc_common.h"
-#define COLIBRI270_RESET_GPIO 53
-#define COLIBRI270_PPEN_GPIO 107
-#define COLIBRI270_BVD1_GPIO 83
-#define COLIBRI270_BVD2_GPIO 82
-#define COLIBRI270_DETECT_GPIO 84
-#define COLIBRI270_READY_GPIO 1
-
-#define COLIBRI320_RESET_GPIO 77
-#define COLIBRI320_PPEN_GPIO 57
-#define COLIBRI320_BVD1_GPIO 53
-#define COLIBRI320_BVD2_GPIO 79
-#define COLIBRI320_DETECT_GPIO 81
-#define COLIBRI320_READY_GPIO 29
-
-enum {
- DETECT = 0,
- READY = 1,
- BVD1 = 2,
- BVD2 = 3,
- PPEN = 4,
- RESET = 5,
-};
-
-/* Contents of this array are configured on-the-fly in init function */
-static struct gpio colibri_pcmcia_gpios[] = {
- { 0, GPIOF_IN, "PCMCIA Detect" },
- { 0, GPIOF_IN, "PCMCIA Ready" },
- { 0, GPIOF_IN, "PCMCIA BVD1" },
- { 0, GPIOF_IN, "PCMCIA BVD2" },
- { 0, GPIOF_INIT_LOW, "PCMCIA PPEN" },
- { 0, GPIOF_INIT_HIGH,"PCMCIA Reset" },
-};
+static struct gpio_desc *ppen_gpio;
static int colibri_pcmcia_hw_init(struct soc_pcmcia_socket *skt)
{
- int ret;
+ struct device *dev = skt->socket.dev.parent;
- ret = gpio_request_array(colibri_pcmcia_gpios,
- ARRAY_SIZE(colibri_pcmcia_gpios));
- if (ret)
- goto err1;
+ skt->stat[SOC_STAT_CD].name = "detect";
+ skt->stat[SOC_STAT_BVD1].name = "bvd1";
+ skt->stat[SOC_STAT_BVD2].name = "bvd2";
+ skt->stat[SOC_STAT_RDY].name = "ready";
- skt->socket.pci_irq = gpio_to_irq(colibri_pcmcia_gpios[READY].gpio);
- skt->stat[SOC_STAT_CD].irq = gpio_to_irq(colibri_pcmcia_gpios[DETECT].gpio);
- skt->stat[SOC_STAT_CD].name = "PCMCIA CD";
+ skt->gpio_reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH);
+ if (IS_ERR(skt->gpio_reset))
+ return PTR_ERR(skt->gpio_reset);
-err1:
- return ret;
-}
+ ppen_gpio = devm_gpiod_get(dev, "ppen", GPIOD_OUT_LOW);
+ if (IS_ERR(ppen_gpio))
+ return PTR_ERR(ppen_gpio);
-static void colibri_pcmcia_hw_shutdown(struct soc_pcmcia_socket *skt)
-{
- gpio_free_array(colibri_pcmcia_gpios,
- ARRAY_SIZE(colibri_pcmcia_gpios));
-}
-
-static void colibri_pcmcia_socket_state(struct soc_pcmcia_socket *skt,
- struct pcmcia_state *state)
-{
-
- state->detect = !!gpio_get_value(colibri_pcmcia_gpios[DETECT].gpio);
- state->ready = !!gpio_get_value(colibri_pcmcia_gpios[READY].gpio);
- state->bvd1 = !!gpio_get_value(colibri_pcmcia_gpios[BVD1].gpio);
- state->bvd2 = !!gpio_get_value(colibri_pcmcia_gpios[BVD2].gpio);
- state->vs_3v = 1;
- state->vs_Xv = 0;
+ return soc_pcmcia_request_gpiods(skt);
}
static int
colibri_pcmcia_configure_socket(struct soc_pcmcia_socket *skt,
const socket_state_t *state)
{
- gpio_set_value(colibri_pcmcia_gpios[PPEN].gpio,
- !(state->Vcc == 33 && state->Vpp < 50));
- gpio_set_value(colibri_pcmcia_gpios[RESET].gpio,
- state->flags & SS_RESET);
+ gpiod_set_value(ppen_gpio, !(state->Vcc == 33 && state->Vpp < 50));
return 0;
}
@@ -102,9 +52,8 @@ static struct pcmcia_low_level colibri_pcmcia_ops = {
.nr = 1,
.hw_init = colibri_pcmcia_hw_init,
- .hw_shutdown = colibri_pcmcia_hw_shutdown,
- .socket_state = colibri_pcmcia_socket_state,
+ .socket_state = soc_common_cf_socket_state,
.configure_socket = colibri_pcmcia_configure_socket,
};
@@ -121,24 +70,6 @@ static int __init colibri_pcmcia_init(void)
if (!colibri_pcmcia_device)
return -ENOMEM;
- /* Colibri PXA270 */
- if (machine_is_colibri()) {
- colibri_pcmcia_gpios[RESET].gpio = COLIBRI270_RESET_GPIO;
- colibri_pcmcia_gpios[PPEN].gpio = COLIBRI270_PPEN_GPIO;
- colibri_pcmcia_gpios[BVD1].gpio = COLIBRI270_BVD1_GPIO;
- colibri_pcmcia_gpios[BVD2].gpio = COLIBRI270_BVD2_GPIO;
- colibri_pcmcia_gpios[DETECT].gpio = COLIBRI270_DETECT_GPIO;
- colibri_pcmcia_gpios[READY].gpio = COLIBRI270_READY_GPIO;
- /* Colibri PXA320 */
- } else if (machine_is_colibri320()) {
- colibri_pcmcia_gpios[RESET].gpio = COLIBRI320_RESET_GPIO;
- colibri_pcmcia_gpios[PPEN].gpio = COLIBRI320_PPEN_GPIO;
- colibri_pcmcia_gpios[BVD1].gpio = COLIBRI320_BVD1_GPIO;
- colibri_pcmcia_gpios[BVD2].gpio = COLIBRI320_BVD2_GPIO;
- colibri_pcmcia_gpios[DETECT].gpio = COLIBRI320_DETECT_GPIO;
- colibri_pcmcia_gpios[READY].gpio = COLIBRI320_READY_GPIO;
- }
-
ret = platform_device_add_data(colibri_pcmcia_device,
&colibri_pcmcia_ops, sizeof(colibri_pcmcia_ops));