diff mbox

[U-Boot,06/15] AT91: fix related arch-at91 header files

Message ID 1298033414-11481-7-git-send-email-u-boot@emk-elektronik.de
State Accepted
Commit 83f1072e06d4e3b9eaceed39799042c7b3bbdbb1
Delegated to: Reinhard Meyer
Headers show

Commit Message

Reinhard Meyer Feb. 18, 2011, 12:50 p.m. UTC
Signed-off-by: Reinhard Meyer <u-boot@emk-elektronik.de>
---
 arch/arm/include/asm/arch-at91/at91_common.h |    3 +--
 arch/arm/include/asm/arch-at91/at91_pio.h    |   19 +++----------------
 arch/arm/include/asm/arch-at91/gpio.h        |   18 +++++++++---------
 arch/arm/include/asm/arch-at91/io.h          |    4 ++--
 4 files changed, 15 insertions(+), 29 deletions(-)

Comments

Andreas Bießmann Feb. 18, 2011, 12:27 p.m. UTC | #1
Am 18.02.2011 13:50, schrieb Reinhard Meyer:
> Signed-off-by: Reinhard Meyer <u-boot@emk-elektronik.de>
> ---
>  arch/arm/include/asm/arch-at91/at91_common.h |    3 +--
>  arch/arm/include/asm/arch-at91/at91_pio.h    |   19 +++----------------
>  arch/arm/include/asm/arch-at91/gpio.h        |   18 +++++++++---------
>  arch/arm/include/asm/arch-at91/io.h          |    4 ++--
>  4 files changed, 15 insertions(+), 29 deletions(-)
> 
> diff --git a/arch/arm/include/asm/arch-at91/at91_common.h b/arch/arm/include/asm/arch-at91/at91_common.h
> index 0067190..330edd8 100644
> --- a/arch/arm/include/asm/arch-at91/at91_common.h
> +++ b/arch/arm/include/asm/arch-at91/at91_common.h
> @@ -28,11 +28,10 @@
>  void at91_can_hw_init(void);
>  void at91_macb_hw_init(void);
>  void at91_mci_hw_init(void);
> -void at91_serial_hw_init(void);
>  void at91_serial0_hw_init(void);
>  void at91_serial1_hw_init(void);
>  void at91_serial2_hw_init(void);
> -void at91_serial3_hw_init(void);
> +void at91_seriald_hw_init(void);
>  void at91_spi0_hw_init(unsigned long cs_mask);
>  void at91_spi1_hw_init(unsigned long cs_mask);
>  void at91_uhp_hw_init(void);
> diff --git a/arch/arm/include/asm/arch-at91/at91_pio.h b/arch/arm/include/asm/arch-at91/at91_pio.h
> index f7915a3..0b5bd69 100644
> --- a/arch/arm/include/asm/arch-at91/at91_pio.h
> +++ b/arch/arm/include/asm/arch-at91/at91_pio.h
> @@ -76,32 +76,19 @@ typedef struct at91_port {
>  	u32	reserved6[85];
>  } at91_port_t;
>  
> -#if defined(CONFIG_AT91SAM9260) || defined(CONFIG_AT91SAM9261) || \
> -	defined(CONFIG_AT91SAM9G10) || defined(CONFIG_AT91SAM9G20)
> -#define AT91_PIO_PORTS	3
> -#elif defined(CONFIG_AT91SAM9263) || defined(CONFIG_AT91SAM9G45) || \
> -	defined(CONFIG_AT91SAM9M10G45)
> -#define AT91_PIO_PORTS	5
> -#elif defined(CONFIG_AT91RM9200) || defined(CONFIG_AT91CAP9) || \
> -	defined(CONFIG_AT91SAM9RL)
> -#define AT91_PIO_PORTS	4
> -#else
> -#error "Unsupported cpu. Please update at91_pio.h"
> -#endif
> -
>  typedef union at91_pio {
>  	struct {
>  		at91_port_t	pioa;
>  		at91_port_t	piob;
>  		at91_port_t	pioc;
> -	#if (AT91_PIO_PORTS > 3)
> +	#if (ATMEL_PIO_PORTS > 3)
>  		at91_port_t	piod;
>  	#endif
> -	#if (AT91_PIO_PORTS > 4)
> +	#if (ATMEL_PIO_PORTS > 4)
>  		at91_port_t	pioe;
>  	#endif
>  	} ;
> -	at91_port_t port[AT91_PIO_PORTS];
> +	at91_port_t port[ATMEL_PIO_PORTS];
>  } at91_pio_t;
>  
>  #ifdef CONFIG_AT91_GPIO
> diff --git a/arch/arm/include/asm/arch-at91/gpio.h b/arch/arm/include/asm/arch-at91/gpio.h
> index 716f81f..293d0bf 100644
> --- a/arch/arm/include/asm/arch-at91/gpio.h
> +++ b/arch/arm/include/asm/arch-at91/gpio.h
> @@ -18,7 +18,7 @@
>  #include <asm/arch/at91_pio.h>
>  #include <asm/arch/hardware.h>
>  
> -#ifdef CONFIG_AT91_LEGACY
> +#ifdef CONFIG_ATMEL_LEGACY

