No functional changes in this patch.  Simply re-writing
read/writes into new macros.

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

diff --git a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
index 005075ff00f7..187db1fbef18 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
@@ -39,38 +39,38 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
 
        /* Program MC. */
        /* Update configuration */
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_SYSTEM_APERTURE_LOW_ADDR),
-               adev->mc.vram_start >> 18);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR),
-               adev->mc.vram_end >> 18);
+       WREG32_SOC15(GC, 0, mmMC_VM_SYSTEM_APERTURE_LOW_ADDR,
+                    adev->mc.vram_start >> 18);
+       WREG32_SOC15(GC, 0, mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR,
+                    adev->mc.vram_end >> 18);
 
        value = adev->vram_scratch.gpu_addr - adev->mc.vram_start
                + adev->vm_manager.vram_base_offset;
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_LSB),
-                               (u32)(value >> 12));
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_MSB),
-                               (u32)(value >> 44));
+       WREG32_SOC15(GC, 0,
+                    mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_LSB,
+                    (u32)(value >> 12));
+       WREG32_SOC15(GC, 0,
+                    mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_MSB,
+                    (u32)(value >> 44));
 
        if (amdgpu_sriov_vf(adev)) {
                /* MC_VM_FB_LOCATION_BASE/TOP is NULL for VF, becuase they are 
VF copy registers so
                vbios post doesn't program them, for SRIOV driver need to 
program them */
-               WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_FB_LOCATION_BASE),
-                               adev->mc.vram_start >> 24);
-               WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_FB_LOCATION_TOP),
-                               adev->mc.vram_end >> 24);
+               WREG32_SOC15(GC, 0, mmMC_VM_FB_LOCATION_BASE,
+                            adev->mc.vram_start >> 24);
+               WREG32_SOC15(GC, 0, mmMC_VM_FB_LOCATION_TOP,
+                            adev->mc.vram_end >> 24);
        }
 
        /* Disable AGP. */
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_AGP_BASE), 0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_AGP_TOP), 0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_AGP_BOT), 0xFFFFFFFF);
+       WREG32_SOC15(GC, 0, mmMC_VM_AGP_BASE, 0);
+       WREG32_SOC15(GC, 0, mmMC_VM_AGP_TOP, 0);
+       WREG32_SOC15(GC, 0, mmMC_VM_AGP_BOT, 0xFFFFFFFF);
 
        /* GART Enable. */
 
        /* Setup TLB control */
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_MX_L1_TLB_CNTL));
+       tmp = RREG32_SOC15(GC, 0, mmMC_VM_MX_L1_TLB_CNTL);
        tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL, ENABLE_L1_TLB, 1);
        tmp = REG_SET_FIELD(tmp,
                                MC_VM_MX_L1_TLB_CNTL,
@@ -96,10 +96,10 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
                                MC_VM_MX_L1_TLB_CNTL,
                                ATC_EN,
                                1);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_MX_L1_TLB_CNTL), tmp);
+       WREG32_SOC15(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp);
 
        /* Setup L2 cache */
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL));
+       tmp = RREG32_SOC15(GC, 0, mmVM_L2_CNTL);
        tmp = REG_SET_FIELD(tmp, VM_L2_CNTL, ENABLE_L2_CACHE, 1);
        tmp = REG_SET_FIELD(tmp,
                                VM_L2_CNTL,
@@ -118,17 +118,17 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
                                VM_L2_CNTL,
                                IDENTITY_MODE_FRAGMENT_SIZE,
                                0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL), tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_CNTL, tmp);
 
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL2));
+       tmp = RREG32_SOC15(GC, 0, mmVM_L2_CNTL2);
        tmp = REG_SET_FIELD(tmp, VM_L2_CNTL2, INVALIDATE_ALL_L1_TLBS, 1);
        tmp = REG_SET_FIELD(tmp, VM_L2_CNTL2, INVALIDATE_L2_CACHE, 1);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL2), tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_CNTL2, tmp);
 
        tmp = mmVM_L2_CNTL3_DEFAULT;
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL3), tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_CNTL3, tmp);
 
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL4));
+       tmp = RREG32_SOC15(GC, 0, mmVM_L2_CNTL4);
        tmp = REG_SET_FIELD(tmp,
                            VM_L2_CNTL4,
                            VMC_TAP_PDE_REQUEST_PHYSICAL,
@@ -137,22 +137,22 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
                            VM_L2_CNTL4,
                            VMC_TAP_PTE_REQUEST_PHYSICAL,
                            0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL4), tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_CNTL4, tmp);
 
        /* setup context0 */
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_CONTEXT0_PAGE_TABLE_START_ADDR_LO32),
-               (u32)(adev->mc.gtt_start >> 12));
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_CONTEXT0_PAGE_TABLE_START_ADDR_HI32),
-               (u32)(adev->mc.gtt_start >> 44));
-
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_CONTEXT0_PAGE_TABLE_END_ADDR_LO32),
-               (u32)(adev->mc.gtt_end >> 12));
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_CONTEXT0_PAGE_TABLE_END_ADDR_HI32),
-               (u32)(adev->mc.gtt_end >> 44));
+       WREG32_SOC15(GC, 0,
+                    mmVM_CONTEXT0_PAGE_TABLE_START_ADDR_LO32,
+                    (u32)(adev->mc.gtt_start >> 12));
+       WREG32_SOC15(GC, 0,
+                    mmVM_CONTEXT0_PAGE_TABLE_START_ADDR_HI32,
+                    (u32)(adev->mc.gtt_start >> 44));
+
+       WREG32_SOC15(GC, 0,
+                    mmVM_CONTEXT0_PAGE_TABLE_END_ADDR_LO32,
+                    (u32)(adev->mc.gtt_end >> 12));
+       WREG32_SOC15(GC, 0,
+                    mmVM_CONTEXT0_PAGE_TABLE_END_ADDR_HI32,
+                    (u32)(adev->mc.gtt_end >> 44));
 
        BUG_ON(adev->gart.table_addr & (~0x0000FFFFFFFFF000ULL));
        value = adev->gart.table_addr - adev->mc.vram_start
