Skip to content

Commit 85865e8

Browse files
krzkandersson
authored andcommitted
remoteproc: qcom_q6v5_adsp: Simplify with dev_err_probe()
Use dev_err_probe() to make error and defer code handling simpler. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Link: https://lore.kernel.org/r/20241011-remote-proc-dev-err-probe-v1-5-5abb4fc61eca@linaro.org Signed-off-by: Bjorn Andersson <andersson@kernel.org>
1 parent 8a47704 commit 85865e8

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

drivers/remoteproc/qcom_q6v5_adsp.c

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -534,15 +534,11 @@ static const struct rproc_ops adsp_ops = {
534534
static int adsp_init_clock(struct qcom_adsp *adsp, const char **clk_ids)
535535
{
536536
int num_clks = 0;
537-
int i, ret;
537+
int i;
538538

539539
adsp->xo = devm_clk_get(adsp->dev, "xo");
540-
if (IS_ERR(adsp->xo)) {
541-
ret = PTR_ERR(adsp->xo);
542-
if (ret != -EPROBE_DEFER)
543-
dev_err(adsp->dev, "failed to get xo clock");
544-
return ret;
545-
}
540+
if (IS_ERR(adsp->xo))
541+
return dev_err_probe(adsp->dev, PTR_ERR(adsp->xo), "failed to get xo clock");
546542

547543
for (i = 0; clk_ids[i]; i++)
548544
num_clks++;
@@ -708,10 +704,9 @@ static int adsp_probe(struct platform_device *pdev)
708704
return ret;
709705

710706
ret = qcom_rproc_pds_attach(adsp, desc->pd_names, desc->num_pds);
711-
if (ret < 0) {
712-
dev_err(&pdev->dev, "Failed to attach proxy power domains\n");
713-
return ret;
714-
}
707+
if (ret < 0)
708+
return dev_err_probe(&pdev->dev, ret,
709+
"Failed to attach proxy power domains\n");
715710

716711
ret = adsp_init_reset(adsp);
717712
if (ret)

0 commit comments

Comments
 (0)