is this still necessary?

>  
>  #define PIN_BASE		32
>  
> @@ -192,13 +192,13 @@
>  #define	AT91_PIN_PE31	(PIN_BASE + 0x80 + 31)
>  
>  static unsigned long at91_pios[] = {
> -	AT91_PIOA,
> -	AT91_PIOB,
> -	AT91_PIOC,
> -#ifdef AT91_PIOD
> -	AT91_PIOD,
> -#ifdef AT91_PIOE
> -	AT91_PIOE
> +	ATMEL_BASE_PIOA,
> +	ATMEL_BASE_PIOB,
> +	ATMEL_BASE_PIOC,
> +#ifdef ATMEL_BASE_PIOD
> +	ATMEL_BASE_PIOD,
> +#ifdef ATMEL_BASE_PIOE
> +	ATMEL_BASE_PIOE
>  #endif
>  #endif
>  };
> @@ -207,7 +207,7 @@ static inline void *pin_to_controller(unsigned pin)
>  {
>  	pin -= PIN_BASE;
>  	pin /= 32;
> -	return (void *)(AT91_BASE_SYS + at91_pios[pin]);
> +	return (void *)(at91_pios[pin]);
>  }
>  
>  static inline unsigned pin_to_mask(unsigned pin)
> diff --git a/arch/arm/include/asm/arch-at91/io.h b/arch/arm/include/asm/arch-at91/io.h
> index 38d185e..d0daa4e 100644
> --- a/arch/arm/include/asm/arch-at91/io.h
> +++ b/arch/arm/include/asm/arch-at91/io.h
> @@ -27,14 +27,14 @@
>  
>  static inline unsigned int at91_sys_read(unsigned int reg_offset)
>  {
> -	void *addr = (void *)AT91_BASE_SYS;
> +	void *addr = (void *)ATMEL_BASE_SYS;
>  
>  	return __raw_readl(addr + reg_offset);
>  }
>  
>  static inline void at91_sys_write(unsigned int reg_offset, unsigned long value)
>  {
> -	void *addr = (void *)AT91_BASE_SYS;
> +	void *addr = (void *)ATMEL_BASE_SYS;
>  
>  	__raw_writel(value, addr + reg_offset);
>  }
Reinhard Meyer Feb. 18, 2011, 12:52 p.m. UTC | #2
Dear Andreas Bießmann,
>> +#ifdef CONFIG_ATMEL_LEGACY
>
> is this still necessary?
I am afraid so. The ATMEL GPIO code should be reworked.
Reinhard
diff mbox

Patch

diff --git a/arch/arm/include/asm/arch-at91/at91_common.h b/arch/arm/include/asm/arch-at91/at91_common.h
index 0067190..330edd8 100644
--- a/arch/arm/include/asm/arch-at91/at91_common.h
+++ b/arch/arm/include/asm/arch-at91/at91_common.h
@@ -28,11 +28,10 @@ 
 void at91_can_hw_init(void);
 void at91_macb_hw_init(void);
 void at91_mci_hw_init(void);
-void at91_serial_hw_init(void);
 void at91_serial0_hw_init(void);
 void at91_serial1_hw_init(void);
 void at91_serial2_hw_init(void);
-void at91_serial3_hw_init(void);
+void at91_seriald_hw_init(void);
 void at91_spi0_hw_init(unsigned long cs_mask);
 void at91_spi1_hw_init(unsigned long cs_mask);
 void at91_uhp_hw_init(void);
