Signed-off-by: Tom St Denis <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/psp_v3_1.c | 34 +++++++++++++++++-----------------
 1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
index d351583785e5..34dbb60da752 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
@@ -172,7 +172,7 @@ int psp_v3_1_bootloader_load_sysdrv(struct psp_context *psp)
        /* Check sOS sign of life register to confirm sys driver and sOS
         * are already been loaded.
         */
-       sol_reg = RREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_81));
+       sol_reg = RREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_81);
        if (sol_reg)
                return 0;
 
@@ -188,11 +188,11 @@ int psp_v3_1_bootloader_load_sysdrv(struct psp_context 
*psp)
        memcpy(psp->fw_pri_buf, psp->sys_start_addr, psp->sys_bin_size);
 
        /* Provide the sys driver to bootrom */
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_36),
-              (uint32_t)(psp->fw_pri_mc_addr >> 20));
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_36,
+                    (uint32_t)(psp->fw_pri_mc_addr >> 20));
        psp_gfxdrv_command_reg = 1 << 16;
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_35),
-              psp_gfxdrv_command_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_35,
+                    psp_gfxdrv_command_reg);
 
        /* there might be handshake issue with hardware which needs delay */
        mdelay(20);
@@ -213,7 +213,7 @@ int psp_v3_1_bootloader_load_sos(struct psp_context *psp)
        /* Check sOS sign of life register to confirm sys driver and sOS
         * are already been loaded.
         */
-       sol_reg = RREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_81));
+       sol_reg = RREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_81);
        if (sol_reg)
                return 0;
 
@@ -229,17 +229,17 @@ int psp_v3_1_bootloader_load_sos(struct psp_context *psp)
        memcpy(psp->fw_pri_buf, psp->sos_start_addr, psp->sos_bin_size);
 
        /* Provide the PSP secure OS to bootrom */
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_36),
-              (uint32_t)(psp->fw_pri_mc_addr >> 20));
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_36,
+                    (uint32_t)(psp->fw_pri_mc_addr >> 20));
        psp_gfxdrv_command_reg = 2 << 16;
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_35),
-              psp_gfxdrv_command_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_35,
+                    psp_gfxdrv_command_reg);
 
        /* there might be handshake issue with hardware which needs delay */
        mdelay(20);
 #if 0
        ret = psp_wait_for(psp, SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_81),
-                          RREG32(SOC15_REG_OFFSET(MP0, 0, 
mmMP0_SMN_C2PMSG_81)),
+                          RREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_81),
                           0, true);
 #endif
 
@@ -299,17 +299,17 @@ int psp_v3_1_ring_create(struct psp_context *psp, enum 
psp_ring_type ring_type)
 
        /* Write low address of the ring to C2PMSG_69 */
        psp_ring_reg = lower_32_bits(ring->ring_mem_mc_addr);
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_69), psp_ring_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_69, psp_ring_reg);
        /* Write high address of the ring to C2PMSG_70 */
        psp_ring_reg = upper_32_bits(ring->ring_mem_mc_addr);
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_70), psp_ring_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_70, psp_ring_reg);
        /* Write size of ring to C2PMSG_71 */
        psp_ring_reg = ring->ring_size;
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_71), psp_ring_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_71, psp_ring_reg);
        /* Write the ring initialization command to C2PMSG_64 */
        psp_ring_reg = ring_type;
        psp_ring_reg = psp_ring_reg << 16;
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_64), psp_ring_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_64, psp_ring_reg);
 
        /* there might be handshake issue with hardware which needs delay */
        mdelay(20);
@@ -334,7 +334,7 @@ int psp_v3_1_cmd_submit(struct psp_context *psp,
        uint32_t rb_frame_size_dw = sizeof(struct psp_gfx_rb_frame) / 4;
 
        /* KM (GPCOM) prepare write pointer */
-       psp_write_ptr_reg = RREG32(SOC15_REG_OFFSET(MP0, 0, 
mmMP0_SMN_C2PMSG_67));
+       psp_write_ptr_reg = RREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_67);
 
        /* Update KM RB frame pointer to new frame */
        /* write_frame ptr increments by size of rb_frame in bytes */
@@ -356,7 +356,7 @@ int psp_v3_1_cmd_submit(struct psp_context *psp,
 
        /* Update the write Pointer in DWORDs */
        psp_write_ptr_reg = (psp_write_ptr_reg + rb_frame_size_dw) % 
ring_size_dw;
-       WREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_67), 
psp_write_ptr_reg);
+       WREG32_SOC15(MP0, 0,  mmMP0_SMN_C2PMSG_67, psp_write_ptr_reg);
 
        return 0;
 }
-- 
2.12.0

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to