[PATCH v4 3/6] remoteproc: qcom: pas: Fix the PAS context creation placement

[Mukesh Ojha]

DTB PAS context creation should be done only for subsystems that support
a DTB firmware binary; otherwise, memory is wasted. Move the context
creation to the appropriate location.

Reviewed-by: Konrad Dybcio <konrad.dyb...@oss.qualcomm.com>
Signed-off-by: Mukesh Ojha <mukesh.o...@oss.qualcomm.com>
---
 drivers/remoteproc/qcom_q6v5_pas.c | 32 ++++++++++++++----------------
 1 file changed, 15 insertions(+), 17 deletions(-)

diff --git a/drivers/remoteproc/qcom_q6v5_pas.c 
b/drivers/remoteproc/qcom_q6v5_pas.c
index 2785d19408d1..a64fbf7225aa 100644
--- a/drivers/remoteproc/qcom_q6v5_pas.c
+++ b/drivers/remoteproc/qcom_q6v5_pas.c
@@ -620,6 +620,7 @@ static void qcom_pas_pds_detach(struct qcom_pas *pas, 
struct device **pds, size_
 
 static int qcom_pas_alloc_memory_region(struct qcom_pas *pas)
 {
+       struct rproc *rproc = pas->rproc;
        struct resource res;
        int ret;
 
@@ -637,6 +638,12 @@ static int qcom_pas_alloc_memory_region(struct qcom_pas 
*pas)
                return PTR_ERR(pas->mem_region);
        }
 
+       pas->pas_ctx = devm_qcom_scm_pas_context_alloc(pas->dev, pas->pas_id,
+                                                      pas->mem_phys, 
pas->mem_size);
+       if (IS_ERR(pas->pas_ctx))
+               return PTR_ERR(pas->pas_ctx);
+
+       pas->pas_ctx->use_tzmem = rproc->has_iommu;
        if (!pas->dtb_pas_id)
                return 0;
 
@@ -654,6 +661,14 @@ static int qcom_pas_alloc_memory_region(struct qcom_pas 
*pas)
                return PTR_ERR(pas->dtb_mem_region);
        }
 
+       pas->dtb_pas_ctx = devm_qcom_scm_pas_context_alloc(pas->dev, 
pas->dtb_pas_id,
+                                                          pas->dtb_mem_phys,
+                                                          pas->dtb_mem_size);
+       if (IS_ERR(pas->dtb_pas_ctx))
+               return PTR_ERR(pas->dtb_pas_ctx);
+
+       pas->dtb_pas_ctx->use_tzmem = rproc->has_iommu;
+
        return 0;
 }
 
@@ -835,23 +850,6 @@ static int qcom_pas_probe(struct platform_device *pdev)
 
        qcom_add_ssr_subdev(rproc, &pas->ssr_subdev, desc->ssr_name);
 
-       pas->pas_ctx = devm_qcom_scm_pas_context_alloc(pas->dev, pas->pas_id,
-                                                      pas->mem_phys, 
pas->mem_size);
-       if (IS_ERR(pas->pas_ctx)) {
-               ret = PTR_ERR(pas->pas_ctx);
-               goto remove_ssr_sysmon;
-       }
-
-       pas->dtb_pas_ctx = devm_qcom_scm_pas_context_alloc(pas->dev, 
pas->dtb_pas_id,
-                                                          pas->dtb_mem_phys,
-                                                          pas->dtb_mem_size);
-       if (IS_ERR(pas->dtb_pas_ctx)) {
-               ret = PTR_ERR(pas->dtb_pas_ctx);
-               goto remove_ssr_sysmon;
-       }
-
-       pas->pas_ctx->use_tzmem = rproc->has_iommu;
-       pas->dtb_pas_ctx->use_tzmem = rproc->has_iommu;
        ret = rproc_add(rproc);
        if (ret)
                goto remove_ssr_sysmon;
-- 
2.53.0