On Tue, Jul 17, 2018 at 10:54 AM, Harry Wentland <harry.wentl...@amd.com> wrote:
> [Why]
> We were only setting this mask for DCN, but should really use it
> universally for all ASICs.
>
> [How]
> Move the assignment out of the Raven switch statement.
>
> Cc: rex....@amd.com
> Cc: feifei...@amd.com
> Cc: kenneth.f...@amd.com
> Cc: evan.q...@amd.com
> Cc: bhawanpreet.la...@amd.com
> Cc: jordan.laz...@amd.com
> Signed-off-by: Harry Wentland <harry.wentl...@amd.com>
I thought we had previously enabled stutter for CZ. Does that use a
different path or was the code reworked in the interim?
Reviewed-by: Alex Deucher <alexander.deuc...@amd.com>
> ---
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 952691c6f81e..004636ead29b 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1528,10 +1528,6 @@ static int amdgpu_dm_initialize_drm_device(struct
> amdgpu_device *adev)
> DRM_ERROR("DM: Failed to initialize IRQ\n");
> goto fail;
> }
> - /*
> - * Temporary disable until pplib/smu interaction is
> implemented
> - */
> - dm->dc->debug.disable_stutter = amdgpu_pp_feature_mask &
> PP_STUTTER_MODE ? false : true;
> break;
> #endif
> default:
> @@ -1539,6 +1535,8 @@ static int amdgpu_dm_initialize_drm_device(struct
> amdgpu_device *adev)
> goto fail;
> }
>
> + dm->dc->debug.disable_stutter = amdgpu_pp_feature_mask &
> PP_STUTTER_MODE ? false : true;
> +
> return 0;
> fail:
> kfree(aencoder);
> --
> 2.17.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
