From: Gabor Juhos <juhosg@openwrt.org>
Date: Sun, 13 Apr 2008 10:47:13 +0000 (+0000)
Subject: fix GPIO direction setup on ar5312, and fix compiler warnings
X-Git-Tag: 8.09_rc1~1360
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=bb0c09267263ac5ef59a5f2aa1feaff816d6d6ac;p=openwrt%2Fsvn-archive%2Fopenwrt.git

fix GPIO direction setup on ar5312, and fix compiler warnings

SVN-Revision: 10811
---

diff --git a/target/linux/atheros/files/include/asm-mips/mach-atheros/gpio.h b/target/linux/atheros/files/include/asm-mips/mach-atheros/gpio.h
index 560b8cc0dc..db7347c221 100644
--- a/target/linux/atheros/files/include/asm-mips/mach-atheros/gpio.h
+++ b/target/linux/atheros/files/include/asm-mips/mach-atheros/gpio.h
@@ -26,9 +26,8 @@ static inline int gpio_direction_input(unsigned gpio) {
 				return -ENXIO;				\
 			} else {					\
 				sysRegWrite(AR531X_GPIO_CR,		\
-					( sysRegRead(AR531X_GPIO_CR) &	\
-					  ~(AR531X_GPIO_CR_M(gpio)) ) |	\
-					  AR531X_GPIO_CR_I(gpio) );	\
+					sysRegRead(AR531X_GPIO_CR) |	\
+					AR531X_GPIO_CR_I(gpio) );	\
 				return 0;				\
 			}						\
 	)
@@ -42,6 +41,7 @@ static inline int gpio_direction_input(unsigned gpio) {
 				return 0;				\
 			}						\
 	)
+	return -ENXIO;
 }
 
 /* Sets a gpio to output with value, or returns ENXIO for non-existent gpio */
@@ -54,8 +54,8 @@ static inline int gpio_direction_output(unsigned gpio, int value) {
 					  ~(1 << gpio) ) |		\
 					  ((value!=0) << gpio)) );	\
 				sysRegWrite(AR531X_GPIO_CR,		\
-					sysRegRead(AR531X_GPIO_CR) |	\
-					AR531X_GPIO_CR_O(gpio) );	\
+					( sysRegRead(AR531X_GPIO_CR) &	\
+					  ~(AR531X_GPIO_CR_M(gpio)) )); \
 				return 0;				\
 			}						\
 	)
@@ -72,12 +72,14 @@ static inline int gpio_direction_output(unsigned gpio, int value) {
 				return 0;				\
 			}						\
 	)
+	return -ENXIO;
 }
 
 /* Reads the gpio pin.  Unchecked function */
 static inline int gpio_get_value(unsigned gpio) {
 	DO_AR5312(return (sysRegRead(AR531X_GPIO_DI) & (1 << gpio));)
 	DO_AR5315(return (sysRegRead(AR5315_GPIO_DI) & (1 << gpio));)
+	return 0;
 }
 
 /* Writes to the gpio pin.  Unchecked function */