diff options
author | Wei Yongjun <weiyj.lk@gmail.com> | 2016-07-29 11:56:52 -0400 |
---|---|---|
committer | Bjorn Andersson <bjorn.andersson@linaro.org> | 2016-08-10 13:51:35 -0400 |
commit | 4e968d9e46ceee57e3f5b96d1486745416c9a4a5 (patch) | |
tree | 18d88d4d0246ee775444e62d67b6feae4c233c93 | |
parent | c14b9a940e53a50f8fc56cc613525d2db40f19be (diff) |
remoteproc: qcom: hexagon: Fix error return code in q6v5_probe()
Fix to return a negative error code from the state get failed error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <weiyj.lk@gmail.com>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
-rw-r--r-- | drivers/remoteproc/qcom_q6v5_pil.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/remoteproc/qcom_q6v5_pil.c b/drivers/remoteproc/qcom_q6v5_pil.c index 2a1b2c7d8f2c..05b04573e87d 100644 --- a/drivers/remoteproc/qcom_q6v5_pil.c +++ b/drivers/remoteproc/qcom_q6v5_pil.c | |||
@@ -863,8 +863,10 @@ static int q6v5_probe(struct platform_device *pdev) | |||
863 | goto free_rproc; | 863 | goto free_rproc; |
864 | 864 | ||
865 | qproc->state = qcom_smem_state_get(&pdev->dev, "stop", &qproc->stop_bit); | 865 | qproc->state = qcom_smem_state_get(&pdev->dev, "stop", &qproc->stop_bit); |
866 | if (IS_ERR(qproc->state)) | 866 | if (IS_ERR(qproc->state)) { |
867 | ret = PTR_ERR(qproc->state); | ||
867 | goto free_rproc; | 868 | goto free_rproc; |
869 | } | ||
868 | 870 | ||
869 | ret = rproc_add(rproc); | 871 | ret = rproc_add(rproc); |
870 | if (ret) | 872 | if (ret) |