soc: qcom: apr: Fixup the error displayed on lookup failure
authorSibi Sankar <sibis@codeaurora.org>
Tue, 15 Sep 2020 15:42:32 +0000 (21:12 +0530)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Tue, 15 Sep 2020 16:12:59 +0000 (16:12 +0000)
APR client incorrectly prints out "ret" variable on pdr_add_lookup failure,
it should be printing the error value returned by the lookup instead.

Fixes: 8347356626028 ("soc: qcom: apr: Add avs/audio tracking functionality")
Signed-off-by: Sibi Sankar <sibis@codeaurora.org>
Link: https://lore.kernel.org/r/20200915154232.27523-1-sibis@codeaurora.org
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
drivers/soc/qcom/apr.c

index 1f35b09..7abfc8c 100644 (file)
@@ -328,7 +328,7 @@ static int of_apr_add_pd_lookups(struct device *dev)
 
                pds = pdr_add_lookup(apr->pdr, service_name, service_path);
                if (IS_ERR(pds) && PTR_ERR(pds) != -EALREADY) {
-                       dev_err(dev, "pdr add lookup failed: %d\n", ret);
+                       dev_err(dev, "pdr add lookup failed: %ld\n", PTR_ERR(pds));
                        return PTR_ERR(pds);
                }
        }