@@ -160,49 +160,49 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
        value &= 0x0000FFFFFFFFF000ULL;
        value |= 0x1; /*valid bit*/
 
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_CONTEXT0_PAGE_TABLE_BASE_ADDR_LO32),
-               (u32)value);
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_CONTEXT0_PAGE_TABLE_BASE_ADDR_HI32),
-               (u32)(value >> 32));
-
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_LO32),
-               (u32)(adev->dummy_page.addr >> 12));
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-                               mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_HI32),
-               (u32)((u64)adev->dummy_page.addr >> 44));
-
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL2));
+       WREG32_SOC15(GC, 0,
+                    mmVM_CONTEXT0_PAGE_TABLE_BASE_ADDR_LO32,
+                    (u32)value);
+       WREG32_SOC15(GC, 0,
+                    mmVM_CONTEXT0_PAGE_TABLE_BASE_ADDR_HI32,
+                    (u32)(value >> 32));
+
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_LO32,
+                    (u32)(adev->dummy_page.addr >> 12));
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_HI32,
+                    (u32)((u64)adev->dummy_page.addr >> 44));
+
+       tmp = RREG32_SOC15(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL2);
        tmp = REG_SET_FIELD(tmp, VM_L2_PROTECTION_FAULT_CNTL2,
                            ACTIVE_PAGE_MIGRATION_PTE_READ_RETRY,
                            1);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL2), tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL2, tmp);
 
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_CONTEXT0_CNTL));
+       tmp = RREG32_SOC15(GC, 0, mmVM_CONTEXT0_CNTL);
        tmp = REG_SET_FIELD(tmp, VM_CONTEXT0_CNTL, ENABLE_CONTEXT, 1);
        tmp = REG_SET_FIELD(tmp, VM_CONTEXT0_CNTL, PAGE_TABLE_DEPTH, 0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_CONTEXT0_CNTL), tmp);
+       WREG32_SOC15(GC, 0, mmVM_CONTEXT0_CNTL, tmp);
 
        /* Disable identity aperture.*/
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-               mmVM_L2_CONTEXT1_IDENTITY_APERTURE_LOW_ADDR_LO32), 0XFFFFFFFF);
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-               mmVM_L2_CONTEXT1_IDENTITY_APERTURE_LOW_ADDR_HI32), 0x0000000F);
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_CONTEXT1_IDENTITY_APERTURE_LOW_ADDR_LO32, 
0XFFFFFFFF);
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_CONTEXT1_IDENTITY_APERTURE_LOW_ADDR_HI32, 
0x0000000F);
 
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-               mmVM_L2_CONTEXT1_IDENTITY_APERTURE_HIGH_ADDR_LO32), 0);
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-               mmVM_L2_CONTEXT1_IDENTITY_APERTURE_HIGH_ADDR_HI32), 0);
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_CONTEXT1_IDENTITY_APERTURE_HIGH_ADDR_LO32, 0);
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_CONTEXT1_IDENTITY_APERTURE_HIGH_ADDR_HI32, 0);
 
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-               mmVM_L2_CONTEXT_IDENTITY_PHYSICAL_OFFSET_LO32), 0);
-       WREG32(SOC15_REG_OFFSET(GC, 0,
-               mmVM_L2_CONTEXT_IDENTITY_PHYSICAL_OFFSET_HI32), 0);
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_CONTEXT_IDENTITY_PHYSICAL_OFFSET_LO32, 0);
+       WREG32_SOC15(GC, 0,
+                    mmVM_L2_CONTEXT_IDENTITY_PHYSICAL_OFFSET_HI32, 0);
 
        for (i = 0; i <= 14; i++) {
-               tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_CONTEXT1_CNTL) + i);
+               tmp = RREG32_SOC15_OFFSET(GC, 0, mmVM_CONTEXT1_CNTL, i);
                tmp = REG_SET_FIELD(tmp, VM_CONTEXT1_CNTL, ENABLE_CONTEXT, 1);
                tmp = REG_SET_FIELD(tmp, VM_CONTEXT1_CNTL, PAGE_TABLE_DEPTH,
                                    adev->vm_manager.num_level);
