remoteproc: qcom_q6v5_mss: Drop accesses to MPSS PERPH register space

7C retail devices using MSA based boot will result in a fuse combination
which will prevent accesses to MSS PERPH register space where the mpss
clocks and halt-nav reside. So drop all accesses to the MPSS PERPH
register space. Issuing HALT NAV request and turning on the mss clocks
as part of SSR will no longer be required since the modem firmware will
have the necessary fixes to ensure that there are no pending NAV DMA
transactions.

Tested-by: Evan Green <evgreen@chromium.org>
Signed-off-by: Sibi Sankar <sibis@codeaurora.org>
Link: https://lore.kernel.org/r/20200415145110.20624-3-sibis@codeaurora.org
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

diff --git a/drivers/remoteproc/qcom_q6v5_mss.c b/drivers/remoteproc/qcom_q6v5_mss.c
index 3a7fcd6485ebfc46a6f7b37a7571bea0cb0898b4..1ded14253a836ae7c1f4bac6ef8a98c158353414 100644 (file)
--- a/drivers/remoteproc/qcom_q6v5_mss.c
+++ b/drivers/remoteproc/qcom_q6v5_mss.c
@@ -69,13 +69,9 @@
 #define AXI_HALTREQ_REG                        0x0
 #define AXI_HALTACK_REG                        0x4
 #define AXI_IDLE_REG                   0x8
-#define NAV_AXI_HALTREQ_BIT            BIT(0)
-#define NAV_AXI_HALTACK_BIT            BIT(1)
-#define NAV_AXI_IDLE_BIT               BIT(2)
 #define AXI_GATING_VALID_OVERRIDE      BIT(0)
 
 #define HALT_ACK_TIMEOUT_US            100000
-#define NAV_HALT_ACK_TIMEOUT_US                200
 
 /* QDSP6SS_RESET */
 #define Q6SS_STOP_CORE                 BIT(0)
