I only skimmed over it, but what I saw looked mostly good to me. Patch #7: Please add a code comment explaining why we do this. Apart from that the rest is Acked-by: Christian König <christian.koenig at amd.com>. Regards, Christian. Am 26.01.2017 um 12:00 schrieb Rex Zhu: > Rex Zhu (12): > drm/amdgpu: when dpm disabled, also can enable uvd cg/pg. > drm/amdgpu: add current_pg_status register define for smu7.1 > drm/amdgpu: power down/up uvd4 when smu disabled. > drm/amdgpu: refine uvd pg code in kv_dpm.c > drm/amdgpu: refine uvd4.2 init/stop code. > drm/amdgpu: refine uvd5.0/6.0 code. > drm/amdgpu: add uvd timeout time. > drm/amdgpu: when dpm disabled, also need to stop/start vce. > drm/amdgpu: refine code for VCE2.0 and related dpm code. > drm/amdgpu: enable vce pg feature on Kv. > drm/amdgpu: move subfunctions to the front of vce_v2_0.c. > drm/amdgpu: refine vce3.0 code and related powerplay pg code. > > drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 12 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 12 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 9 + > drivers/gpu/drm/amd/amdgpu/ci_dpm.c | 9 - > drivers/gpu/drm/amd/amdgpu/cik.c | 4 +- > drivers/gpu/drm/amd/amdgpu/kv_dpm.c | 68 +--- > drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c | 130 ++++-- > drivers/gpu/drm/amd/amdgpu/uvd_v5_0.c | 24 +- > drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 21 +- > drivers/gpu/drm/amd/amdgpu/vce_v2_0.c | 451 +++++++++++---------- > drivers/gpu/drm/amd/amdgpu/vce_v3_0.c | 17 +- > .../gpu/drm/amd/include/asic_reg/smu/smu_7_0_1_d.h | 1 + > .../drm/amd/powerplay/hwmgr/cz_clockpowergating.c | 71 ++-- > .../amd/powerplay/hwmgr/smu7_clockpowergating.c | 23 +- > 14 files changed, 438 insertions(+), 414 deletions(-) >