@@ -223,13 +223,13 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
                tmp = REG_SET_FIELD(tmp, VM_CONTEXT1_CNTL,
                                PAGE_TABLE_BLOCK_SIZE,
                                adev->vm_manager.block_size - 9);
-               WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_CONTEXT1_CNTL) + i, tmp);
-               WREG32(SOC15_REG_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_START_ADDR_LO32) + i*2, 0);
-               WREG32(SOC15_REG_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_START_ADDR_HI32) + i*2, 0);
-               WREG32(SOC15_REG_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_END_ADDR_LO32) + i*2,
-                       lower_32_bits(adev->vm_manager.max_pfn - 1));
-               WREG32(SOC15_REG_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_END_ADDR_HI32) + i*2,
-                       upper_32_bits(adev->vm_manager.max_pfn - 1));
+               WREG32_SOC15_OFFSET(GC, 0, mmVM_CONTEXT1_CNTL, i, tmp);
+               WREG32_SOC15_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_START_ADDR_LO32, i*2, 0);
+               WREG32_SOC15_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_START_ADDR_HI32, i*2, 0);
+               WREG32_SOC15_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_END_ADDR_LO32, i*2,
+                                   lower_32_bits(adev->vm_manager.max_pfn - 
1));
+               WREG32_SOC15_OFFSET(GC, 0, 
mmVM_CONTEXT1_PAGE_TABLE_END_ADDR_HI32, i*2,
+                                   upper_32_bits(adev->vm_manager.max_pfn - 
1));
        }
 
 
@@ -243,22 +243,22 @@ void gfxhub_v1_0_gart_disable(struct amdgpu_device *adev)
 
        /* Disable all tables */
        for (i = 0; i < 16; i++)
-               WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_CONTEXT0_CNTL) + i, 0);
+               WREG32_SOC15_OFFSET(GC, 0, mmVM_CONTEXT0_CNTL, i, 0);
 
        /* Setup TLB control */
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_MX_L1_TLB_CNTL));
+       tmp = RREG32_SOC15(GC, 0, mmMC_VM_MX_L1_TLB_CNTL);
        tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL, ENABLE_L1_TLB, 0);
        tmp = REG_SET_FIELD(tmp,
                                MC_VM_MX_L1_TLB_CNTL,
                                ENABLE_ADVANCED_DRIVER_MODEL,
                                0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_MX_L1_TLB_CNTL), tmp);
+       WREG32_SOC15(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp);
 
        /* Setup L2 cache */
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL));
+       tmp = RREG32_SOC15(GC, 0, mmVM_L2_CNTL);
        tmp = REG_SET_FIELD(tmp, VM_L2_CNTL, ENABLE_L2_CACHE, 0);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL), tmp);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL3), 0);
+       WREG32_SOC15(GC, 0, mmVM_L2_CNTL, tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_CNTL3, 0);
 }
 
 /**
@@ -271,7 +271,7 @@ void gfxhub_v1_0_set_fault_enable_default(struct 
amdgpu_device *adev,
                                          bool value)
 {
        u32 tmp;
-       tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL));
+       tmp = RREG32_SOC15(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL);
        tmp = REG_SET_FIELD(tmp, VM_L2_PROTECTION_FAULT_CNTL,
                        RANGE_PROTECTION_FAULT_ENABLE_DEFAULT, value);
        tmp = REG_SET_FIELD(tmp, VM_L2_PROTECTION_FAULT_CNTL,
@@ -296,7 +296,7 @@ void gfxhub_v1_0_set_fault_enable_default(struct 
amdgpu_device *adev,
                        WRITE_PROTECTION_FAULT_ENABLE_DEFAULT, value);
        tmp = REG_SET_FIELD(tmp, VM_L2_PROTECTION_FAULT_CNTL,
                        EXECUTE_PROTECTION_FAULT_ENABLE_DEFAULT, value);
-       WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL), tmp);
+       WREG32_SOC15(GC, 0, mmVM_L2_PROTECTION_FAULT_CNTL, tmp);
 }
 
 static int gfxhub_v1_0_early_init(void *handle)
@@ -345,12 +345,12 @@ static int gfxhub_v1_0_hw_init(void *handle)
        unsigned i;
 
        for (i = 0 ; i < 18; ++i) {
-               WREG32(SOC15_REG_OFFSET(GC, 0,
-                                       mmVM_INVALIDATE_ENG0_ADDR_RANGE_LO32) +
-                      2 * i, 0xffffffff);
-               WREG32(SOC15_REG_OFFSET(GC, 0,
-                                       mmVM_INVALIDATE_ENG0_ADDR_RANGE_HI32) +
-                      2 * i, 0x1f);
+               WREG32_SOC15_OFFSET(GC, 0,
+                                   mmVM_INVALIDATE_ENG0_ADDR_RANGE_LO32,
+                                   2 * i, 0xffffffff);
+               WREG32_SOC15_OFFSET(GC, 0,
+                                   mmVM_INVALIDATE_ENG0_ADDR_RANGE_HI32,
+                                   2 * i, 0x1f);
        }
 
        return 0;
-- 
2.12.0

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

Reply via email to