@@ -143,7 +139,7 @@ struct rproc_hexagon_res {
        int version;
        bool need_mem_protection;
        bool has_alt_reset;
-       bool has_halt_nav;
+       bool has_spare_reg;
 };
 
 struct q6v5 {
@@ -154,13 +150,11 @@ struct q6v5 {
        void __iomem *rmb_base;
 
        struct regmap *halt_map;
-       struct regmap *halt_nav_map;
        struct regmap *conn_map;
 
        u32 halt_q6;
        u32 halt_modem;
        u32 halt_nc;
-       u32 halt_nav;
        u32 conn_box;
 
        struct reset_control *mss_restart;
@@ -206,7 +200,7 @@ struct q6v5 {
        struct qcom_sysmon *sysmon;
        bool need_mem_protection;
        bool has_alt_reset;
-       bool has_halt_nav;
+       bool has_spare_reg;
        int mpss_perm;
        int mba_perm;
        const char *hexagon_mdt_image;
@@ -427,21 +421,19 @@ static int q6v5_reset_assert(struct q6v5 *qproc)
                reset_control_assert(qproc->pdc_reset);
                ret = reset_control_reset(qproc->mss_restart);
                reset_control_deassert(qproc->pdc_reset);
-       } else if (qproc->has_halt_nav) {
+       } else if (qproc->has_spare_reg) {
                /*
                 * When the AXI pipeline is being reset with the Q6 modem partly
                 * operational there is possibility of AXI valid signal to
                 * glitch, leading to spurious transactions and Q6 hangs. A work
                 * around is employed by asserting the AXI_GATING_VALID_OVERRIDE
-                * BIT before triggering Q6 MSS reset. Both the HALTREQ and
-                * AXI_GATING_VALID_OVERRIDE are withdrawn post MSS assert
-                * followed by a MSS deassert, while holding the PDC reset.
+                * BIT before triggering Q6 MSS reset. AXI_GATING_VALID_OVERRIDE
+                * is withdrawn post MSS assert followed by a MSS deassert,
+                * while holding the PDC reset.
                 */
                reset_control_assert(qproc->pdc_reset);
                regmap_update_bits(qproc->conn_map, qproc->conn_box,
                                   AXI_GATING_VALID_OVERRIDE, 1);
-               regmap_update_bits(qproc->halt_nav_map, qproc->halt_nav,
-                                  NAV_AXI_HALTREQ_BIT, 0);
                reset_control_assert(qproc->mss_restart);
                reset_control_deassert(qproc->pdc_reset);
                regmap_update_bits(qproc->conn_map, qproc->conn_box,
@@ -464,7 +456,7 @@ static int q6v5_reset_deassert(struct q6v5 *qproc)
                ret = reset_control_reset(qproc->mss_restart);
                writel(0, qproc->rmb_base + RMB_MBA_ALT_RESET);
                reset_control_deassert(qproc->pdc_reset);
-       } else if (qproc->has_halt_nav) {
+       } else if (qproc->has_spare_reg) {
                ret = reset_control_reset(qproc->mss_restart);
        } else {
                ret = reset_control_deassert(qproc->mss_restart);
@@ -761,32 +753,6 @@ static void q6v5proc_halt_axi_port(struct q6v5 *qproc,
        regmap_write(halt_map, offset + AXI_HALTREQ_REG, 0);
 }
 
-static void q6v5proc_halt_nav_axi_port(struct q6v5 *qproc,
-                                      struct regmap *halt_map,
-                                      u32 offset)
-{
-       unsigned int val;
-       int ret;
-
-       /* Check if we're already idle */
-       ret = regmap_read(halt_map, offset, &val);
-       if (!ret && (val & NAV_AXI_IDLE_BIT))
-               return;
-
-       /* Assert halt request */
-       regmap_update_bits(halt_map, offset, NAV_AXI_HALTREQ_BIT,
-                          NAV_AXI_HALTREQ_BIT);
-
-       /* Wait for halt ack*/
-       regmap_read_poll_timeout(halt_map, offset, val,
-                                (val & NAV_AXI_HALTACK_BIT),
-                                5, NAV_HALT_ACK_TIMEOUT_US);
-
-       ret = regmap_read(halt_map, offset, &val);
-       if (ret || !(val & NAV_AXI_IDLE_BIT))
-               dev_err(qproc->dev, "port failed halt\n");
-}
-
 static int q6v5_mpss_init_image(struct q6v5 *qproc, const struct firmware *fw)
 {
        unsigned long dma_attrs = DMA_ATTR_FORCE_CONTIGUOUS;
@@ -951,9 +917,6 @@ static int q6v5_mba_load(struct q6v5 *qproc)
 halt_axi_ports:
        q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_q6);
        q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_modem);
-       if (qproc->has_halt_nav)
-               q6v5proc_halt_nav_axi_port(qproc, qproc->halt_nav_map,
-                                          qproc->halt_nav);
        q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_nc);
 
 reclaim_mba:
@@ -1001,9 +964,6 @@ static void q6v5_mba_reclaim(struct q6v5 *qproc)
 
        q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_q6);
        q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_modem);
