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 <[email protected]>
Signed-off-by: Linus Walleij <[email protected]>
---
It seems to be unclear who maintains platform/x86, so I'm just
gonna merge this patch into the GPIO tree unless there is a protest
from someone who wants to have a say.
---
 drivers/platform/x86/intel_pmic_gpio.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/platform/x86/intel_pmic_gpio.c 
b/drivers/platform/x86/intel_pmic_gpio.c
index 0e73fd10ba72..63b371d6ee55 100644
--- a/drivers/platform/x86/intel_pmic_gpio.c
+++ b/drivers/platform/x86/intel_pmic_gpio.c
@@ -30,7 +30,7 @@
 #include <linux/ioport.h>
 #include <linux/init.h>
 #include <linux/io.h>
-#include <linux/gpio.h>
+#include <linux/gpio/driver.h>
 #include <asm/intel_scu_ipc.h>
 #include <linux/device.h>
 #include <linux/intel_pmic_gpio.h>
@@ -174,7 +174,7 @@ static int pmic_irq_type(struct irq_data *data, unsigned 
type)
 
 static int pmic_gpio_to_irq(struct gpio_chip *chip, unsigned offset)
 {
-       struct pmic_gpio *pg = container_of(chip, struct pmic_gpio, chip);
+       struct pmic_gpio *pg = gpiochip_get_data(chip);
 
        return pg->irq_base + offset;
 }
@@ -279,7 +279,7 @@ static int platform_pmic_gpio_probe(struct platform_device 
*pdev)
        mutex_init(&pg->buslock);
 
        pg->chip.parent = dev;
-       retval = gpiochip_add(&pg->chip);
+       retval = gpiochip_add_data(&pg->chip, pg);
        if (retval) {
                pr_err("Can not add pmic gpio chip\n");
                goto err;
-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to