Skip to content

Commit

Permalink
[ARM] pxa: Avoid pxa_gpio_mode() in gpio_direction_{in,out}put()
Browse files Browse the repository at this point in the history
pxa_gpio_mode() is a universal call that fiddles with the GAFR
(gpio alternate function register.)  GAFR does not exist on PXA3
CPUs, but instead the alternate functions are controlled via the
MFP support code.

Platforms are expected to configure the MFP according to their
needs in their platform support code rather than drivers.  We
extend this idea to the GAFR, and make the gpio_direction_*()
functions purely operate on the GPIO level.

This means platform support code is entirely responsible for
configuring the GPIOs alternate functions on all PXA CPU types.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
  • Loading branch information
Russell King authored and Russell King committed Oct 15, 2007
1 parent 39cbd48 commit 3e0cc7e
Show file tree
Hide file tree
Showing 2 changed files with 40 additions and 10 deletions.
38 changes: 38 additions & 0 deletions arch/arm/mach-pxa/generic.c
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,44 @@ int pxa_gpio_mode(int gpio_mode)

EXPORT_SYMBOL(pxa_gpio_mode);

int gpio_direction_input(unsigned gpio)
{
unsigned long flags;
u32 mask;

if (gpio > pxa_last_gpio)
return -EINVAL;

mask = GPIO_bit(gpio);
local_irq_save(flags);
GPDR(gpio) &= ~mask;
local_irq_restore(flags);

return 0;
}
EXPORT_SYMBOL(gpio_direction_input);

int gpio_direction_output(unsigned gpio, int value)
{
unsigned long flags;
u32 mask;

if (gpio > pxa_last_gpio)
return -EINVAL;

mask = GPIO_bit(gpio);
local_irq_save(flags);
if (value)
GPSR(gpio) = mask;
else
GPCR(gpio) = mask;
GPDR(gpio) |= mask;
local_irq_restore(flags);

return 0;
}
EXPORT_SYMBOL(gpio_direction_output);

/*
* Return GPIO level
*/
Expand Down
12 changes: 2 additions & 10 deletions include/asm-arm/arch-pxa/gpio.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,16 +38,8 @@ static inline void gpio_free(unsigned gpio)
return;
}

static inline int gpio_direction_input(unsigned gpio)
{
return pxa_gpio_mode(gpio | GPIO_IN);
}

static inline int gpio_direction_output(unsigned gpio, int value)
{
return pxa_gpio_mode(gpio | GPIO_OUT |
(value ? GPIO_DFLT_HIGH : GPIO_DFLT_LOW));
}
extern int gpio_direction_input(unsigned gpio);
extern int gpio_direction_output(unsigned gpio, int value);

static inline int __gpio_get_value(unsigned gpio)
{
Expand Down

0 comments on commit 3e0cc7e

Please sign in to comment.