gpio: rc5t583: use gpiochip data pointer
authorLinus Walleij <linus.walleij@linaro.org>
Mon, 7 Dec 2015 13:08:12 +0000 (14:08 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 5 Jan 2016 10:21:12 +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().

Acked-by: Laxman Dewangan <ldewangan@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-rc5t583.c

index f26d9f7..1e2d210 100644 (file)
@@ -34,14 +34,9 @@ struct rc5t583_gpio {
        struct rc5t583 *rc5t583;
 };
 
-static inline struct rc5t583_gpio *to_rc5t583_gpio(struct gpio_chip *chip)
-{
-       return container_of(chip, struct rc5t583_gpio, gpio_chip);
-}
-
 static int rc5t583_gpio_get(struct gpio_chip *gc, unsigned int offset)
 {
-       struct rc5t583_gpio *rc5t583_gpio = to_rc5t583_gpio(gc);
+       struct rc5t583_gpio *rc5t583_gpio = gpiochip_get_data(gc);
        struct device *parent = rc5t583_gpio->rc5t583->dev;
        uint8_t val = 0;
        int ret;
@@ -55,7 +50,7 @@ static int rc5t583_gpio_get(struct gpio_chip *gc, unsigned int offset)
 
 static void rc5t583_gpio_set(struct gpio_chip *gc, unsigned int offset, int val)
 {
-       struct rc5t583_gpio *rc5t583_gpio = to_rc5t583_gpio(gc);
+       struct rc5t583_gpio *rc5t583_gpio = gpiochip_get_data(gc);
        struct device *parent = rc5t583_gpio->rc5t583->dev;
        if (val)
                rc5t583_set_bits(parent, RC5T583_GPIO_IOOUT, BIT(offset));
@@ -65,7 +60,7 @@ static void rc5t583_gpio_set(struct gpio_chip *gc, unsigned int offset, int val)
 
 static int rc5t583_gpio_dir_input(struct gpio_chip *gc, unsigned int offset)
 {
-       struct rc5t583_gpio *rc5t583_gpio = to_rc5t583_gpio(gc);
+       struct rc5t583_gpio *rc5t583_gpio = gpiochip_get_data(gc);
        struct device *parent = rc5t583_gpio->rc5t583->dev;
        int ret;
 
@@ -80,7 +75,7 @@ static int rc5t583_gpio_dir_input(struct gpio_chip *gc, unsigned int offset)
 static int rc5t583_gpio_dir_output(struct gpio_chip *gc, unsigned offset,
                        int value)
 {
-       struct rc5t583_gpio *rc5t583_gpio = to_rc5t583_gpio(gc);
+       struct rc5t583_gpio *rc5t583_gpio = gpiochip_get_data(gc);
        struct device *parent = rc5t583_gpio->rc5t583->dev;
        int ret;
 
@@ -95,7 +90,7 @@ static int rc5t583_gpio_dir_output(struct gpio_chip *gc, unsigned offset,
 
 static int rc5t583_gpio_to_irq(struct gpio_chip *gc, unsigned offset)
 {
-       struct rc5t583_gpio *rc5t583_gpio = to_rc5t583_gpio(gc);
+       struct rc5t583_gpio *rc5t583_gpio = gpiochip_get_data(gc);
 
        if (offset < RC5T583_MAX_GPIO)
                return rc5t583_gpio->rc5t583->irq_base +
@@ -105,7 +100,7 @@ static int rc5t583_gpio_to_irq(struct gpio_chip *gc, unsigned offset)
 
 static void rc5t583_gpio_free(struct gpio_chip *gc, unsigned offset)
 {
-       struct rc5t583_gpio *rc5t583_gpio = to_rc5t583_gpio(gc);
+       struct rc5t583_gpio *rc5t583_gpio = gpiochip_get_data(gc);
        struct device *parent = rc5t583_gpio->rc5t583->dev;
 
        rc5t583_set_bits(parent, RC5T583_GPIO_PGSEL, BIT(offset));
@@ -141,7 +136,7 @@ static int rc5t583_gpio_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, rc5t583_gpio);
 
-       return gpiochip_add(&rc5t583_gpio->gpio_chip);
+       return gpiochip_add_data(&rc5t583_gpio->gpio_chip, rc5t583_gpio);
 }
 
 static int rc5t583_gpio_remove(struct platform_device *pdev)