Commit 510f4871 authored by Linus Walleij's avatar Linus Walleij

gpio: pch: use gpiochip data pointer

This makes the driver use the data pointer added to the gpio_chip
to store a pointer to the state container instead of relying on
container_of().

Cc: Thierry Reding <treding@nvidia.com>
Cc: Daniel Krueger <daniel.krueger@systec-electronic.com>
Reviewed-by: default avatarJean Delvare <jdelvare@suse.de>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 597358e4
...@@ -109,7 +109,7 @@ struct pch_gpio { ...@@ -109,7 +109,7 @@ struct pch_gpio {
static void pch_gpio_set(struct gpio_chip *gpio, unsigned nr, int val) static void pch_gpio_set(struct gpio_chip *gpio, unsigned nr, int val)
{ {
u32 reg_val; u32 reg_val;
struct pch_gpio *chip = container_of(gpio, struct pch_gpio, gpio); struct pch_gpio *chip = gpiochip_get_data(gpio);
unsigned long flags; unsigned long flags;
spin_lock_irqsave(&chip->spinlock, flags); spin_lock_irqsave(&chip->spinlock, flags);
...@@ -125,7 +125,7 @@ static void pch_gpio_set(struct gpio_chip *gpio, unsigned nr, int val) ...@@ -125,7 +125,7 @@ static void pch_gpio_set(struct gpio_chip *gpio, unsigned nr, int val)
static int pch_gpio_get(struct gpio_chip *gpio, unsigned nr) static int pch_gpio_get(struct gpio_chip *gpio, unsigned nr)
{ {
struct pch_gpio *chip = container_of(gpio, struct pch_gpio, gpio); struct pch_gpio *chip = gpiochip_get_data(gpio);
return !!(ioread32(&chip->reg->pi) & (1 << nr)); return !!(ioread32(&chip->reg->pi) & (1 << nr));
} }
...@@ -133,7 +133,7 @@ static int pch_gpio_get(struct gpio_chip *gpio, unsigned nr) ...@@ -133,7 +133,7 @@ static int pch_gpio_get(struct gpio_chip *gpio, unsigned nr)
static int pch_gpio_direction_output(struct gpio_chip *gpio, unsigned nr, static int pch_gpio_direction_output(struct gpio_chip *gpio, unsigned nr,
int val) int val)
{ {
struct pch_gpio *chip = container_of(gpio, struct pch_gpio, gpio); struct pch_gpio *chip = gpiochip_get_data(gpio);
u32 pm; u32 pm;
u32 reg_val; u32 reg_val;
unsigned long flags; unsigned long flags;
...@@ -158,7 +158,7 @@ static int pch_gpio_direction_output(struct gpio_chip *gpio, unsigned nr, ...@@ -158,7 +158,7 @@ static int pch_gpio_direction_output(struct gpio_chip *gpio, unsigned nr,
static int pch_gpio_direction_input(struct gpio_chip *gpio, unsigned nr) static int pch_gpio_direction_input(struct gpio_chip *gpio, unsigned nr)
{ {
struct pch_gpio *chip = container_of(gpio, struct pch_gpio, gpio); struct pch_gpio *chip = gpiochip_get_data(gpio);
u32 pm; u32 pm;
unsigned long flags; unsigned long flags;
...@@ -211,7 +211,7 @@ static void pch_gpio_restore_reg_conf(struct pch_gpio *chip) ...@@ -211,7 +211,7 @@ static void pch_gpio_restore_reg_conf(struct pch_gpio *chip)
static int pch_gpio_to_irq(struct gpio_chip *gpio, unsigned offset) static int pch_gpio_to_irq(struct gpio_chip *gpio, unsigned offset)
{ {
struct pch_gpio *chip = container_of(gpio, struct pch_gpio, gpio); struct pch_gpio *chip = gpiochip_get_data(gpio);
return chip->irq_base + offset; return chip->irq_base + offset;
} }
...@@ -397,7 +397,7 @@ static int pch_gpio_probe(struct pci_dev *pdev, ...@@ -397,7 +397,7 @@ static int pch_gpio_probe(struct pci_dev *pdev,
#ifdef CONFIG_OF_GPIO #ifdef CONFIG_OF_GPIO
chip->gpio.of_node = pdev->dev.of_node; chip->gpio.of_node = pdev->dev.of_node;
#endif #endif
ret = gpiochip_add(&chip->gpio); ret = gpiochip_add_data(&chip->gpio, chip);
if (ret) { if (ret) {
dev_err(&pdev->dev, "PCH gpio: Failed to register GPIO\n"); dev_err(&pdev->dev, "PCH gpio: Failed to register GPIO\n");
goto err_gpiochip_add; goto err_gpiochip_add;
......
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