gpio: ml-ioh: use gpiochip data pointer
authorLinus Walleij <linus.walleij@linaro.org>
Mon, 7 Dec 2015 09:12:05 +0000 (10:12 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 5 Jan 2016 10:21:08 +0000 (11:21 +0100)
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: Feng Tang <feng.tang@intel.com>
Cc: Tomoya MORINAGA <tomoya-linux@dsn.okisemi.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-ml-ioh.c

index 7ef704343f056455d944a5cb987c34f4f9b41070..796a5a4bc4f5e574a3dd05af6f546aa0775b01a2 100644 (file)
@@ -106,7 +106,7 @@ static const int num_ports[] = {6, 12, 16, 16, 15, 16, 16, 12};
 static void ioh_gpio_set(struct gpio_chip *gpio, unsigned nr, int val)
 {
        u32 reg_val;
-       struct ioh_gpio *chip = container_of(gpio, struct ioh_gpio, gpio);
+       struct ioh_gpio *chip = gpiochip_get_data(gpio);
        unsigned long flags;
 
        spin_lock_irqsave(&chip->spinlock, flags);
@@ -122,7 +122,7 @@ static void ioh_gpio_set(struct gpio_chip *gpio, unsigned nr, int val)
 
 static int ioh_gpio_get(struct gpio_chip *gpio, unsigned nr)
 {
-       struct ioh_gpio *chip = container_of(gpio, struct ioh_gpio, gpio);
+       struct ioh_gpio *chip = gpiochip_get_data(gpio);
 
        return !!(ioread32(&chip->reg->regs[chip->ch].pi) & (1 << nr));
 }
@@ -130,7 +130,7 @@ static int ioh_gpio_get(struct gpio_chip *gpio, unsigned nr)
 static int ioh_gpio_direction_output(struct gpio_chip *gpio, unsigned nr,
                                     int val)
 {
-       struct ioh_gpio *chip = container_of(gpio, struct ioh_gpio, gpio);
+       struct ioh_gpio *chip = gpiochip_get_data(gpio);
        u32 pm;
        u32 reg_val;
        unsigned long flags;
@@ -155,7 +155,7 @@ static int ioh_gpio_direction_output(struct gpio_chip *gpio, unsigned nr,
 
 static int ioh_gpio_direction_input(struct gpio_chip *gpio, unsigned nr)
 {
-       struct ioh_gpio *chip = container_of(gpio, struct ioh_gpio, gpio);
+       struct ioh_gpio *chip = gpiochip_get_data(gpio);
        u32 pm;
        unsigned long flags;
 
@@ -225,7 +225,7 @@ static void ioh_gpio_restore_reg_conf(struct ioh_gpio *chip)
 
 static int ioh_gpio_to_irq(struct gpio_chip *gpio, unsigned offset)
 {
-       struct ioh_gpio *chip = container_of(gpio, struct ioh_gpio, gpio);
+       struct ioh_gpio *chip = gpiochip_get_data(gpio);
        return chip->irq_base + offset;
 }
 
@@ -450,7 +450,7 @@ static int ioh_gpio_probe(struct pci_dev *pdev,
                chip->ch = i;
                spin_lock_init(&chip->spinlock);
                ioh_gpio_setup(chip, num_ports[i]);
-               ret = gpiochip_add(&chip->gpio);
+               ret = gpiochip_add_data(&chip->gpio, chip);
                if (ret) {
                        dev_err(&pdev->dev, "IOH gpio: Failed to register GPIO\n");
                        goto err_gpiochip_add;
This page took 0.026143 seconds and 5 git commands to generate.