1. skip to load smu firmware in sriov mode for aldebaran chip
2. using vbios pptable if in sriov mode.

Signed-off-by: Kevin Wang <kevin1.w...@amd.com>
---
 .../gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c    | 66 ++++++++++---------
 1 file changed, 36 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
index a421ba85bd6d..a0ca7e7a0903 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
@@ -85,6 +85,10 @@ int smu_v13_0_init_microcode(struct smu_context *smu)
        const struct common_firmware_header *header;
        struct amdgpu_firmware_info *ucode = NULL;
 
+       /* doesn't need to load smu firmware in IOV mode */
+       if (amdgpu_sriov_vf(adev))
+               return 0;
+
        switch (adev->asic_type) {
        case CHIP_ALDEBARAN:
                chip_name = "aldebaran";
@@ -279,41 +283,43 @@ int smu_v13_0_setup_pptable(struct smu_context *smu)
        void *table;
        uint16_t version_major, version_minor;
 
+       if (!amdgpu_sriov_vf(adev)) {
+               if (amdgpu_smu_pptable_id >= 0) {
+                       smu->smu_table.boot_values.pp_table_id = 
amdgpu_smu_pptable_id;
+                       dev_info(adev->dev, "override pptable id %d\n", 
amdgpu_smu_pptable_id);
+               }
 
-       if (amdgpu_smu_pptable_id >= 0) {
-               smu->smu_table.boot_values.pp_table_id = amdgpu_smu_pptable_id;
-               dev_info(adev->dev, "override pptable id %d\n", 
amdgpu_smu_pptable_id);
-       }
-
-       hdr = (const struct smc_firmware_header_v1_0 *) adev->pm.fw->data;
-       version_major = le16_to_cpu(hdr->header.header_version_major);
-       version_minor = le16_to_cpu(hdr->header.header_version_minor);
-       if (version_major == 2 && smu->smu_table.boot_values.pp_table_id > 0) {
-               dev_info(adev->dev, "use driver provided pptable %d\n", 
smu->smu_table.boot_values.pp_table_id);
-               switch (version_minor) {
-               case 1:
-                       ret = smu_v13_0_set_pptable_v2_1(smu, &table, &size,
-                                                        
smu->smu_table.boot_values.pp_table_id);
-                       break;
-               default:
-                       ret = -EINVAL;
-                       break;
+               hdr = (const struct smc_firmware_header_v1_0 *) 
adev->pm.fw->data;
+               version_major = le16_to_cpu(hdr->header.header_version_major);
+               version_minor = le16_to_cpu(hdr->header.header_version_minor);
+               if (version_major == 2 && 
smu->smu_table.boot_values.pp_table_id > 0) {
+                       dev_info(adev->dev, "use driver provided pptable %d\n", 
smu->smu_table.boot_values.pp_table_id);
+                       switch (version_minor) {
+                       case 1:
+                               ret = smu_v13_0_set_pptable_v2_1(smu, &table, 
&size,
+                                                                
smu->smu_table.boot_values.pp_table_id);
+                               break;
+                       default:
+                               ret = -EINVAL;
+                               break;
+                       }
+                       if (ret)
+                               return ret;
+                       goto out;
                }
-               if (ret)
-                       return ret;
+       }
 
-       } else {
-               dev_info(adev->dev, "use vbios provided pptable\n");
-               index = 
get_index_into_master_table(atom_master_list_of_data_tables_v2_1,
-                                                   powerplayinfo);
+       dev_info(adev->dev, "use vbios provided pptable\n");
+       index = 
get_index_into_master_table(atom_master_list_of_data_tables_v2_1,
+                                           powerplayinfo);
 
-               ret = amdgpu_atombios_get_data_table(adev, index, 
&atom_table_size, &frev, &crev,
-                                                    (uint8_t **)&table);
-               if (ret)
-                       return ret;
-               size = atom_table_size;
-       }
+       ret = amdgpu_atombios_get_data_table(adev, index, &atom_table_size, 
&frev, &crev,
+                                            (uint8_t **)&table);
+       if (ret)
+               return ret;
 
+       size = atom_table_size;
+out:
        if (!smu->smu_table.power_play_table)
                smu->smu_table.power_play_table = table;
        if (!smu->smu_table.power_play_table_size)
-- 
2.25.1

Reply via email to