ASoC: samsung: Fix invalid argument when devm_gpiod_get is called
authorJaechul Lee <jcsing.lee@samsung.com>
Wed, 6 Sep 2017 01:04:15 +0000 (10:04 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Sep 2018 06:38:14 +0000 (08:38 +0200)
commit 975b6a93088e83a41ba2f0dec2f086678fdb2a7a upstream.

devm_gpiod_get is called with GPIOF_OUT_INIT_LOW but the function doesn't
allow the parameters. Unluckily, GPIOF_OUT_INIT_LOW is same value as
GPIOD_ASIS and gpio direction isn't set properly.

Muted stream comes up when I try recording some sounds on TM2. mic-bias
gpiod state can't be changed because the gpiod is created with the invalid
parameter. The gpio should be set GPIOD_OUT_HIGH.

Fixes: 1bfbc260a5b4 ("ASoC: samsung: Add machine driver for Exynos5433 based TM2 board")
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
sound/soc/samsung/tm2_wm5110.c

index 68698f3..52e2fbe 100644 (file)
@@ -436,8 +436,7 @@ static int tm2_probe(struct platform_device *pdev)
        snd_soc_card_set_drvdata(card, priv);
        card->dev = dev;
 
-       priv->gpio_mic_bias = devm_gpiod_get(dev, "mic-bias",
-                                               GPIOF_OUT_INIT_LOW);
+       priv->gpio_mic_bias = devm_gpiod_get(dev, "mic-bias", GPIOD_OUT_HIGH);
        if (IS_ERR(priv->gpio_mic_bias)) {
                dev_err(dev, "Failed to get mic bias gpio\n");
                return PTR_ERR(priv->gpio_mic_bias);