ASoC: stm32: sai: add missing put_device()
authorWen Yang <yellowriver2010@hotmail.com>
Sat, 9 Feb 2019 10:41:09 +0000 (10:41 +0000)
committerMark Brown <broonie@kernel.org>
Thu, 14 Feb 2019 13:12:07 +0000 (13:12 +0000)
The of_find_device_by_node() takes a reference to the underlying device
structure, we should release that reference.

Fixes: 7dd0d835582f ("ASoC: stm32: sai: simplify sync modes management")
Signed-off-by: Wen Yang <yellowriver2010@hotmail.com>
Acked-by: Olivier Moysan <olivier.moysan@st.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/stm/stm32_sai.c

index bcb35cae2a2c3b8992885d66d6336d2f69e5ba4b..14c9591aae4260d94f6d19231aaa56ad436e5469 100644 (file)
@@ -112,16 +112,21 @@ static int stm32_sai_set_sync(struct stm32_sai_data *sai_client,
        if (!sai_provider) {
                dev_err(&sai_client->pdev->dev,
                        "SAI sync provider data not found\n");
-               return -EINVAL;
+               ret = -EINVAL;
+               goto out_put_dev;
        }
 
        /* Configure sync client */
        ret = stm32_sai_sync_conf_client(sai_client, synci);
        if (ret < 0)
-               return ret;
+               goto out_put_dev;
 
        /* Configure sync provider */
-       return stm32_sai_sync_conf_provider(sai_provider, synco);
+       ret = stm32_sai_sync_conf_provider(sai_provider, synco);
+
+out_put_dev:
+       put_device(&pdev->dev);
+       return ret;
 }
 
 static int stm32_sai_probe(struct platform_device *pdev)