Signed-off-by: Huang Rui <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c |  61 ++++++++++---------
 drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c  | 101 ++++++++++++++-----------------
 2 files changed, 81 insertions(+), 81 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
index b21607c..523a769 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
@@ -66,30 +66,52 @@ static void gfxhub_v1_0_init_gart_aperture_regs(struct 
amdgpu_device *adev)
                (u32)(adev->mc.gtt_end >> 44));
 }
 
-int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
+static void gfxhub_v1_0_init_system_aperture_regs(struct amdgpu_device *adev)
 {
-       u32 tmp;
-       u64 value;
-       u32 i;
+       uint64_t value;
+       uint32_t tmp;
 
-       /* Program MC. */
-       gfxhub_v1_0_init_pt_regs(adev);
-       gfxhub_v1_0_init_gart_aperture_regs(adev);
+       /* 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);
 
+       /* Program the system aperture low logical page number. */
        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);
 
+       /* Set default page address. */
        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));
+              (u32)(value >> 12));
        WREG32(SOC15_REG_OFFSET(GC, 0,
                                mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_MSB),
-                               (u32)(value >> 44));
+              (u32)(value >> 44));
 
+       /* Program "protection fault". */
+       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));
+       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);
+}
+
+int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
+{
+       u32 tmp;
+       u32 i;
+
+       DRM_INFO("%s -- in\n", __func__);
        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 */
@@ -99,12 +121,10 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
                                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);
-
        /* GART Enable. */
+       gfxhub_v1_0_init_pt_regs(adev);
+       gfxhub_v1_0_init_gart_aperture_regs(adev);
+       gfxhub_v1_0_init_system_aperture_regs(adev);
 
        /* Setup TLB control */
        tmp = RREG32(SOC15_REG_OFFSET(GC, 0, mmMC_VM_MX_L1_TLB_CNTL));
@@ -176,19 +196,6 @@ int gfxhub_v1_0_gart_enable(struct amdgpu_device *adev)
                            0);
        WREG32(SOC15_REG_OFFSET(GC, 0, mmVM_L2_CNTL4), tmp);
 
-       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));
-       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);
-
        tmp = RREG32(SOC15_REG_OFFSET(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);
diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c 
b/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c
index 84cdca2..df49f32 100644
--- a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c
@@ -79,32 +79,53 @@ static void mmhub_v1_0_init_gart_aperture_regs(struct 
amdgpu_device *adev)
                (u32)(adev->mc.gtt_end >> 44));
 }
 
-int mmhub_v1_0_gart_enable(struct amdgpu_device *adev)
+static void mmhub_v1_0_init_system_aperture_regs(struct amdgpu_device *adev)
 {
-       u32 tmp;
-       u64 value;
-       uint64_t addr;
-       u32 i;
+       uint64_t value;
+       uint32_t tmp;
 
-       /* Program MC. */
-       mmhub_v1_0_init_pt_regs(adev);
-       mmhub_v1_0_init_gart_aperture_regs(adev);
+       /* Disable AGP. */
+       WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_AGP_BASE), 0);
+       WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_AGP_TOP), 0);
+       WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_AGP_BOT), 0x00FFFFFF);
 
-       /* Update configuration */
-       DRM_INFO("%s -- in\n", __func__);
+       /* Program the system aperture low logical page number. */
        WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_SYSTEM_APERTURE_LOW_ADDR),
                adev->mc.vram_start >> 18);
        WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR),
                adev->mc.vram_end >> 18);
+
+       /* Set default page address. */
        value = adev->vram_scratch.gpu_addr - adev->mc.vram_start +
                adev->vm_manager.vram_base_offset;
        WREG32(SOC15_REG_OFFSET(MMHUB, 0,
                                mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_LSB),
-                               (u32)(value >> 12));
+              (u32)(value >> 12));
        WREG32(SOC15_REG_OFFSET(MMHUB, 0,
                                mmMC_VM_SYSTEM_APERTURE_DEFAULT_ADDR_MSB),
-                               (u32)(value >> 44));
+              (u32)(value >> 44));
 
+       /* Program "protection fault". */
+       WREG32(SOC15_REG_OFFSET(MMHUB, 0,
+                               mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_LO32),
+              (u32)(adev->dummy_page.addr >> 12));
+       WREG32(SOC15_REG_OFFSET(MMHUB, 0,
+                               mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_HI32),
+              (u32)((u64)adev->dummy_page.addr >> 44));
+
+       tmp = RREG32(SOC15_REG_OFFSET(MMHUB, 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(MMHUB, 0, mmVM_L2_PROTECTION_FAULT_CNTL2), tmp);
+}
+
+int mmhub_v1_0_gart_enable(struct amdgpu_device *adev)
+{
+       u32 tmp;
+       uint64_t addr;
+       u32 i;
+
+       DRM_INFO("%s -- in\n", __func__);
        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 */
@@ -114,40 +135,25 @@ int mmhub_v1_0_gart_enable(struct amdgpu_device *adev)
                        adev->mc.vram_end >> 24);
        }
 
-       /* Disable AGP. */
-       WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_AGP_BASE), 0);
-       WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_AGP_TOP), 0);
-       WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_AGP_BOT), 0x00FFFFFF);
-
        /* GART Enable. */
+       mmhub_v1_0_init_pt_regs(adev);
+       mmhub_v1_0_init_gart_aperture_regs(adev);
+       mmhub_v1_0_init_system_aperture_regs(adev);
 
        /* Setup TLB control */
        tmp = RREG32(SOC15_REG_OFFSET(MMHUB, 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,
-                               SYSTEM_ACCESS_MODE,
-                               3);
-       tmp = REG_SET_FIELD(tmp,
-                               MC_VM_MX_L1_TLB_CNTL,
-                               ENABLE_ADVANCED_DRIVER_MODEL,
-                               1);
-       tmp = REG_SET_FIELD(tmp,
-                               MC_VM_MX_L1_TLB_CNTL,
-                               SYSTEM_APERTURE_UNMAPPED_ACCESS,
-                               0);
-       tmp = REG_SET_FIELD(tmp,
-                               MC_VM_MX_L1_TLB_CNTL,
-                               ECO_BITS,
-                               0);
-       tmp = REG_SET_FIELD(tmp,
-                               MC_VM_MX_L1_TLB_CNTL,
-                               MTYPE,
-                               MTYPE_UC);/* XXX for emulation. */
-       tmp = REG_SET_FIELD(tmp,
-                               MC_VM_MX_L1_TLB_CNTL,
-                               ATC_EN,
-                               1);
+       tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL, SYSTEM_ACCESS_MODE, 3);
+       tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL,
+                           ENABLE_ADVANCED_DRIVER_MODEL, 1);
+       tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL,
+                           SYSTEM_APERTURE_UNMAPPED_ACCESS, 0);
+       tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL, ECO_BITS, 0);
+       tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL,
+                           MTYPE, MTYPE_UC);/* XXX for emulation. */
+       tmp = REG_SET_FIELD(tmp, MC_VM_MX_L1_TLB_CNTL, ATC_EN, 1);
+
        WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmMC_VM_MX_L1_TLB_CNTL), tmp);
 
        /* Setup L2 cache */
@@ -191,19 +197,6 @@ int mmhub_v1_0_gart_enable(struct amdgpu_device *adev)
                            0);
        WREG32(SOC15_REG_OFFSET(MMHUB, 0, mmVM_L2_CNTL4), tmp);
 
-       WREG32(SOC15_REG_OFFSET(MMHUB, 0,
-                               mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_LO32),
-               (u32)(adev->dummy_page.addr >> 12));
-       WREG32(SOC15_REG_OFFSET(MMHUB, 0,
-                               mmVM_L2_PROTECTION_FAULT_DEFAULT_ADDR_HI32),
-               (u32)((u64)adev->dummy_page.addr >> 44));
-
-       tmp = RREG32(SOC15_REG_OFFSET(MMHUB, 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(MMHUB, 0, mmVM_L2_PROTECTION_FAULT_CNTL2), tmp);
-
        addr = SOC15_REG_OFFSET(MMHUB, 0, mmVM_CONTEXT0_CNTL);
        tmp = RREG32(addr);
 
-- 
2.7.4

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

Reply via email to