diff --git a/arch/arm/include/asm/arch-at91/at91_pio.h b/arch/arm/include/asm/arch-at91/at91_pio.h
index f7915a3..0b5bd69 100644
--- a/arch/arm/include/asm/arch-at91/at91_pio.h
+++ b/arch/arm/include/asm/arch-at91/at91_pio.h
@@ -76,32 +76,19 @@  typedef struct at91_port {
 	u32	reserved6[85];
 } at91_port_t;
 
-#if defined(CONFIG_AT91SAM9260) || defined(CONFIG_AT91SAM9261) || \
-	defined(CONFIG_AT91SAM9G10) || defined(CONFIG_AT91SAM9G20)
-#define AT91_PIO_PORTS	3
-#elif defined(CONFIG_AT91SAM9263) || defined(CONFIG_AT91SAM9G45) || \
-	defined(CONFIG_AT91SAM9M10G45)
-#define AT91_PIO_PORTS	5
-#elif defined(CONFIG_AT91RM9200) || defined(CONFIG_AT91CAP9) || \
-	defined(CONFIG_AT91SAM9RL)
-#define AT91_PIO_PORTS	4
-#else
-#error "Unsupported cpu. Please update at91_pio.h"
-#endif
-
 typedef union at91_pio {
 	struct {
 		at91_port_t	pioa;
 		at91_port_t	piob;
 		at91_port_t	pioc;
-	#if (AT91_PIO_PORTS > 3)
+	#if (ATMEL_PIO_PORTS > 3)
 		at91_port_t	piod;
 	#endif
-	#if (AT91_PIO_PORTS > 4)
+	#if (ATMEL_PIO_PORTS > 4)
 		at91_port_t	pioe;
 	#endif
 	} ;
-	at91_port_t port[AT91_PIO_PORTS];
+	at91_port_t port[ATMEL_PIO_PORTS];
 } at91_pio_t;
 
 #ifdef CONFIG_AT91_GPIO
diff --git a/arch/arm/include/asm/arch-at91/gpio.h b/arch/arm/include/asm/arch-at91/gpio.h
index 716f81f..293d0bf 100644
--- a/arch/arm/include/asm/arch-at91/gpio.h
+++ b/arch/arm/include/asm/arch-at91/gpio.h
@@ -18,7 +18,7 @@ 
 #include <asm/arch/at91_pio.h>
 #include <asm/arch/hardware.h>
 
-#ifdef CONFIG_AT91_LEGACY
+#ifdef CONFIG_ATMEL_LEGACY
 
 #define PIN_BASE		32
 
@@ -192,13 +192,13 @@ 
 #define	AT91_PIN_PE31	(PIN_BASE + 0x80 + 31)
 
 static unsigned long at91_pios[] = {
-	AT91_PIOA,
-	AT91_PIOB,
-	AT91_PIOC,
-#ifdef AT91_PIOD
-	AT91_PIOD,
-#ifdef AT91_PIOE
-	AT91_PIOE
+	ATMEL_BASE_PIOA,
+	ATMEL_BASE_PIOB,
+	ATMEL_BASE_PIOC,
+#ifdef ATMEL_BASE_PIOD
+	ATMEL_BASE_PIOD,
+#ifdef ATMEL_BASE_PIOE
+	ATMEL_BASE_PIOE
 #endif
 #endif
 };
@@ -207,7 +207,7 @@  static inline void *pin_to_controller(unsigned pin)
 {
 	pin -= PIN_BASE;
 	pin /= 32;
-	return (void *)(AT91_BASE_SYS + at91_pios[pin]);
+	return (void *)(at91_pios[pin]);
 }
 
 static inline unsigned pin_to_mask(unsigned pin)
diff --git a/arch/arm/include/asm/arch-at91/io.h b/arch/arm/include/asm/arch-at91/io.h
index 38d185e..d0daa4e 100644
--- a/arch/arm/include/asm/arch-at91/io.h
+++ b/arch/arm/include/asm/arch-at91/io.h
@@ -27,14 +27,14 @@ 
 
 static inline unsigned int at91_sys_read(unsigned int reg_offset)
 {
-	void *addr = (void *)AT91_BASE_SYS;
+	void *addr = (void *)ATMEL_BASE_SYS;
 
 	return __raw_readl(addr + reg_offset);
 }
 
 static inline void at91_sys_write(unsigned int reg_offset, unsigned long value)
 {
-	void *addr = (void *)AT91_BASE_SYS;
+	void *addr = (void *)ATMEL_BASE_SYS;
 
 	__raw_writel(value, addr + reg_offset);
 }