Commit d6a2fa04 authored by Alexander Shiyan's avatar Alexander Shiyan Committed by Linus Walleij

GPIO: clps711x: Fix direction logic for PORTD

PORTD have different direction logic, i.e. "0" is output and "1" is input.
This patch fix this issue.
Signed-off-by: default avatarAlexander Shiyan <shc_work@mail.ru>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 41b3996e
...@@ -65,7 +65,7 @@ static void gpio_clps711x_set(struct gpio_chip *chip, unsigned offset, ...@@ -65,7 +65,7 @@ static void gpio_clps711x_set(struct gpio_chip *chip, unsigned offset,
spin_unlock_irqrestore(&gpio->lock, flags); spin_unlock_irqrestore(&gpio->lock, flags);
} }
static int gpio_clps711x_direction_in(struct gpio_chip *chip, unsigned offset) static int gpio_clps711x_dir_in(struct gpio_chip *chip, unsigned offset)
{ {
int tmp; int tmp;
unsigned long flags; unsigned long flags;
...@@ -79,7 +79,7 @@ static int gpio_clps711x_direction_in(struct gpio_chip *chip, unsigned offset) ...@@ -79,7 +79,7 @@ static int gpio_clps711x_direction_in(struct gpio_chip *chip, unsigned offset)
return 0; return 0;
} }
static int gpio_clps711x_direction_out(struct gpio_chip *chip, unsigned offset, static int gpio_clps711x_dir_out(struct gpio_chip *chip, unsigned offset,
int value) int value)
{ {
int tmp; int tmp;
...@@ -98,17 +98,49 @@ static int gpio_clps711x_direction_out(struct gpio_chip *chip, unsigned offset, ...@@ -98,17 +98,49 @@ static int gpio_clps711x_direction_out(struct gpio_chip *chip, unsigned offset,
return 0; return 0;
} }
struct clps711x_gpio_port { static int gpio_clps711x_dir_in_inv(struct gpio_chip *chip, unsigned offset)
{
int tmp;
unsigned long flags;
struct clps711x_gpio *gpio = dev_get_drvdata(chip->dev);
spin_lock_irqsave(&gpio->lock, flags);
tmp = readb(clps711x_pdir(chip)) | (1 << offset);
writeb(tmp, clps711x_pdir(chip));
spin_unlock_irqrestore(&gpio->lock, flags);
return 0;
}
static int gpio_clps711x_dir_out_inv(struct gpio_chip *chip, unsigned offset,
int value)
{
int tmp;
unsigned long flags;
struct clps711x_gpio *gpio = dev_get_drvdata(chip->dev);
spin_lock_irqsave(&gpio->lock, flags);
tmp = readb(clps711x_pdir(chip)) & ~(1 << offset);
writeb(tmp, clps711x_pdir(chip));
tmp = readb(clps711x_port(chip)) & ~(1 << offset);
if (value)
tmp |= 1 << offset;
writeb(tmp, clps711x_port(chip));
spin_unlock_irqrestore(&gpio->lock, flags);
return 0;
}
static struct {
char *name; char *name;
int nr; int nr;
}; int inv_dir;
} clps711x_gpio_ports[] __initconst = {
static const struct clps711x_gpio_port clps711x_gpio_ports[] __initconst = { { "PORTA", 8, 0, },
{ "PORTA", 8, }, { "PORTB", 8, 0, },
{ "PORTB", 8, }, { "PORTC", 8, 0, },
{ "PORTC", 8, }, { "PORTD", 8, 1, },
{ "PORTD", 8, }, { "PORTE", 3, 0, },
{ "PORTE", 3, },
}; };
static int __init gpio_clps711x_init(void) static int __init gpio_clps711x_init(void)
...@@ -145,10 +177,15 @@ static int __init gpio_clps711x_init(void) ...@@ -145,10 +177,15 @@ static int __init gpio_clps711x_init(void)
gpio->chip[i].label = clps711x_gpio_ports[i].name; gpio->chip[i].label = clps711x_gpio_ports[i].name;
gpio->chip[i].base = i * 8; gpio->chip[i].base = i * 8;
gpio->chip[i].ngpio = clps711x_gpio_ports[i].nr; gpio->chip[i].ngpio = clps711x_gpio_ports[i].nr;
gpio->chip[i].direction_input = gpio_clps711x_direction_in;
gpio->chip[i].get = gpio_clps711x_get; gpio->chip[i].get = gpio_clps711x_get;
gpio->chip[i].direction_output = gpio_clps711x_direction_out;
gpio->chip[i].set = gpio_clps711x_set; gpio->chip[i].set = gpio_clps711x_set;
if (!clps711x_gpio_ports[i].inv_dir) {
gpio->chip[i].direction_input = gpio_clps711x_dir_in;
gpio->chip[i].direction_output = gpio_clps711x_dir_out;
} else {
gpio->chip[i].direction_input = gpio_clps711x_dir_in_inv;
gpio->chip[i].direction_output = gpio_clps711x_dir_out_inv;
}
WARN_ON(gpiochip_add(&gpio->chip[i])); WARN_ON(gpiochip_add(&gpio->chip[i]));
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment