drm/amdgpu: fix unknown vram mem type for vega20
authorHawking Zhang <Hawking.Zhang@amd.com>
Mon, 17 Sep 2018 12:19:48 +0000 (20:19 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 19 Sep 2018 17:37:39 +0000 (12:37 -0500)
vega20 should use umc_info v3_3 instead of v3_1. There are
serveral versions of umc_info for vega series. Compared to
various versions of these structures, vram_info strucure is
unified for vega series. The patch switch to query mem_type
from vram_info structure for all the vega series dGPU.

Signed-off-by: Hawking Zhang <Hawking.Zhang@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_atomfirmware.c

index 5461d0d..b61e1dc 100644 (file)
@@ -117,6 +117,10 @@ union igp_info {
 union umc_info {
        struct atom_umc_info_v3_1 v31;
 };
+
+union vram_info {
+       struct atom_vram_info_header_v2_3 v23;
+};
 /*
  * Return vram width from integrated system info table, if available,
  * or 0 if not.
@@ -195,7 +199,7 @@ int amdgpu_atomfirmware_get_vram_type(struct amdgpu_device *adev)
        int index;
        u16 data_offset, size;
        union igp_info *igp_info;
-       union umc_info *umc_info;
+       union vram_info *vram_info;
        u8 frev, crev;
        u8 mem_type;
 
@@ -204,7 +208,7 @@ int amdgpu_atomfirmware_get_vram_type(struct amdgpu_device *adev)
                                                    integratedsysteminfo);
        else
                index = get_index_into_master_table(atom_master_list_of_data_tables_v2_1,
-                                                   umc_info);
+                                                   vram_info);
        if (amdgpu_atom_parse_data_header(mode_info->atom_context,
                                          index, &size,
                                          &frev, &crev, &data_offset)) {
@@ -219,11 +223,11 @@ int amdgpu_atomfirmware_get_vram_type(struct amdgpu_device *adev)
                                return 0;
                        }
                } else {
-                       umc_info = (union umc_info *)
+                       vram_info = (union vram_info *)
                                (mode_info->atom_context->bios + data_offset);
                        switch (crev) {
-                       case 1:
-                               mem_type = umc_info->v31.vram_type;
+                       case 3:
+                               mem_type = vram_info->v23.vram_module[0].memory_type;
                                return convert_atom_mem_type_to_vram_type(adev, mem_type);
                        default:
                                return 0;