https://bugs.freedesktop.org/show_bug.cgi?id=109135
--- Comment #23 from Alex Deucher alexdeucher@gmail.com --- (In reply to iive from comment #22)
It is code refactoring. It doesn't remove, add or modify any functionality. It just changes how some functions are called. (1 function pointer and switch/case, instead of 3 function pointers.) I honestly could not spot what might be wrong with it.
this changed: @@ -1751,10 +1751,10 @@ void amdgpu_dpm_enable_uvd(struct amdgpu_device *adev, bool enable)
void amdgpu_dpm_enable_vce(struct amdgpu_device *adev, bool enable) { - if (adev->powerplay.pp_funcs->powergate_vce) { + if (adev->powerplay.pp_funcs->set_powergating_by_smu) {
CI asic never had a powergate_vce callback before, so that code was never called for vce previously, at least for the old dpm implementation. For the new on, it actually had a callback for vce powergating, but perhaps there was a bug in that code around the time this code was changed.