diff options
author | Russell King <rmk+kernel@armlinux.org.uk> | 2016-09-03 23:01:43 +0100 |
---|---|---|
committer | Russell King <rmk+kernel@armlinux.org.uk> | 2020-10-12 21:55:24 +0100 |
commit | c48c142be69a3891cbd9bcc9c3ab5816f8152723 (patch) | |
tree | e549c149d08c88f0f62caea027fa48fa5d2677c9 | |
parent | 64ba939dd2beab5c66a5c16db9e5dfabb11a2a15 (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.c | 24 | ||||
-rw-r--r-- | arch/arm/mach-pxa/colibri-pxa320.c | 23 | ||||
-rw-r--r-- | 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 <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)); |