<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr"> I just tried it on my Carrizo this morning (after sending out the dce6 patch) and everything seems peachy. Tom <div style="color: rgb(0, 0, 0);"> <div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr">From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Deucher, Alexander <Alexander.Deucher@amd.com> Sent: Monday, November 14, 2016 11:56 To: Zhu, Rex; amd-gfx@lists.freedesktop.org Cc: Zhu, Rex Subject: RE: [PATCH] drm/amdgpu: refine cz uvd clock gate logic. <div> </div> </div> </div> <div class="PlainText">> -----Original Message----- > From: amd-gfx [<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">mailto:amd-gfx-bounces@lists.freedesktop.org</a>] On Behalf > Of Rex Zhu > Sent: Friday, November 11, 2016 12:25 AM > To: amd-gfx@lists.freedesktop.org > Cc: Zhu, Rex > Subject: [PATCH] drm/amdgpu: refine cz uvd clock gate logic. > > sw clockgate was used on uvd6.0. > when uvd is idle, we gate the uvd clock. > when decode, we ungate the uvd clock. > > Change-Id: I79ecdc5d0f48e97919386a08acca994f1fa05484 > Signed-off-by: Rex Zhu <Rex.Zhu@amd.com> Assuming clockgating still works properly: Reviewed-by: Alex Deucher <alexander.deucher@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/cz_dpm.c | 6 ++---- > drivers/gpu/drm/amd/powerplay/hwmgr/cz_clockpowergating.c | 4 ++-- > 2 files changed, 4 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/cz_dpm.c > b/drivers/gpu/drm/amd/amdgpu/cz_dpm.c > index 41fa351..ba2b66b 100644 > --- a/drivers/gpu/drm/amd/amdgpu/cz_dpm.c > +++ b/drivers/gpu/drm/amd/amdgpu/cz_dpm.c > @@ -2111,9 +2111,8 @@ static void cz_dpm_powergate_uvd(struct > amdgpu_device *adev, bool gate) > > if (gate) { > if (pi->caps_uvd_pg) { > - /* disable clockgating so we can properly shut down > the block */ > ret = amdgpu_set_clockgating_state(adev, > AMD_IP_BLOCK_TYPE_UVD, > - > AMD_CG_STATE_UNGATE); > + > AMD_CG_STATE_GATE); > if (ret) { > DRM_ERROR("UVD DPM Power Gating failed > to set clockgating state\n"); > return; > @@ -2159,9 +2158,8 @@ static void cz_dpm_powergate_uvd(struct > amdgpu_device *adev, bool gate) > return; > } > > - /* enable clockgating. hw will dynamically > gate/ungate clocks on the fly */ > ret = amdgpu_set_clockgating_state(adev, > AMD_IP_BLOCK_TYPE_UVD, > - > AMD_CG_STATE_GATE); > + > AMD_CG_STATE_UNGATE); > if (ret) { > DRM_ERROR("UVD DPM Power Gating Failed > to set clockgating state\n"); > return; > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/cz_clockpowergating.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/cz_clockpowergating.c > index 2028980..b0c63c5 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/cz_clockpowergating.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/cz_clockpowergating.c > @@ -169,7 +169,7 @@ int cz_dpm_powergate_uvd(struct pp_hwmgr > *hwmgr, bool bgate) > if (bgate) { > cgs_set_clockgating_state(hwmgr->device, > AMD_IP_BLOCK_TYPE_UVD, > - AMD_CG_STATE_UNGATE); > + AMD_CG_STATE_GATE); > cgs_set_powergating_state(hwmgr->device, > AMD_IP_BLOCK_TYPE_UVD, > AMD_PG_STATE_GATE); > @@ -182,7 +182,7 @@ int cz_dpm_powergate_uvd(struct pp_hwmgr > *hwmgr, bool bgate) > AMD_CG_STATE_UNGATE); > cgs_set_clockgating_state(hwmgr->device, > AMD_IP_BLOCK_TYPE_UVD, > - AMD_PG_STATE_GATE); > + AMD_PG_STATE_UNGATE); > cz_dpm_update_uvd_dpm(hwmgr, false); > } > > -- > 1.9.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </div> </div> </div> </body> </html>