Since GCE has been moved to MMINFRA in MT8196, all transactions from
MMINFRA to DRAM will have their addresses adjusted by subtracting a
mminfra_offset.

Therefore, the CMDQ helper driver needs to get the mminfra_offset value
of the SoC from cmdq_mbox_priv of cmdq_pkt and then add it to the DRAM
address when generating instructions to ensure GCE accesses the correct
DRAM address. CMDQ users can then call CMDQ helper APIs as usual.

Signed-off-by: Jason-JH Lin <jason-jh....@mediatek.com>
---
 drivers/soc/mediatek/mtk-cmdq-helper.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/soc/mediatek/mtk-cmdq-helper.c 
b/drivers/soc/mediatek/mtk-cmdq-helper.c
index fd978fbd684b..0a718ffa5f3b 100644
--- a/drivers/soc/mediatek/mtk-cmdq-helper.c
+++ b/drivers/soc/mediatek/mtk-cmdq-helper.c
@@ -358,6 +358,7 @@ int cmdq_pkt_mem_move(struct cmdq_pkt *pkt, dma_addr_t 
src_addr, dma_addr_t dst_
        int ret;
 
        /* read the value of src_addr into high_addr_reg_idx */
+       src_addr += pkt->priv.mminfra_offset;
        ret = cmdq_pkt_assign(pkt, high_addr_reg_idx, CMDQ_ADDR_HIGH(src_addr));
        if (ret < 0)
                return ret;
@@ -366,6 +367,7 @@ int cmdq_pkt_mem_move(struct cmdq_pkt *pkt, dma_addr_t 
src_addr, dma_addr_t dst_
                return ret;
 
        /* write the value of value_reg_idx into dst_addr */
+       dst_addr += pkt->priv.mminfra_offset;
        ret = cmdq_pkt_assign(pkt, high_addr_reg_idx, CMDQ_ADDR_HIGH(dst_addr));
        if (ret < 0)
                return ret;
@@ -491,7 +493,7 @@ int cmdq_pkt_poll_addr(struct cmdq_pkt *pkt, dma_addr_t 
addr, u32 value, u32 mas
        inst.op = CMDQ_CODE_MASK;
        inst.dst_t = CMDQ_REG_TYPE;
        inst.sop = CMDQ_POLL_ADDR_GPR;
-       inst.value = addr;
+       inst.value = addr + pkt->priv.mminfra_offset;
        ret = cmdq_pkt_append_command(pkt, inst);
        if (ret < 0)
                return ret;
@@ -551,7 +553,7 @@ int cmdq_pkt_jump_abs(struct cmdq_pkt *pkt, dma_addr_t addr)
        struct cmdq_instruction inst = {
                .op = CMDQ_CODE_JUMP,
                .offset = CMDQ_JUMP_ABSOLUTE,
-               .value = addr >> pkt->priv.shift_pa
+               .value = (addr +  pkt->priv.mminfra_offset) >> 
pkt->priv.shift_pa
        };
        return cmdq_pkt_append_command(pkt, inst);
 }
-- 
2.43.0

Reply via email to