Skip to contentSkip to conversation

MIPS: BCM47xx: fix build with GENERIC_GPIO configuration

PATCH In review Florian Fainelli 2 participants 2 messages Nov 4, 2011

Subsystems (2)
MIPS Broadcom BCM47XX MIPS architecture
lore
Share & tools
lore

Florian Fainelli

author
#0 lore raw

PATCH v2 1/1 MIPS: BCM47xx: fix build with GENERIC_GPIO configuration

Since eb9ae7f2 (gpio: fix build error in include/asm-generic/gpio.h) the generic version of gpio.h calls __gpio_{set,get}_value which we do not define. Get rid of asm-generic/gpio.h and define the missing stubs directly for BCM47xx to build.

Reported-by Ralf Baechle <[email protected]>
CC Hauke Mehrtens <[email protected]>
Signed-off-by Florian Fainelli <[email protected]>

---
Changes since v2:
- fixed the subject (BCM47xxx -> BCM47xx)
- fixed gpio_is_valid to check for the proper gpio number

arch/mips/include/asm/mach-bcm47xx/gpio.h +15 -3
index 76961ca..2ef17e8 100644
@@ -36,6 +36,8 @@ static inline int gpio_get_value(unsigned gpio)
 	return -EINVAL;
 }
 
+#define gpio_get_value_cansleep	gpio_get_value
+
 static inline void gpio_set_value(unsigned gpio, int value)
 {
 	switch (bcm47xx_bus_type) {

@@ -54,6 +56,19 @@ static inline void gpio_set_value(unsigned gpio, int value)
 	}
 }
 
+#define gpio_set_value_cansleep gpio_set_value
+
+static inline int gpio_cansleep(unsigned gpio)
+{
+	return 0;
+}
+
+static inline int gpio_is_valid(unsigned gpio)
+{
+	return gpio < (BCM47XX_EXTIF_GPIO_LINES + BCM47XX_CHIPCO_GPIO_LINES);
+}
+
+
 static inline int gpio_direction_input(unsigned gpio)
 {
 	switch (bcm47xx_bus_type) {

@@ -137,7 +152,4 @@ static inline int gpio_polarity(unsigned gpio, int value)
 }
 
 
-/* cansleep wrappers */
-#include <asm-generic/gpio.h>
-
 #endif /* __BCM47XX_GPIO_H */
Signature · 1 line
1.7.5.4
#1 lore raw

On Fri, Nov 04, 2011 at 11:17:46AM +0100, Florian Fainelli wrote:

I've reverted the old one and applied this new one.

Thanks everybody!

Ralf