Commit e1827a30 authored by Rex Zhu's avatar Rex Zhu Committed by Alex Deucher

drm/amd/powerplay: delete flag PP_VALID

don't need to check pp_valid, all pp
export functions are moved to ip_funcs
and pp_funcs. so just need to check the
function point.
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarRex Zhu <Rex.Zhu@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3811f8f0
...@@ -37,7 +37,7 @@ static int pp_dpm_dispatch_tasks(void *handle, enum amd_pp_task task_id, ...@@ -37,7 +37,7 @@ static int pp_dpm_dispatch_tasks(void *handle, enum amd_pp_task task_id,
static inline int pp_check(struct pp_instance *handle) static inline int pp_check(struct pp_instance *handle)
{ {
if (handle == NULL || handle->pp_valid != PP_VALID) if (handle == NULL)
return -EINVAL; return -EINVAL;
if (handle->hwmgr == NULL || handle->hwmgr->smumgr_funcs == NULL) if (handle->hwmgr == NULL || handle->hwmgr->smumgr_funcs == NULL)
...@@ -64,7 +64,6 @@ static int amd_powerplay_create(struct amd_pp_init *pp_init, ...@@ -64,7 +64,6 @@ static int amd_powerplay_create(struct amd_pp_init *pp_init,
if (instance == NULL) if (instance == NULL)
return -ENOMEM; return -ENOMEM;
instance->pp_valid = PP_VALID;
instance->chip_family = pp_init->chip_family; instance->chip_family = pp_init->chip_family;
instance->chip_id = pp_init->chip_id; instance->chip_id = pp_init->chip_id;
instance->pm_en = pp_init->pm_en; instance->pm_en = pp_init->pm_en;
......
...@@ -25,10 +25,7 @@ ...@@ -25,10 +25,7 @@
#include "hwmgr.h" #include "hwmgr.h"
#define PP_VALID 0x1F1F1F1F
struct pp_instance { struct pp_instance {
uint32_t pp_valid;
uint32_t chip_family; uint32_t chip_family;
uint32_t chip_id; uint32_t chip_id;
bool pm_en; bool pm_en;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment