Commit c9eac462 authored by Axel Lin's avatar Axel Lin Committed by Mark Brown

ASoC: sta350: Use devm_gpiod_get_optional at appropriate place

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 34d7c390
("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: default avatarAxel Lin <axel.lin@ingics.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent b787f68c
...@@ -1218,7 +1218,7 @@ static int sta350_i2c_probe(struct i2c_client *i2c, ...@@ -1218,7 +1218,7 @@ static int sta350_i2c_probe(struct i2c_client *i2c,
if (IS_ERR(sta350->gpiod_nreset)) if (IS_ERR(sta350->gpiod_nreset))
return PTR_ERR(sta350->gpiod_nreset); return PTR_ERR(sta350->gpiod_nreset);
sta350->gpiod_power_down = devm_gpiod_get(dev, "power-down", sta350->gpiod_power_down = devm_gpiod_get_optional(dev, "power-down",
GPIOD_OUT_LOW); GPIOD_OUT_LOW);
if (IS_ERR(sta350->gpiod_power_down)) if (IS_ERR(sta350->gpiod_power_down))
return PTR_ERR(sta350->gpiod_power_down); return PTR_ERR(sta350->gpiod_power_down);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment