ASoC: sta350: Use devm_gpiod_get_optional at appropriate place
authorAxel Lin <axel.lin@ingics.com>
Sat, 9 May 2015 15:09:32 +0000 (23:09 +0800)
committerMark Brown <broonie@kernel.org>
Tue, 12 May 2015 17:52:42 +0000 (18:52 +0100)
devm_gpiod_get_optional is equivalent to devm_gpiod_get(), except that
when no GPIO was assigned to the requested function it will return NULL.
This is convenient for drivers that need to handle optional GPIOs.

I just checked the code in commit 34d7c3905adb9a9
("ASoC: improve usage of gpiod API") and found that it should use
devm_gpiod_get_optional rather than devm_gpiod_get here.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/sta350.c

index 669e3228241e39d14a2cb009a508cbdb1f6b9fcb..cc67a24c6e318a9f2ca2da1752d7303647e695bf 100644 (file)
@@ -1218,8 +1218,8 @@ static int sta350_i2c_probe(struct i2c_client *i2c,
        if (IS_ERR(sta350->gpiod_nreset))
                return PTR_ERR(sta350->gpiod_nreset);
 
-       sta350->gpiod_power_down = devm_gpiod_get(dev, "power-down",
-                                                 GPIOD_OUT_LOW);
+       sta350->gpiod_power_down = devm_gpiod_get_optional(dev, "power-down",
+                                                          GPIOD_OUT_LOW);
        if (IS_ERR(sta350->gpiod_power_down))
                return PTR_ERR(sta350->gpiod_power_down);