From c48c142be69a3891cbd9bcc9c3ab5816f8152723 Mon Sep 17 00:00:00 2001 From: Russell King Date: Sat, 3 Sep 2016 23:01:43 +0100 Subject: 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 --- arch/arm/mach-pxa/colibri-pxa270.c | 24 ++++++++- arch/arm/mach-pxa/colibri-pxa320.c | 23 +++++++++ drivers/pcmcia/pxa2xx_colibri.c | 101 ++++++------------------------------- 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 * Marek Vasut */ - +#include #include #include #include @@ -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 #include #include +#include #include #include @@ -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 */ +#include #include #include -#include -#include #include #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)); -- cgit