ASoC: max98357a: use flags argument of devm_gpiod_get to set direction
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tue, 19 May 2015 06:58:09 +0000 (08:58 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 20 May 2015 18:07:51 +0000 (19:07 +0100)
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.

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

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Kenneth Westfield <kwestfie@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/max98357a.c

index bf3e933ee895752352e90f610195f39fc6cc058f..3a2fda08a893c865023d61fec636e65e3c977577 100644 (file)
@@ -60,13 +60,12 @@ static int max98357a_codec_probe(struct snd_soc_codec *codec)
 {
        struct gpio_desc *sdmode;
 
-       sdmode = devm_gpiod_get(codec->dev, "sdmode");
+       sdmode = devm_gpiod_get(codec->dev, "sdmode", GPIOD_OUT_LOW);
        if (IS_ERR(sdmode)) {
                dev_err(codec->dev, "%s() unable to get sdmode GPIO: %ld\n",
                                __func__, PTR_ERR(sdmode));
                return PTR_ERR(sdmode);
        }
-       gpiod_direction_output(sdmode, 0);
        snd_soc_codec_set_drvdata(codec, sdmode);
 
        return 0;