Skip to content

Commit

Permalink
bcm47xx: fix GPIO API return codes
Browse files Browse the repository at this point in the history
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.

Signed-off-by: Michael Buesch <mb@bu3sch.de>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: David Brownell <david-b@pacbell.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
  • Loading branch information
Michael Buesch authored and Linus Torvalds committed Apr 1, 2009
1 parent c0aa24b commit e0f7ad5
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions arch/mips/include/asm/mach-bcm47xx/gpio.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,24 +31,28 @@ static inline void gpio_set_value(unsigned gpio, int value)

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;
}


Expand Down

0 comments on commit e0f7ad5

Please sign in to comment.