Skip to content

Commit

Permalink
gpio: menz127: Drop lock field from struct men_z127_gpio
Browse files Browse the repository at this point in the history
Current code uses a uninitialized spin lock.
bgpio_init() already initialized a spin lock, so let's switch to use
&gc->bgpio_lock instead and remove the lock from struct men_z127_gpio.

Fixes: f436bc2 "gpio: add driver for MEN 16Z127 GPIO controller"
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
  • Loading branch information
Axel Lin authored and Linus Walleij committed Mar 30, 2016
1 parent f55532a commit fd975a7
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions drivers/gpio/gpio-menz127.c
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ struct men_z127_gpio {
void __iomem *reg_base;
struct mcb_device *mdev;
struct resource *mem;
spinlock_t lock;
};

static int men_z127_debounce(struct gpio_chip *gc, unsigned gpio,
Expand Down Expand Up @@ -69,7 +68,7 @@ static int men_z127_debounce(struct gpio_chip *gc, unsigned gpio,
debounce /= 50;
}

spin_lock(&priv->lock);
spin_lock(&gc->bgpio_lock);

db_en = readl(priv->reg_base + MEN_Z127_DBER);

Expand All @@ -84,7 +83,7 @@ static int men_z127_debounce(struct gpio_chip *gc, unsigned gpio,
writel(db_en, priv->reg_base + MEN_Z127_DBER);
writel(db_cnt, priv->reg_base + GPIO_TO_DBCNT_REG(gpio));

spin_unlock(&priv->lock);
spin_unlock(&gc->bgpio_lock);

return 0;
}
Expand All @@ -97,7 +96,7 @@ static int men_z127_request(struct gpio_chip *gc, unsigned gpio_pin)
if (gpio_pin >= gc->ngpio)
return -EINVAL;

spin_lock(&priv->lock);
spin_lock(&gc->bgpio_lock);
od_en = readl(priv->reg_base + MEN_Z127_ODER);

if (gpiochip_line_is_open_drain(gc, gpio_pin))
Expand All @@ -106,7 +105,7 @@ static int men_z127_request(struct gpio_chip *gc, unsigned gpio_pin)
od_en &= ~BIT(gpio_pin);

writel(od_en, priv->reg_base + MEN_Z127_ODER);
spin_unlock(&priv->lock);
spin_unlock(&gc->bgpio_lock);

return 0;
}
Expand Down

0 comments on commit fd975a7

Please sign in to comment.