* [U-Boot] [PATCH 3/9 V3] add a new AT91 GPIO driver
@ 2010-01-23 11:03 Jens Scharsig
2010-01-31 1:08 ` Tom
2010-02-03 21:46 ` [U-Boot] [PATCH 3/9 V4] " Jens Scharsig
0 siblings, 2 replies; 3+ messages in thread
From: Jens Scharsig @ 2010-01-23 11:03 UTC (permalink / raw)
To: u-boot
* add a real AT91 GPIO driver instead of header inline code
* change board config files to use new driver
Signed-off-by: Jens Scharsig <js_at_ng@scharsoft.de>
---
drivers/gpio/Makefile | 1 +
drivers/gpio/at91_gpio.c | 240 ++++++++++++++++++++++++++++++++++++
include/asm-arm/arch-at91/gpio.h | 151 +----------------------
include/configs/afeb9260.h | 1 +
include/configs/at91cap9adk.h | 1 +
include/configs/at91sam9260ek.h | 2 +
include/configs/at91sam9261ek.h | 2 +
include/configs/at91sam9263ek.h | 2 +
include/configs/at91sam9m10g45ek.h | 2 +
include/configs/at91sam9rlek.h | 2 +
include/configs/cpu9260.h | 2 +
include/configs/meesc.h | 2 +
include/configs/pm9261.h | 2 +
include/configs/pm9263.h | 2 +
include/configs/sbc35_a9g20.h | 2 +
include/configs/tny_a9260.h | 2 +
16 files changed, 268 insertions(+), 148 deletions(-)
create mode 100644 drivers/gpio/at91_gpio.c
diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile
index acba56c..d966082 100644
--- a/drivers/gpio/Makefile
+++ b/drivers/gpio/Makefile
@@ -25,6 +25,7 @@ include $(TOPDIR)/config.mk
LIB := $(obj)libgpio.a
+COBJS-$(CONFIG_AT91_GPIO) += at91_gpio.o
COBJS-$(CONFIG_KIRKWOOD_GPIO) += kw_gpio.o
COBJS-$(CONFIG_MX31_GPIO) += mx31_gpio.o
COBJS-$(CONFIG_PCA953X) += pca953x.o
diff --git a/drivers/gpio/at91_gpio.c b/drivers/gpio/at91_gpio.c
new file mode 100644
index 0000000..5e50ead
--- /dev/null
+++ b/drivers/gpio/at91_gpio.c
@@ -0,0 +1,240 @@
+/*
+ * Memory Setup stuff - taken from blob memsetup.S
+ *
+ * Copyright (C) 2009 Jens Scharsig (js_at_ng at scharsoft.de)
+ *
+ * Copyright (C) 2005 HP Labs
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <common.h>
+#include <asm/sizes.h>
+#include <asm/arch/hardware.h>
+#include <asm/arch/io.h>
+#include <asm/arch/at91_pio.h>
+
+#define PIN_BASE 32
+
+u32 portpin_to_port(u32 portpin)
+{
+ return (portpin - PIN_BASE) / 32;
+}
+
+u32 portpin_to_pin(u32 portpin)
+{
+ return 1 << ((portpin - PIN_BASE) % 32);
+}
+
+/*
+ * mux the pin to the "GPIO" peripheral role.
+ */
+int at91_set_gpio_periph(unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ writel(mask, &pio->port[port].idr);
+ if (use_pullup)
+ writel(mask, &pio->port[port].puer);
+ else
+ writel(mask, &pio->port[port].pudr);
+ writel(mask, &pio->port[port].per);
+
+ return 0;
+}
+
+/*
+ * mux the pin to the "A" internal peripheral role.
+ */
+int at91_set_a_periph(unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ writel(mask, &pio->port[port].idr);
+ if (use_pullup)
+ writel(mask, &pio->port[port].puer);
+ else
+ writel(mask, &pio->port[port].pudr);
+ writel(mask, &pio->port[port].asr);
+ writel(mask, &pio->port[port].pdr);
+
+ return 0;
+}
+
+/*
+ * mux the pin to the "B" internal peripheral role.
+ */
+int at91_set_b_periph(unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ writel(mask, &pio->port[port].idr);
+ if (use_pullup)
+ writel(mask, &pio->port[port].puer);
+ else
+ writel(mask, &pio->port[port].pudr);
+ writel(mask, &pio->port[port].bsr);
+ writel(mask, &pio->port[port].pdr);
+
+ return 0;
+}
+
+/*
+ * mux the pin to the gpio controller (instead of "A" or "B" peripheral), and
+ * configure it for an input.
+ */
+int at91_set_gpio_input(u32 pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ writel(mask, &pio->port[port].idr);
+ if (use_pullup)
+ writel(mask, &pio->port[port].puer);
+ else
+ writel(mask, &pio->port[port].pudr);
+ writel(mask, &pio->port[port].odr);
+ writel(mask, &pio->port[port].per);
+ return 0;
+}
+
+/*
+ * mux the pin to the gpio controller (instead of "A" or "B" peripheral),
+ * and configure it for an output.
+ */
+int at91_set_gpio_output(u32 pin, int value)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ writel(mask, &pio->port[port].idr);
+ writel(mask, &pio->port[port].pudr);
+ if (value)
+ writel(mask, &pio->port[port].sodr);
+ else
+ writel(mask, &pio->port[port].codr);
+ writel(mask, &pio->port[port].oer);
+ writel(mask, &pio->port[port].per);
+
+ return 0;
+}
+
+/*
+ * enable/disable the glitch filter; mostly used with IRQ handling.
+ */
+int at91_set_deglitch(unsigned pin, int is_on)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ if (is_on)
+ writel(mask, &pio->port[port].ifer);
+ else
+ writel(mask, &pio->port[port].ifdr);
+ return 0;
+}
+
+/*
+ * enable/disable the multi-driver; This is only valid for output and
+ * allows the output pin to run as an open collector output.
+ */
+int at91_set_multi_drive(unsigned pin, int is_on)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ if (is_on)
+ writel(mask, &pio->port[port].mder);
+ else
+ writel(mask, &pio->port[port].mddr);
+ return 0;
+}
+
+/*
+int gpio_direction_input(unsigned pin)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ if (!(readl(&pio->port[port].psr) & mask))
+ return -EINVAL;
+ writel(mask, &pio->port[port].odr);
+ return 0;
+}
+
+int gpio_direction_output(unsigned pin, int value)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ if (!(readl(&pio->port[port].psr) & mask))
+ return -EINVAL;
+ if (value)
+ writel(mask, &pio->port[port].sodr);
+ else
+ writel(mask, &pio->port[port].codr);
+ writel(mask, &pio->port[port].oer);
+
+ return 0;
+}
+*/
+/*
+ * assuming the pin is muxed as a gpio output, set its value.
+ */
+int at91_set_gpio_value(unsigned pin, int value)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+
+ /* printf("%d = Port %d Pin %d (%d)",pin, port, mask, value); */
+ if (value)
+ writel(mask, &pio->port[port].sodr);
+ else
+ writel(mask, &pio->port[port].codr);
+ return 0;
+}
+
+/*
+ * read the pin's value (works even if it's not muxed as a gpio).
+ */
+int at91_get_gpio_value(unsigned pin)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 port = portpin_to_port(pin);
+ u32 mask = portpin_to_pin(pin);
+ u32 pdsr;
+
+ pdsr = readl(&pio->port[port]);
+ return (pdsr & mask) != 0;
+}
+
diff --git a/include/asm-arm/arch-at91/gpio.h b/include/asm-arm/arch-at91/gpio.h
index bc53171..b0c572f 100644
--- a/include/asm-arm/arch-at91/gpio.h
+++ b/include/asm-arm/arch-at91/gpio.h
@@ -216,155 +216,10 @@ static inline unsigned pin_to_mask(unsigned pin)
return 1 << (pin % 32);
}
-/*
- * mux the pin to the "GPIO" peripheral role.
- */
-static inline int at91_set_GPIO_periph(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_PER);
- return 0;
-}
-
-/*
- * mux the pin to the "A" internal peripheral role.
- */
-static inline int at91_set_A_periph(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_ASR);
- __raw_writel(mask, pio + PIO_PDR);
- return 0;
-}
-
-/*
- * mux the pin to the "B" internal peripheral role.
- */
-static inline int at91_set_B_periph(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_BSR);
- __raw_writel(mask, pio + PIO_PDR);
- return 0;
-}
-
-/*
- * mux the pin to the gpio controller (instead of "A" or "B" peripheral), and
- * configure it for an input.
- */
-static inline int at91_set_gpio_input(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_ODR);
- __raw_writel(mask, pio + PIO_PER);
- return 0;
-}
-
-/*
- * mux the pin to the gpio controller (instead of "A" or "B" peripheral),
- * and configure it for an output.
- */
-static inline int at91_set_gpio_output(unsigned pin, int value)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + PIO_PUDR);
- __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
- __raw_writel(mask, pio + PIO_OER);
- __raw_writel(mask, pio + PIO_PER);
- return 0;
-}
-
-/*
- * enable/disable the glitch filter; mostly used with IRQ handling.
- */
-static inline int at91_set_deglitch(unsigned pin, int is_on)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + (is_on ? PIO_IFER : PIO_IFDR));
- return 0;
-}
-
-/*
- * enable/disable the multi-driver; This is only valid for output and
- * allows the output pin to run as an open collector output.
- */
-static inline int at91_set_multi_drive(unsigned pin, int is_on)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + (is_on ? PIO_MDER : PIO_MDDR));
- return 0;
-}
-
-static inline int gpio_direction_input(unsigned pin)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- if (!(__raw_readl(pio + PIO_PSR) & mask))
- return -EINVAL;
- __raw_writel(mask, pio + PIO_ODR);
- return 0;
-}
+#define at91_set_GPIO_periph at91_set_gpio_periph
+#define at91_set_A_periph at91_set_a_periph
+#define at91_set_B_periph at91_set_b_periph
-static inline int gpio_direction_output(unsigned pin, int value)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- if (!(__raw_readl(pio + PIO_PSR) & mask))
- return -EINVAL;
- __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
- __raw_writel(mask, pio + PIO_OER);
- return 0;
-}
-
-/*
- * assuming the pin is muxed as a gpio output, set its value.
- */
-static inline int at91_set_gpio_value(unsigned pin, int value)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
- return 0;
-}
-
-/*
- * read the pin's value (works even if it's not muxed as a gpio).
- */
-static inline int at91_get_gpio_value(unsigned pin)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
- u32 pdsr;
-
- pdsr = __raw_readl(pio + PIO_PDSR);
- return (pdsr & mask) != 0;
-}
#endif
#endif
diff --git a/include/configs/afeb9260.h b/include/configs/afeb9260.h
index 3b69de8..9f8c567 100644
--- a/include/configs/afeb9260.h
+++ b/include/configs/afeb9260.h
@@ -47,6 +47,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91cap9adk.h b/include/configs/at91cap9adk.h
index 4c2782a..9da5846 100644
--- a/include/configs/at91cap9adk.h
+++ b/include/configs/at91cap9adk.h
@@ -49,6 +49,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
index a620d57..6d8969b 100644
--- a/include/configs/at91sam9260ek.h
+++ b/include/configs/at91sam9260ek.h
@@ -54,6 +54,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9261ek.h b/include/configs/at91sam9261ek.h
index 832b1cd..e67d899 100644
--- a/include/configs/at91sam9261ek.h
+++ b/include/configs/at91sam9261ek.h
@@ -52,6 +52,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9263ek.h b/include/configs/at91sam9263ek.h
index 1d82a15..dc010f2 100644
--- a/include/configs/at91sam9263ek.h
+++ b/include/configs/at91sam9263ek.h
@@ -51,6 +51,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9m10g45ek.h b/include/configs/at91sam9m10g45ek.h
index 50b118f..d034863 100644
--- a/include/configs/at91sam9m10g45ek.h
+++ b/include/configs/at91sam9m10g45ek.h
@@ -52,6 +52,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9rlek.h b/include/configs/at91sam9rlek.h
index 8db296a..423890b 100644
--- a/include/configs/at91sam9rlek.h
+++ b/include/configs/at91sam9rlek.h
@@ -49,6 +49,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/cpu9260.h b/include/configs/cpu9260.h
index e967e7c..50e31ff 100644
--- a/include/configs/cpu9260.h
+++ b/include/configs/cpu9260.h
@@ -244,6 +244,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/meesc.h b/include/configs/meesc.h
index 253a53d..13e60e7 100644
--- a/include/configs/meesc.h
+++ b/include/configs/meesc.h
@@ -58,6 +58,8 @@
*/
/* Console output */
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/pm9261.h b/include/configs/pm9261.h
index 26a2fad..25b6689 100644
--- a/include/configs/pm9261.h
+++ b/include/configs/pm9261.h
@@ -159,6 +159,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/pm9263.h b/include/configs/pm9263.h
index e55098c..f318a86 100644
--- a/include/configs/pm9263.h
+++ b/include/configs/pm9263.h
@@ -173,6 +173,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/sbc35_a9g20.h b/include/configs/sbc35_a9g20.h
index 7bdc729..3f0723f 100644
--- a/include/configs/sbc35_a9g20.h
+++ b/include/configs/sbc35_a9g20.h
@@ -59,6 +59,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART
#define CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/tny_a9260.h b/include/configs/tny_a9260.h
index 21475f8..466d27b 100644
--- a/include/configs/tny_a9260.h
+++ b/include/configs/tny_a9260.h
@@ -68,6 +68,8 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
+
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH 3/9 V3] add a new AT91 GPIO driver
2010-01-23 11:03 [U-Boot] [PATCH 3/9 V3] add a new AT91 GPIO driver Jens Scharsig
@ 2010-01-31 1:08 ` Tom
2010-02-03 21:46 ` [U-Boot] [PATCH 3/9 V4] " Jens Scharsig
1 sibling, 0 replies; 3+ messages in thread
From: Tom @ 2010-01-31 1:08 UTC (permalink / raw)
To: u-boot
Jens Scharsig wrote:
> * add a real AT91 GPIO driver instead of header inline code
> * change board config files to use new driver
>
> Signed-off-by: Jens Scharsig <js_at_ng@scharsoft.de>
> ---
> drivers/gpio/Makefile | 1 +
> drivers/gpio/at91_gpio.c | 240 ++++++++++++++++++++++++++++++++++++
> include/asm-arm/arch-at91/gpio.h | 151 +----------------------
> include/configs/afeb9260.h | 1 +
> include/configs/at91cap9adk.h | 1 +
> include/configs/at91sam9260ek.h | 2 +
> include/configs/at91sam9261ek.h | 2 +
> include/configs/at91sam9263ek.h | 2 +
> include/configs/at91sam9m10g45ek.h | 2 +
> include/configs/at91sam9rlek.h | 2 +
> include/configs/cpu9260.h | 2 +
> include/configs/meesc.h | 2 +
> include/configs/pm9261.h | 2 +
> include/configs/pm9263.h | 2 +
> include/configs/sbc35_a9g20.h | 2 +
> include/configs/tny_a9260.h | 2 +
> 16 files changed, 268 insertions(+), 148 deletions(-)
> create mode 100644 drivers/gpio/at91_gpio.c
>
> diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile
> index acba56c..d966082 100644
> --- a/drivers/gpio/Makefile
> +++ b/drivers/gpio/Makefile
> @@ -25,6 +25,7 @@ include $(TOPDIR)/config.mk
>
> LIB := $(obj)libgpio.a
>
> +COBJS-$(CONFIG_AT91_GPIO) += at91_gpio.o
> COBJS-$(CONFIG_KIRKWOOD_GPIO) += kw_gpio.o
> COBJS-$(CONFIG_MX31_GPIO) += mx31_gpio.o
> COBJS-$(CONFIG_PCA953X) += pca953x.o
> diff --git a/drivers/gpio/at91_gpio.c b/drivers/gpio/at91_gpio.c
> new file mode 100644
> index 0000000..5e50ead
> --- /dev/null
> +++ b/drivers/gpio/at91_gpio.c
> @@ -0,0 +1,240 @@
> +/*
> + * Memory Setup stuff - taken from blob memsetup.S
> + *
> + * Copyright (C) 2009 Jens Scharsig (js_at_ng at scharsoft.de)
> + *
> + * Copyright (C) 2005 HP Labs
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
ws
extra tab
Look for these globally
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */
> +
> +#include <config.h>
> +#include <common.h>
> +#include <asm/sizes.h>
> +#include <asm/arch/hardware.h>
> +#include <asm/arch/io.h>
> +#include <asm/arch/at91_pio.h>
> +
> +#define PIN_BASE 32
> +
> +u32 portpin_to_port(u32 portpin)
> +{
> + return (portpin - PIN_BASE) / 32;
> +}
> +
> +u32 portpin_to_pin(u32 portpin)
> +{
> + return 1 << ((portpin - PIN_BASE) % 32);
> +}
> +
This are only used locally
Change to static
May want to check input.
> +/*
> + * mux the pin to the "GPIO" peripheral role.
> + */
> +int at91_set_gpio_periph(unsigned pin, int use_pullup)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + writel(mask, &pio->port[port].idr);
> + if (use_pullup)
> + writel(mask, &pio->port[port].puer);
> + else
> + writel(mask, &pio->port[port].pudr);
> + writel(mask, &pio->port[port].per);
The next several functions are similar.
The common part could be pulled out into its own function
> +
> + return 0;
> +}
> +
> +/*
> + * mux the pin to the "A" internal peripheral role.
> + */
> +int at91_set_a_periph(unsigned pin, int use_pullup)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + writel(mask, &pio->port[port].idr);
> + if (use_pullup)
> + writel(mask, &pio->port[port].puer);
> + else
> + writel(mask, &pio->port[port].pudr);
> + writel(mask, &pio->port[port].asr);
> + writel(mask, &pio->port[port].pdr);
> +
> + return 0;
> +}
> +
> +/*
> + * mux the pin to the "B" internal peripheral role.
> + */
> +int at91_set_b_periph(unsigned pin, int use_pullup)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + writel(mask, &pio->port[port].idr);
> + if (use_pullup)
> + writel(mask, &pio->port[port].puer);
> + else
> + writel(mask, &pio->port[port].pudr);
> + writel(mask, &pio->port[port].bsr);
> + writel(mask, &pio->port[port].pdr);
> +
> + return 0;
> +}
> +
> +/*
> + * mux the pin to the gpio controller (instead of "A" or "B" peripheral), and
> + * configure it for an input.
> + */
> +int at91_set_gpio_input(u32 pin, int use_pullup)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + writel(mask, &pio->port[port].idr);
> + if (use_pullup)
> + writel(mask, &pio->port[port].puer);
> + else
> + writel(mask, &pio->port[port].pudr);
> + writel(mask, &pio->port[port].odr);
> + writel(mask, &pio->port[port].per);
> + return 0;
> +}
> +
> +/*
> + * mux the pin to the gpio controller (instead of "A" or "B" peripheral),
> + * and configure it for an output.
> + */
> +int at91_set_gpio_output(u32 pin, int value)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + writel(mask, &pio->port[port].idr);
> + writel(mask, &pio->port[port].pudr);
> + if (value)
> + writel(mask, &pio->port[port].sodr);
> + else
> + writel(mask, &pio->port[port].codr);
> + writel(mask, &pio->port[port].oer);
> + writel(mask, &pio->port[port].per);
> +
> + return 0;
> +}
> +
> +/*
> + * enable/disable the glitch filter; mostly used with IRQ handling.
> + */
grammer
';' -> .
Similar below
> +int at91_set_deglitch(unsigned pin, int is_on)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + if (is_on)
> + writel(mask, &pio->port[port].ifer);
> + else
> + writel(mask, &pio->port[port].ifdr);
> + return 0;
> +}
> +
> +/*
> + * enable/disable the multi-driver; This is only valid for output and
> + * allows the output pin to run as an open collector output.
> + */
> +int at91_set_multi_drive(unsigned pin, int is_on)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + if (is_on)
> + writel(mask, &pio->port[port].mder);
> + else
> + writel(mask, &pio->port[port].mddr);
> + return 0;
> +}
> +
> +/*
> +int gpio_direction_input(unsigned pin)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + if (!(readl(&pio->port[port].psr) & mask))
> + return -EINVAL;
> + writel(mask, &pio->port[port].odr);
> + return 0;
> +}
> +
> +int gpio_direction_output(unsigned pin, int value)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + if (!(readl(&pio->port[port].psr) & mask))
> + return -EINVAL;
> + if (value)
> + writel(mask, &pio->port[port].sodr);
> + else
> + writel(mask, &pio->port[port].codr);
> + writel(mask, &pio->port[port].oer);
> +
> + return 0;
> +}
> +*/
These are commented out!
These replaces an functions removed.
Please uncomment.
> +/*
> + * assuming the pin is muxed as a gpio output, set its value.
> + */
> +int at91_set_gpio_value(unsigned pin, int value)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> +
> + /* printf("%d = Port %d Pin %d (%d)",pin, port, mask, value); */
Remove printf or add as a debug
> + if (value)
> + writel(mask, &pio->port[port].sodr);
> + else
> + writel(mask, &pio->port[port].codr);
> + return 0;
> +}
> +
> +/*
> + * read the pin's value (works even if it's not muxed as a gpio).
> + */
> +int at91_get_gpio_value(unsigned pin)
> +{
> + at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
> + u32 port = portpin_to_port(pin);
> + u32 mask = portpin_to_pin(pin);
> + u32 pdsr;
> +
> + pdsr = readl(&pio->port[port]);
> + return (pdsr & mask) != 0;
> +}
> +
> diff --git a/include/asm-arm/arch-at91/gpio.h b/include/asm-arm/arch-at91/gpio.h
> index bc53171..b0c572f 100644
> --- a/include/asm-arm/arch-at91/gpio.h
> +++ b/include/asm-arm/arch-at91/gpio.h
> @@ -216,155 +216,10 @@ static inline unsigned pin_to_mask(unsigned pin)
> return 1 << (pin % 32);
> }
>
> -/*
> - * mux the pin to the "GPIO" peripheral role.
> - */
> -static inline int at91_set_GPIO_periph(unsigned pin, int use_pullup)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + PIO_IDR);
> - __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
> - __raw_writel(mask, pio + PIO_PER);
> - return 0;
> -}
> -
> -/*
> - * mux the pin to the "A" internal peripheral role.
> - */
> -static inline int at91_set_A_periph(unsigned pin, int use_pullup)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + PIO_IDR);
> - __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
> - __raw_writel(mask, pio + PIO_ASR);
> - __raw_writel(mask, pio + PIO_PDR);
> - return 0;
> -}
> -
> -/*
> - * mux the pin to the "B" internal peripheral role.
> - */
> -static inline int at91_set_B_periph(unsigned pin, int use_pullup)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + PIO_IDR);
> - __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
> - __raw_writel(mask, pio + PIO_BSR);
> - __raw_writel(mask, pio + PIO_PDR);
> - return 0;
> -}
> -
> -/*
> - * mux the pin to the gpio controller (instead of "A" or "B" peripheral), and
> - * configure it for an input.
> - */
> -static inline int at91_set_gpio_input(unsigned pin, int use_pullup)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + PIO_IDR);
> - __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
> - __raw_writel(mask, pio + PIO_ODR);
> - __raw_writel(mask, pio + PIO_PER);
> - return 0;
> -}
> -
> -/*
> - * mux the pin to the gpio controller (instead of "A" or "B" peripheral),
> - * and configure it for an output.
> - */
> -static inline int at91_set_gpio_output(unsigned pin, int value)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + PIO_IDR);
> - __raw_writel(mask, pio + PIO_PUDR);
> - __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
> - __raw_writel(mask, pio + PIO_OER);
> - __raw_writel(mask, pio + PIO_PER);
> - return 0;
> -}
> -
> -/*
> - * enable/disable the glitch filter; mostly used with IRQ handling.
> - */
> -static inline int at91_set_deglitch(unsigned pin, int is_on)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + (is_on ? PIO_IFER : PIO_IFDR));
> - return 0;
> -}
> -
> -/*
> - * enable/disable the multi-driver; This is only valid for output and
> - * allows the output pin to run as an open collector output.
> - */
> -static inline int at91_set_multi_drive(unsigned pin, int is_on)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + (is_on ? PIO_MDER : PIO_MDDR));
> - return 0;
> -}
> -
> -static inline int gpio_direction_input(unsigned pin)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - if (!(__raw_readl(pio + PIO_PSR) & mask))
> - return -EINVAL;
> - __raw_writel(mask, pio + PIO_ODR);
> - return 0;
> -}
> +#define at91_set_GPIO_periph at91_set_gpio_periph
> +#define at91_set_A_periph at91_set_a_periph
> +#define at91_set_B_periph at91_set_b_periph
>
> -static inline int gpio_direction_output(unsigned pin, int value)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - if (!(__raw_readl(pio + PIO_PSR) & mask))
> - return -EINVAL;
> - __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
> - __raw_writel(mask, pio + PIO_OER);
> - return 0;
> -}
> -
> -/*
> - * assuming the pin is muxed as a gpio output, set its value.
> - */
> -static inline int at91_set_gpio_value(unsigned pin, int value)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> -
> - __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
> - return 0;
> -}
> -
> -/*
> - * read the pin's value (works even if it's not muxed as a gpio).
> - */
> -static inline int at91_get_gpio_value(unsigned pin)
> -{
> - void *pio = pin_to_controller(pin);
> - unsigned mask = pin_to_mask(pin);
> - u32 pdsr;
> -
> - pdsr = __raw_readl(pio + PIO_PDSR);
> - return (pdsr & mask) != 0;
> -}
>
> #endif
> #endif
> diff --git a/include/configs/afeb9260.h b/include/configs/afeb9260.h
> index 3b69de8..9f8c567 100644
> --- a/include/configs/afeb9260.h
> +++ b/include/configs/afeb9260.h
> @@ -47,6 +47,7 @@
> /*
> * Hardware drivers
> */
> +#define CONFIG_AT91_GPIO 1
> #define CONFIG_ATMEL_USART 1
> #undef CONFIG_USART0
> #undef CONFIG_USART1
> diff --git a/include/configs/at91cap9adk.h b/include/configs/at91cap9adk.h
> index 4c2782a..9da5846 100644
> --- a/include/configs/at91cap9adk.h
> +++ b/include/configs/at91cap9adk.h
> @@ -49,6 +49,7 @@
> /*
> * Hardware drivers
> */
> +#define CONFIG_AT91_GPIO 1
> #define CONFIG_ATMEL_USART 1
> #undef CONFIG_USART0
> #undef CONFIG_USART1
> diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
> index a620d57..6d8969b 100644
> --- a/include/configs/at91sam9260ek.h
> +++ b/include/configs/at91sam9260ek.h
> @@ -54,6 +54,8 @@
> /*
> * Hardware drivers
> */
> +#define CONFIG_AT91_GPIO 1
> +
> #define CONFIG_ATMEL_USART 1
> #undef CONFIG_USART0
> #undef CONFIG_USART1
> diff --git a/include/configs/at91sam9261ek.h b/include/configs/at91sam9261ek.h
> index 832b1cd..e67d899 100644
> --- a/include/configs/at91sam9261ek.h
> +++ b/include/configs/at91sam9261ek.h
> @@ -52,6 +52,8 @@
> /*
> * Hardware drivers
> */
> +#define CONFIG_AT91_GPIO 1
> +
extra spaces are not needed
remove here and other similar below
Tom
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH 3/9 V4] add a new AT91 GPIO driver
2010-01-23 11:03 [U-Boot] [PATCH 3/9 V3] add a new AT91 GPIO driver Jens Scharsig
2010-01-31 1:08 ` Tom
@ 2010-02-03 21:46 ` Jens Scharsig
1 sibling, 0 replies; 3+ messages in thread
From: Jens Scharsig @ 2010-02-03 21:46 UTC (permalink / raw)
To: u-boot
* add a real AT91 GPIO driver instead of header inline code
* resolve the mixing of port and pins
* change board config files to use new driver
* add macros to gpio to realize backward compatibility
Signed-off-by: Jens Scharsig <js_at_ng@scharsoft.de>
---
drivers/gpio/Makefile | 1 +
drivers/gpio/at91_gpio.c | 214 ++++++++++++++++++++++++++++++++++
include/asm-arm/arch-at91/at91_pio.h | 14 ++-
include/asm-arm/arch-at91/gpio.h | 168 +++------------------------
include/configs/afeb9260.h | 1 +
include/configs/at91cap9adk.h | 1 +
include/configs/at91sam9260ek.h | 1 +
include/configs/at91sam9261ek.h | 1 +
include/configs/at91sam9263ek.h | 1 +
include/configs/at91sam9m10g45ek.h | 1 +
include/configs/at91sam9rlek.h | 1 +
include/configs/cpu9260.h | 1 +
include/configs/meesc.h | 1 +
include/configs/pm9261.h | 1 +
include/configs/pm9263.h | 1 +
include/configs/sbc35_a9g20.h | 1 +
include/configs/tny_a9260.h | 1 +
17 files changed, 258 insertions(+), 152 deletions(-)
create mode 100644 drivers/gpio/at91_gpio.c
diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile
index acba56c..d966082 100644
--- a/drivers/gpio/Makefile
+++ b/drivers/gpio/Makefile
@@ -25,6 +25,7 @@ include $(TOPDIR)/config.mk
LIB := $(obj)libgpio.a
+COBJS-$(CONFIG_AT91_GPIO) += at91_gpio.o
COBJS-$(CONFIG_KIRKWOOD_GPIO) += kw_gpio.o
COBJS-$(CONFIG_MX31_GPIO) += mx31_gpio.o
COBJS-$(CONFIG_PCA953X) += pca953x.o
diff --git a/drivers/gpio/at91_gpio.c b/drivers/gpio/at91_gpio.c
new file mode 100644
index 0000000..c0a97bc
--- /dev/null
+++ b/drivers/gpio/at91_gpio.c
@@ -0,0 +1,214 @@
+/*
+ * Memory Setup stuff - taken from blob memsetup.S
+ *
+ * Copyright (C) 2009 Jens Scharsig (js_at_ng at scharsoft.de)
+ *
+ * Copyright (C) 2005 HP Labs
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <common.h>
+#include <asm/sizes.h>
+#include <asm/arch/hardware.h>
+#include <asm/arch/io.h>
+#include <asm/arch/at91_pio.h>
+
+int at91_set_pio_pullup(unsigned port, unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ if (use_pullup)
+ writel(1 << pin, &pio->port[port].puer);
+ else
+ writel(1 << pin, &pio->port[port].pudr);
+ writel(mask, &pio->port[port].per);
+ }
+ return 0;
+}
+
+/*
+ * mux the pin to the "GPIO" peripheral role.
+ */
+int at91_set_pio_periph(unsigned port, unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ writel(mask, &pio->port[port].idr);
+ at91_set_pio_pullup(port, pin, use_pullup);
+ writel(mask, &pio->port[port].per);
+ }
+ return 0;
+}
+
+/*
+ * mux the pin to the "A" internal peripheral role.
+ */
+int at91_set_a_periph(unsigned port, unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ writel(mask, &pio->port[port].idr);
+ at91_set_pio_pullup(port, pin, use_pullup);
+ writel(mask, &pio->port[port].asr);
+ writel(mask, &pio->port[port].pdr);
+ }
+ return 0;
+}
+
+/*
+ * mux the pin to the "B" internal peripheral role.
+ */
+int at91_set_b_periph(unsigned port, unsigned pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ writel(mask, &pio->port[port].idr);
+ at91_set_pio_pullup(port, pin, use_pullup);
+ writel(mask, &pio->port[port].bsr);
+ writel(mask, &pio->port[port].pdr);
+ }
+ return 0;
+}
+
+/*
+ * mux the pin to the gpio controller (instead of "A" or "B" peripheral), and
+ * configure it for an input.
+ */
+int at91_set_pio_input(unsigned port, u32 pin, int use_pullup)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ writel(mask, &pio->port[port].idr);
+ at91_set_pio_pullup(port, pin, use_pullup);
+ writel(mask, &pio->port[port].odr);
+ writel(mask, &pio->port[port].per);
+ }
+ return 0;
+}
+
+/*
+ * mux the pin to the gpio controller (instead of "A" or "B" peripheral),
+ * and configure it for an output.
+ */
+int at91_set_pio_output(unsigned port, u32 pin, int value)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ writel(mask, &pio->port[port].idr);
+ writel(mask, &pio->port[port].pudr);
+ if (value)
+ writel(mask, &pio->port[port].sodr);
+ else
+ writel(mask, &pio->port[port].codr);
+ writel(mask, &pio->port[port].oer);
+ writel(mask, &pio->port[port].per);
+ }
+ return 0;
+}
+
+/*
+ * enable/disable the glitch filter. mostly used with IRQ handling.
+ */
+int at91_set_pio_deglitch(unsigned port, unsigned pin, int is_on)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ if (is_on)
+ writel(mask, &pio->port[port].ifer);
+ else
+ writel(mask, &pio->port[port].ifdr);
+ }
+ return 0;
+}
+
+/*
+ * enable/disable the multi-driver. This is only valid for output and
+ * allows the output pin to run as an open collector output.
+ */
+int at91_set_pio_multi_drive(unsigned port, unsigned pin, int is_on)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ if (is_on)
+ writel(mask, &pio->port[port].mder);
+ else
+ writel(mask, &pio->port[port].mddr);
+ }
+ return 0;
+}
+
+/*
+ * assuming the pin is muxed as a gpio output, set its value.
+ */
+int at91_set_pio_value(unsigned port, unsigned pin, int value)
+{
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ if (value)
+ writel(mask, &pio->port[port].sodr);
+ else
+ writel(mask, &pio->port[port].codr);
+ }
+ return 0;
+}
+
+/*
+ * read the pin's value (works even if it's not muxed as a gpio).
+ */
+int at91_get_pio_value(unsigned port, unsigned pin)
+{
+ u32 pdsr = 0;
+ at91_pio_t *pio = (at91_pio_t *) AT91_PIO_BASE;
+ u32 mask;
+
+ if ((port < AT91_PIO_PORTS) && (pin < 32)) {
+ mask = 1 << pin;
+ pdsr = readl(&pio->port[port].pdsr) & mask;
+ }
+ return pdsr != 0;
+}
diff --git a/include/asm-arm/arch-at91/at91_pio.h b/include/asm-arm/arch-at91/at91_pio.h
index 92c3717..e5946ca 100644
--- a/include/asm-arm/arch-at91/at91_pio.h
+++ b/include/asm-arm/arch-at91/at91_pio.h
@@ -100,10 +100,20 @@ typedef union at91_pio {
at91_port_t port[AT91_PIO_PORTS];
} at91_pio_t;
+#ifdef CONFIG_AT91_GPIO
+int at91_set_a_periph(unsigned port, unsigned pin, int use_pullup);
+int at91_set_b_periph(unsigned port, unsigned pin, int use_pullup);
+int at91_set_pio_input(unsigned port, unsigned pin, int use_pullup);
+int at91_set_pio_multi_drive(unsigned port, unsigned pin, int is_on);
+int at91_set_pio_output(unsigned port, unsigned pin, int value);
+int at91_set_pio_periph(unsigned port, unsigned pin, int use_pullup);
+int at91_set_pio_pullup(unsigned port, unsigned pin, int use_pullup);
+int at91_set_pio_deglitch(unsigned port, unsigned pin, int is_on);
+int at91_set_pio_value(unsigned port, unsigned pin, int value);
+int at91_get_pio_value(unsigned port, unsigned pin);
+#endif
#endif
-#define AT91_PIN_TO_MASK(x) (1<<x)
-#define AT91_PORTPIN(PORT, PIN) ((0x0##PORT - 9) * 32 + ((PIN) & 0x1F))
#define AT91_PIO_PORTA 0x0
#define AT91_PIO_PORTB 0x1
#define AT91_PIO_PORTC 0x2
diff --git a/include/asm-arm/arch-at91/gpio.h b/include/asm-arm/arch-at91/gpio.h
index bc53171..716f81f 100644
--- a/include/asm-arm/arch-at91/gpio.h
+++ b/include/asm-arm/arch-at91/gpio.h
@@ -216,155 +216,23 @@ static inline unsigned pin_to_mask(unsigned pin)
return 1 << (pin % 32);
}
-/*
- * mux the pin to the "GPIO" peripheral role.
- */
-static inline int at91_set_GPIO_periph(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_PER);
- return 0;
-}
-
-/*
- * mux the pin to the "A" internal peripheral role.
- */
-static inline int at91_set_A_periph(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_ASR);
- __raw_writel(mask, pio + PIO_PDR);
- return 0;
-}
-
-/*
- * mux the pin to the "B" internal peripheral role.
- */
-static inline int at91_set_B_periph(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_BSR);
- __raw_writel(mask, pio + PIO_PDR);
- return 0;
-}
-
-/*
- * mux the pin to the gpio controller (instead of "A" or "B" peripheral), and
- * configure it for an input.
- */
-static inline int at91_set_gpio_input(unsigned pin, int use_pullup)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + (use_pullup ? PIO_PUER : PIO_PUDR));
- __raw_writel(mask, pio + PIO_ODR);
- __raw_writel(mask, pio + PIO_PER);
- return 0;
-}
-
-/*
- * mux the pin to the gpio controller (instead of "A" or "B" peripheral),
- * and configure it for an output.
- */
-static inline int at91_set_gpio_output(unsigned pin, int value)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + PIO_IDR);
- __raw_writel(mask, pio + PIO_PUDR);
- __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
- __raw_writel(mask, pio + PIO_OER);
- __raw_writel(mask, pio + PIO_PER);
- return 0;
-}
-
-/*
- * enable/disable the glitch filter; mostly used with IRQ handling.
- */
-static inline int at91_set_deglitch(unsigned pin, int is_on)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + (is_on ? PIO_IFER : PIO_IFDR));
- return 0;
-}
-
-/*
- * enable/disable the multi-driver; This is only valid for output and
- * allows the output pin to run as an open collector output.
- */
-static inline int at91_set_multi_drive(unsigned pin, int is_on)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + (is_on ? PIO_MDER : PIO_MDDR));
- return 0;
-}
-
-static inline int gpio_direction_input(unsigned pin)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- if (!(__raw_readl(pio + PIO_PSR) & mask))
- return -EINVAL;
- __raw_writel(mask, pio + PIO_ODR);
- return 0;
-}
-
-static inline int gpio_direction_output(unsigned pin, int value)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- if (!(__raw_readl(pio + PIO_PSR) & mask))
- return -EINVAL;
- __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
- __raw_writel(mask, pio + PIO_OER);
- return 0;
-}
-
-/*
- * assuming the pin is muxed as a gpio output, set its value.
- */
-static inline int at91_set_gpio_value(unsigned pin, int value)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
-
- __raw_writel(mask, pio + (value ? PIO_SODR : PIO_CODR));
- return 0;
-}
-
-/*
- * read the pin's value (works even if it's not muxed as a gpio).
- */
-static inline int at91_get_gpio_value(unsigned pin)
-{
- void *pio = pin_to_controller(pin);
- unsigned mask = pin_to_mask(pin);
- u32 pdsr;
-
- pdsr = __raw_readl(pio + PIO_PDSR);
- return (pdsr & mask) != 0;
-}
-
+/* The following macros are need for backward compatibility */
+#define at91_set_GPIO_periph(x, y) \
+ at91_set_gpio_periph((x - PIN_BASE) / 32,(x % 32), y)
+#define at91_set_A_periph(x, y) \
+ at91_set_a_periph((x - PIN_BASE) / 32,(x % 32), y)
+#define at91_set_B_periph(x, y) \
+ at91_set_b_periph((x - PIN_BASE) / 32,(x % 32), y)
+#define at91_set_gpio_output(x, y) \
+ at91_set_pio_output((x - PIN_BASE) / 32,(x % 32), y)
+#define at91_set_gpio_input(x, y) \
+ at91_set_pio_input((x - PIN_BASE) / 32,(x % 32), y)
+#define at91_set_gpio_value(x, y) \
+ at91_set_pio_value((x - PIN_BASE) / 32,(x % 32), y)
+#define at91_get_gpio_value(x) \
+ at91_get_pio_value((x - PIN_BASE) / 32,(x % 32))
+#else
+#define at91_set_gpio_value(x, y) at91_set_pio_value(x, y)
+#define at91_get_gpio_value(x) at91_get_pio_value(x)
#endif
#endif
diff --git a/include/configs/afeb9260.h b/include/configs/afeb9260.h
index 3b69de8..9f8c567 100644
--- a/include/configs/afeb9260.h
+++ b/include/configs/afeb9260.h
@@ -47,6 +47,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91cap9adk.h b/include/configs/at91cap9adk.h
index 4c2782a..9da5846 100644
--- a/include/configs/at91cap9adk.h
+++ b/include/configs/at91cap9adk.h
@@ -49,6 +49,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
index a620d57..ff1e5b3 100644
--- a/include/configs/at91sam9260ek.h
+++ b/include/configs/at91sam9260ek.h
@@ -54,6 +54,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9261ek.h b/include/configs/at91sam9261ek.h
index 832b1cd..63b55d2 100644
--- a/include/configs/at91sam9261ek.h
+++ b/include/configs/at91sam9261ek.h
@@ -52,6 +52,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9263ek.h b/include/configs/at91sam9263ek.h
index 1d82a15..e39762b 100644
--- a/include/configs/at91sam9263ek.h
+++ b/include/configs/at91sam9263ek.h
@@ -51,6 +51,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9m10g45ek.h b/include/configs/at91sam9m10g45ek.h
index 50b118f..ce21831 100644
--- a/include/configs/at91sam9m10g45ek.h
+++ b/include/configs/at91sam9m10g45ek.h
@@ -52,6 +52,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/at91sam9rlek.h b/include/configs/at91sam9rlek.h
index 8db296a..95f3d6c 100644
--- a/include/configs/at91sam9rlek.h
+++ b/include/configs/at91sam9rlek.h
@@ -49,6 +49,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/cpu9260.h b/include/configs/cpu9260.h
index e967e7c..38f34c2 100644
--- a/include/configs/cpu9260.h
+++ b/include/configs/cpu9260.h
@@ -244,6 +244,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/meesc.h b/include/configs/meesc.h
index 253a53d..d469baa 100644
--- a/include/configs/meesc.h
+++ b/include/configs/meesc.h
@@ -58,6 +58,7 @@
*/
/* Console output */
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/pm9261.h b/include/configs/pm9261.h
index 26a2fad..8ee132b 100644
--- a/include/configs/pm9261.h
+++ b/include/configs/pm9261.h
@@ -159,6 +159,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/pm9263.h b/include/configs/pm9263.h
index e55098c..a6fdcaa 100644
--- a/include/configs/pm9263.h
+++ b/include/configs/pm9263.h
@@ -173,6 +173,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/sbc35_a9g20.h b/include/configs/sbc35_a9g20.h
index 7bdc729..1f2ff9e 100644
--- a/include/configs/sbc35_a9g20.h
+++ b/include/configs/sbc35_a9g20.h
@@ -59,6 +59,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART
#define CONFIG_USART0
#undef CONFIG_USART1
diff --git a/include/configs/tny_a9260.h b/include/configs/tny_a9260.h
index 21475f8..a159a40 100644
--- a/include/configs/tny_a9260.h
+++ b/include/configs/tny_a9260.h
@@ -68,6 +68,7 @@
/*
* Hardware drivers
*/
+#define CONFIG_AT91_GPIO 1
#define CONFIG_ATMEL_USART 1
#undef CONFIG_USART0
#undef CONFIG_USART1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-02-03 21:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-23 11:03 [U-Boot] [PATCH 3/9 V3] add a new AT91 GPIO driver Jens Scharsig
2010-01-31 1:08 ` Tom
2010-02-03 21:46 ` [U-Boot] [PATCH 3/9 V4] " Jens Scharsig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox