drm/amdgpu: correct ih_chicken programming for vega10/vega20 ih blocks
authorHawking Zhang <Hawking.Zhang@amd.com>
Mon, 30 Nov 2020 16:04:36 +0000 (00:04 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 23 Dec 2020 20:06:01 +0000 (15:06 -0500)
IH_CHICKEN.MC_SPACE_FBPA_ENABLE field is only
valid when IH_RB_CNTL.MC_SPACE is programed to 0x3,
frame buffer physical address. For both bus address
and gpu virtual address, don't program MC_SPACE_FBPA_ENABLE
field

Signed-off-by: Hawking Zhang <Hawking.Zhang@amd.com>
Acked-by: Christian König <christian.koenig@amd.com>
Acked-by: Felix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: Dennis Li <Dennis.Li@amd.com>
Reviewed-by: Feifei Xu <Feifei.Xu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/vega10_ih.c
drivers/gpu/drm/amd/amdgpu/vega20_ih.c

index 6694df7..17c0a07 100644 (file)
@@ -281,9 +281,6 @@ static int vega10_ih_irq_init(struct amdgpu_device *adev)
                if (adev->irq.ih.use_bus_addr) {
                        ih_chicken = REG_SET_FIELD(ih_chicken, IH_CHICKEN,
                                                   MC_SPACE_GPA_ENABLE, 1);
-               } else {
-                       ih_chicken = REG_SET_FIELD(ih_chicken, IH_CHICKEN,
-                                                  MC_SPACE_FBPA_ENABLE, 1);
                }
                WREG32_SOC15(OSSSYS, 0, mmIH_CHICKEN, ih_chicken);
        }
index 913b763..e381a25 100644 (file)
@@ -312,9 +312,6 @@ static int vega20_ih_irq_init(struct amdgpu_device *adev)
                if (adev->irq.ih.use_bus_addr) {
                        ih_chicken = REG_SET_FIELD(ih_chicken, IH_CHICKEN,
                                                   MC_SPACE_GPA_ENABLE, 1);
-               } else {
-                       ih_chicken = REG_SET_FIELD(ih_chicken, IH_CHICKEN,
-                                                  MC_SPACE_FBPA_ENABLE, 1);
                }
                WREG32_SOC15(OSSSYS, 0, mmIH_CHICKEN, ih_chicken);
        }