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)
committerMark Brown <broonie@kernel.org>
Tue, 19 Sep 2017 14:59:07 +0000 (15:59 +0100)
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>
sound/soc/samsung/tm2_wm5110.c

index 710e215..a55d187 100644 (file)
@@ -439,8 +439,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);