iio: light: stk3310: use flags argument of devm_gpiod_get
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 12 Jun 2015 07:04:55 +0000 (09:04 +0200)
committerUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Mon, 6 Jul 2015 08:10:21 +0000 (10:10 +0200)
Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions)
which appeared in v3.17-rc1, the gpiod_get* functions take an additional
parameter that allows to specify direction and initial value for output.

Simplify driver accordingly. Furthermore this is one caller less that
stops us making the flags argument to gpiod_get*() mandatory.

Acked-by: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
drivers/iio/light/stk3310.c

index fee4297d7c8f8e039dd1716c655deff831f01a84..84c77d42a2c66dff706b8b0846fc250c5336a8ce 100644 (file)
@@ -488,16 +488,12 @@ static int stk3310_gpio_probe(struct i2c_client *client)
        dev = &client->dev;
 
        /* gpio interrupt pin */
-       gpio = devm_gpiod_get_index(dev, STK3310_GPIO, 0);
+       gpio = devm_gpiod_get_index(dev, STK3310_GPIO, 0, GPIOD_IN);
        if (IS_ERR(gpio)) {
                dev_err(dev, "acpi gpio get index failed\n");
                return PTR_ERR(gpio);
        }
 
-       ret = gpiod_direction_input(gpio);
-       if (ret)
-               return ret;
-
        ret = gpiod_to_irq(gpio);
        dev_dbg(dev, "GPIO resource, no:%d irq:%d\n", desc_to_gpio(gpio), ret);