ASoC: tegra: max98090: fix invalid gpio value

hp_mute gpio is not detected properly leading
to kernel crash.

Bug 1656789

Change-Id: I35eb8890a7f916105ce5128ee01fcec48adf327b
Signed-off-by: Bibek Basu <bbasu@nvidia.com>
Reviewed-on: http://git-master/r/781581
Reviewed-by: Laxman Dewangan <ldewangan@nvidia.com>

diff --git a/sound/soc/tegra/tegra_max98090.c b/sound/soc/tegra/tegra_max98090.c
index 53f4bfb9b591732d3d9a4b01e24c8dd9143fb2c0..5dcf88daf54416ae6d772b9ff3317bc83ab7b582 100644 (file)
--- a/sound/soc/tegra/tegra_max98090.c
+++ b/sound/soc/tegra/tegra_max98090.c
@@ -1050,7 +1050,8 @@ static int tegra_max98090_init(struct snd_soc_pcm_runtime *rtd)
        if (gpio_is_valid(pdata->gpio_hp_mute)) {
                ret = gpio_request(pdata->gpio_hp_mute, "hp_mute");
                if (ret) {
-                       dev_err(card->dev, "cannot get hp_mute gpio\n");
+                       dev_err(card->dev, "cannot get hp_mute gpio %d\n",
+                                       pdata->gpio_hp_mute);
                        return ret;
                }
                machine->gpio_requested |= GPIO_HP_MUTE;
@@ -1349,6 +1350,7 @@ static struct tegra_asoc_platform_data *
 {
        struct tegra_asoc_platform_data *pdata;
        struct device_node *np = NULL;
+       int ret;
 
        np = dev->of_node;
        if (!np)
@@ -1366,8 +1368,14 @@ static struct tegra_asoc_platform_data *
                                &pdata->codec_name);
        of_property_read_string(np, "nvidia,codec-dai-name",
                                &pdata->codec_dai_name);
-       of_property_read_u32(np, "nvidia,gpio-hp-det",
+       ret = of_property_read_u32(np, "nvidia,gpio-hp-det",
                                &pdata->gpio_hp_det);
+       if (ret)
+               pdata->gpio_hp_det = ret;
+       ret = of_property_read_u32(np, "nvidia,gpio-hp-mute",
+                               &pdata->gpio_hp_mute);
+       if (ret)
+               pdata->gpio_hp_mute = ret;
        of_property_read_u32(np, "nvidia,num-links",
                                &pdata->num_links);
        pdata->edp_support =