-       if (qproc->has_halt_nav)
-               q6v5proc_halt_nav_axi_port(qproc, qproc->halt_nav_map,
-                                          qproc->halt_nav);
        q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_nc);
        if (qproc->version == MSS_MSM8996) {
                /*
@@ -1434,36 +1394,12 @@ static int q6v5_init_mem(struct q6v5 *qproc, struct platform_device *pdev)
        qproc->halt_modem = args.args[1];
        qproc->halt_nc = args.args[2];
 
-       if (qproc->has_halt_nav) {
-               struct platform_device *nav_pdev;
-
+       if (qproc->has_spare_reg) {
                ret = of_parse_phandle_with_fixed_args(pdev->dev.of_node,
-                                                      "qcom,halt-nav-regs",
+                                                      "qcom,spare-regs",
                                                       1, 0, &args);
                if (ret < 0) {
-                       dev_err(&pdev->dev, "failed to parse halt-nav-regs\n");
-                       return -EINVAL;
-               }
-
-               nav_pdev = of_find_device_by_node(args.np);
-               of_node_put(args.np);
-               if (!nav_pdev) {
-                       dev_err(&pdev->dev, "failed to get mss clock device\n");
-                       return -EPROBE_DEFER;
-               }
-
-               qproc->halt_nav_map = dev_get_regmap(&nav_pdev->dev, NULL);
-               if (!qproc->halt_nav_map) {
-                       dev_err(&pdev->dev, "failed to get map from device\n");
-                       return -EINVAL;
-               }
-               qproc->halt_nav = args.args[0];
-
-               ret = of_parse_phandle_with_fixed_args(pdev->dev.of_node,
-                                                      "qcom,halt-nav-regs",
-                                                      1, 1, &args);
-               if (ret < 0) {
-                       dev_err(&pdev->dev, "failed to parse halt-nav-regs\n");
+                       dev_err(&pdev->dev, "failed to parse spare-regs\n");
                        return -EINVAL;
                }
 
@@ -1549,7 +1485,7 @@ static int q6v5_init_reset(struct q6v5 *qproc)
                return PTR_ERR(qproc->mss_restart);
        }
 
-       if (qproc->has_alt_reset || qproc->has_halt_nav) {
+       if (qproc->has_alt_reset || qproc->has_spare_reg) {
                qproc->pdc_reset = devm_reset_control_get_exclusive(qproc->dev,
                                                                    "pdc_reset");
                if (IS_ERR(qproc->pdc_reset)) {
@@ -1697,7 +1633,7 @@ static int q6v5_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, qproc);
 
-       qproc->has_halt_nav = desc->has_halt_nav;
+       qproc->has_spare_reg = desc->has_spare_reg;
        ret = q6v5_init_mem(qproc, pdev);
        if (ret)
                goto free_rproc;
@@ -1839,8 +1775,6 @@ static const struct rproc_hexagon_res sc7180_mss = {
        .active_clk_names = (char*[]){
                "mnoc_axi",
                "nav",
-               "mss_nav",
-               "mss_crypto",
                NULL
        },
        .active_pd_names = (char*[]){
@@ -1855,7 +1789,7 @@ static const struct rproc_hexagon_res sc7180_mss = {
        },
        .need_mem_protection = true,
        .has_alt_reset = false,
-       .has_halt_nav = true,
+       .has_spare_reg = true,
        .version = MSS_SC7180,
 };
 
@@ -1890,7 +1824,7 @@ static const struct rproc_hexagon_res sdm845_mss = {
        },
        .need_mem_protection = true,
        .has_alt_reset = true,
-       .has_halt_nav = false,
+       .has_spare_reg = false,
        .version = MSS_SDM845,
 };
 
@@ -1917,7 +1851,7 @@ static const struct rproc_hexagon_res msm8998_mss = {
        },
        .need_mem_protection = true,
        .has_alt_reset = false,
-       .has_halt_nav = false,
+       .has_spare_reg = false,
        .version = MSS_MSM8998,
 };
 
@@ -1947,7 +1881,7 @@ static const struct rproc_hexagon_res msm8996_mss = {
        },
        .need_mem_protection = true,
        .has_alt_reset = false,
-       .has_halt_nav = false,
+       .has_spare_reg = false,
        .version = MSS_MSM8996,
 };
 
@@ -1980,7 +1914,7 @@ static const struct rproc_hexagon_res msm8916_mss = {
        },
        .need_mem_protection = false,
        .has_alt_reset = false,
-       .has_halt_nav = false,
+       .has_spare_reg = false,
        .version = MSS_MSM8916,
 };
 
@@ -2021,7 +1955,7 @@ static const struct rproc_hexagon_res msm8974_mss = {
        },
        .need_mem_protection = false,
        .has_alt_reset = false,
-       .has_halt_nav = false,
+       .has_spare_reg = false,
        .version = MSS_MSM8974,
 };