Commit aad4e2db authored by Evan Quan's avatar Evan Quan Committed by Alex Deucher

drm/amd/powerplay: always refetch the enabled features status on dpm enablement

Otherwise, the cached dpm features status may be inconsistent under some
case(e.g. baco reset of Navi asic).
Signed-off-by: default avatarEvan Quan <evan.quan@amd.com>
Acked-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent c6f8c440
...@@ -898,6 +898,9 @@ int smu_v11_0_system_features_control(struct smu_context *smu, ...@@ -898,6 +898,9 @@ int smu_v11_0_system_features_control(struct smu_context *smu,
if (ret) if (ret)
return ret; return ret;
bitmap_zero(feature->enabled, feature->feature_num);
bitmap_zero(feature->supported, feature->feature_num);
if (en) { if (en) {
ret = smu_feature_get_enabled_mask(smu, feature_mask, 2); ret = smu_feature_get_enabled_mask(smu, feature_mask, 2);
if (ret) if (ret)
...@@ -907,9 +910,6 @@ int smu_v11_0_system_features_control(struct smu_context *smu, ...@@ -907,9 +910,6 @@ int smu_v11_0_system_features_control(struct smu_context *smu,
feature->feature_num); feature->feature_num);
bitmap_copy(feature->supported, (unsigned long *)&feature_mask, bitmap_copy(feature->supported, (unsigned long *)&feature_mask,
feature->feature_num); feature->feature_num);
} else {
bitmap_zero(feature->enabled, feature->feature_num);
bitmap_zero(feature->supported, feature->feature_num);
} }
return ret; return ret;
......
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