drm/amdgpu: Avoid fetching VRAM vendor info

Lijo Lazar · alexdeucher · commit 3d1554d99983 · 2024-02-07T12:26:23.000-05:00
The present way to fetch VRAM vendor information turns out to be not
reliable on GFX 9.4.3 dGPUs as well. Avoid using the data.

Signed-off-by: Lijo Lazar &lt;lijo.lazar@amd.com&gt;
Reviewed-by: Hawking Zhang &lt;Hawking.Zhang@amd.com&gt;
Signed-off-by: Alex Deucher &lt;alexander.deucher@amd.com&gt;
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -1949,14 +1949,6 @@ static int gmc_v9_0_init_mem_ranges(struct amdgpu_device *adev)
 
 static void gmc_v9_4_3_init_vram_info(struct amdgpu_device *adev)
 {
-	static const u32 regBIF_BIOS_SCRATCH_4 = 0x50;
-	u32 vram_info;
-
-	/* Only for dGPU, vendor informaton is reliable */
-	if (!amdgpu_sriov_vf(adev) && !(adev->flags & AMD_IS_APU)) {
-		vram_info = RREG32(regBIF_BIOS_SCRATCH_4);
-		adev->gmc.vram_vendor = vram_info & 0xF;
-	}
 	adev->gmc.vram_type = AMDGPU_VRAM_TYPE_HBM;
 	adev->gmc.vram_width = 128 * 64;
 }

Original file line number	Diff line number	Diff line change
`@@ -1949,14 +1949,6 @@ static int gmc_v9_0_init_mem_ranges(struct amdgpu_device *adev)`
`1949`	`1949`
`1950`	`1950`	`static void gmc_v9_4_3_init_vram_info(struct amdgpu_device *adev)`
`1951`	`1951`	`{`
`1952`		`- static const u32 regBIF_BIOS_SCRATCH_4 = 0x50;`
`1953`		`- u32 vram_info;`
`1954`		`-`
`1955`		`- /* Only for dGPU, vendor informaton is reliable */`
`1956`		`- if (!amdgpu_sriov_vf(adev) && !(adev->flags & AMD_IS_APU)) {`
`1957`		`- vram_info = RREG32(regBIF_BIOS_SCRATCH_4);`
`1958`		`- adev->gmc.vram_vendor = vram_info & 0xF;`
`1959`		`- }`
`1960`	`1952`	`adev->gmc.vram_type = AMDGPU_VRAM_TYPE_HBM;`
`1961`	`1953`	`adev->gmc.vram_width = 128 * 64;`
`1962`	`1954`	`}`