Fix bcm47xx GPIO dirin/dirout API return codes.
authorMichael Büsch <mb@bu3sch.de>
Sat, 14 Feb 2009 20:22:31 +0000 (20:22 +0000)
committerMichael Büsch <mb@bu3sch.de>
Sat, 14 Feb 2009 20:22:31 +0000 (20:22 +0000)
SVN-Revision: 14509

target/linux/brcm47xx/patches-2.6.28/811-bcm47xx-fix-gpio-direction-retval.patch [new file with mode: 0644]

diff --git a/target/linux/brcm47xx/patches-2.6.28/811-bcm47xx-fix-gpio-direction-retval.patch b/target/linux/brcm47xx/patches-2.6.28/811-bcm47xx-fix-gpio-direction-retval.patch
new file mode 100644 (file)
index 0000000..5c4ecfe
--- /dev/null
@@ -0,0 +1,49 @@
+The GPIO API is supposed to return 0 or a negative error code,
+but the SSB GPIO functions return the bitmask of the GPIO register.
+Fix this by ignoring the bitmask and always returning 0. The SSB GPIO functions can't fail.
+
+--mb
+
+
+
+Index: linux-2.6.28.5/arch/mips/include/asm/mach-bcm47xx/gpio.h
+===================================================================
+--- linux-2.6.28.5.orig/arch/mips/include/asm/mach-bcm47xx/gpio.h      2009-02-14 20:52:31.000000000 +0100
++++ linux-2.6.28.5/arch/mips/include/asm/mach-bcm47xx/gpio.h   2009-02-14 21:01:20.000000000 +0100
+@@ -31,24 +31,28 @@ static inline void gpio_set_value(unsign
+ static inline int gpio_direction_input(unsigned gpio)
+ {
+-      return ssb_gpio_outen(&ssb_bcm47xx, 1 << gpio, 0);
++      ssb_gpio_outen(&ssb_bcm47xx, 1 << gpio, 0);
++      return 0;
+ }
+ static inline int gpio_direction_output(unsigned gpio, int value)
+ {
+-      return ssb_gpio_outen(&ssb_bcm47xx, 1 << gpio, 1 << gpio);
++      ssb_gpio_outen(&ssb_bcm47xx, 1 << gpio, 1 << gpio);
++      return 0;
+ }
+-static int gpio_intmask(unsigned gpio, int value)
++static inline int gpio_intmask(unsigned gpio, int value)
+ {
+-      return ssb_gpio_intmask(&ssb_bcm47xx, 1 << gpio,
+-                              value ? 1 << gpio : 0);
++      ssb_gpio_intmask(&ssb_bcm47xx, 1 << gpio,
++                       value ? 1 << gpio : 0);
++      return 0;
+ }
+-static int gpio_polarity(unsigned gpio, int value)
++static inline int gpio_polarity(unsigned gpio, int value)
+ {
+-      return ssb_gpio_polarity(&ssb_bcm47xx, 1 << gpio,
+-                               value ? 1 << gpio : 0);
++      ssb_gpio_polarity(&ssb_bcm47xx, 1 << gpio,
++                        value ? 1 << gpio : 0);
++      return 0;
+ }