remoteproc: qcom_wcnss: Use devm_qcom_smem_state_get()

Use the new managed devm_qcom_smem_state_get() variant instead of
calling qcom_smem_state_put() explicitly in remove() to simplify the
code a bit.

Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Link: https://lore.kernel.org/r/20210618111556.53416-3-stephan@gerhold.net
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

diff --git a/drivers/remoteproc/qcom_wcnss.c b/drivers/remoteproc/qcom_wcnss.c
index 5f3455aa7e0eb9ecf904e940d58e0f5eea359d3c..f1cbc6b2edbb3191719d1cf5e5ecac12925649b0 100644 (file)
--- a/drivers/remoteproc/qcom_wcnss.c
+++ b/drivers/remoteproc/qcom_wcnss.c
@@ -624,8 +624,8 @@ static int wcnss_probe(struct platform_device *pdev)
        wcnss->stop_ack_irq = ret;
 
        if (wcnss->stop_ack_irq) {
-               wcnss->state = qcom_smem_state_get(&pdev->dev, "stop",
-                                                  &wcnss->stop_bit);
+               wcnss->state = devm_qcom_smem_state_get(&pdev->dev, "stop",
+                                                       &wcnss->stop_bit);
                if (IS_ERR(wcnss->state)) {
                        ret = PTR_ERR(wcnss->state);
                        goto detach_pds;
@@ -659,7 +659,6 @@ static int wcnss_remove(struct platform_device *pdev)
 
        of_platform_depopulate(&pdev->dev);
 
-       qcom_smem_state_put(wcnss->state);
        rproc_del(wcnss->rproc);
 
        qcom_remove_sysmon_subdev(wcnss->sysmon);