drm/amd/pp: Clean up header file include
authorRex Zhu <Rex.Zhu@amd.com>
Wed, 7 Mar 2018 08:40:21 +0000 (16:40 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 15 Mar 2018 14:55:56 +0000 (09:55 -0500)
smu7_smumgr.h should not be included in rv and vega, The common functions
for all smu7 asics are put in smu_smumgr.c.

Not include cgs interface in smumgr.c. the code used cgs interface has been
deleted.

Not include smu_ucode_xfer_vi.h in rv/vega

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Rex Zhu <Rex.Zhu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/smumgr/rv_smumgr.c
drivers/gpu/drm/amd/powerplay/smumgr/smumgr.c
drivers/gpu/drm/amd/powerplay/smumgr/vega10_smumgr.c

index 2d68fae8f1612f341c3c8396c1d688452a2dccd0..ac3350af93745e6d1cc54de1463562def46be7b4 100644 (file)
@@ -31,8 +31,7 @@
 #include "smu10.h"
 #include "ppatomctrl.h"
 #include "pp_debug.h"
-#include "smu_ucode_xfer_vi.h"
-#include "smu7_smumgr.h"
+
 
 #define VOLTAGE_SCALE 4
 
index 3645127c8ee2c51851b14fa8fa0855a5a9c05371..68d943de1a0223558b4ee19b96859d85852900ca 100644 (file)
@@ -28,7 +28,6 @@
 #include <linux/types.h>
 #include <drm/amdgpu_drm.h>
 #include "smumgr.h"
-#include "cgs_common.h"
 
 MODULE_FIRMWARE("amdgpu/topaz_smc.bin");
 MODULE_FIRMWARE("amdgpu/topaz_k_smc.bin");
index bc467badcba39b8e9a86b1bc64a6fa0b26cbb3a3..e08a6116ac0565d84e415ebc4fe22d5803c0d3aa 100644 (file)
 #include "vega10_smumgr.h"
 #include "vega10_ppsmc.h"
 #include "smu9_driver_if.h"
-
 #include "ppatomctrl.h"
 #include "pp_debug.h"
-#include "smu_ucode_xfer_vi.h"
-#include "smu7_smumgr.h"
+
 
 #define AVFS_EN_MSB            1568
 #define AVFS_EN_LSB            1568
@@ -383,7 +381,7 @@ static int vega10_smu_init(struct pp_hwmgr *hwmgr)
        struct cgs_firmware_info info = {0};
 
        ret = cgs_get_firmware_info(hwmgr->device,
-                                   smu7_convert_fw_type_to_cgs(UCODE_ID_SMU),
+                                   CGS_UCODE_ID_SMU,
                                    &info);
        if (ret || !info.kptr)
                return -EINVAL;