Comment # 23
on bug 109135
from Alex Deucher
(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.
You are receiving this mail because:
- You are the assignee for the bug.
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
