Looks good to me. Reviewed-by: Eric Huang <JinhuiEric.Huang at amd.com>
Regards,
Eric
On 07/18/2016 12:59 PM, Rex Zhu wrote:
> Change-Id: I68aa5431146b21990a998a777e00139f0478407f
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/vce_v3_0.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vce_v3_0.c b/drivers/gpu/drm/amd/amdgpu/vce_v3_0.c
> index 30e8099..962aa5e 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vce_v3_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vce_v3_0.c
> @@ -604,6 +604,18 @@ static int vce_v3_0_process_interrupt(struct amdgpu_device *adev,
> return 0;
> }
>
> +static void vce_v3_set_bypass_mode(struct amdgpu_device *adev, bool enable)
> +{
> + u32 tmp = RREG32_SMC(ixGCK_DFS_BYPASS_CNTL);
> +
> + if (enable)
> + tmp |= GCK_DFS_BYPASS_CNTL__BYPASSECLK_MASK;
> + else
> + tmp &= ~GCK_DFS_BYPASS_CNTL__BYPASSECLK_MASK;
> +
> + WREG32_SMC(ixGCK_DFS_BYPASS_CNTL, tmp);
> +}
> +
> static int vce_v3_0_set_clockgating_state(void *handle,
> enum amd_clockgating_state state)
> {
> @@ -611,6 +623,9 @@ static int vce_v3_0_set_clockgating_state(void *handle,
> bool enable = (state == AMD_CG_STATE_GATE) ? true : false;
> int i;
>
> + if (adev->asic_type == CHIP_POLARIS10)
> + vce_v3_set_bypass_mode(adev, enable);
> +
> if (!(adev->cg_flags & AMD_CG_SUPPORT_VCE_MGCG))
> return 0;
>
