On Thu, Jul 19, 2018 at 3:49 PM, Harry Wentland <harry.wentland at 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 for all ASICs
> other than Stoney and Carrizo.
>
> v2: Keep stutter always on for Carrizo and Stoney (Alex)
>
> Cc: Rex.Zhu at amd.com
> Cc: Feifei.Xu at amd.com
> Cc: Kenneth.Feng at amd.com
> Cc: Evan.Quan at amd.com
> Cc: Bhawanpreet.Lakha at amd.com
> Cc: Jordan.Lazare at amd.com
> Signed-off-by: Harry Wentland <harry.wentland at amd.com>
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> ---
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 +++----
> 1 file changed, 3 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 8e3ebd988043..4d6e5914256c 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,9 @@ static int amdgpu_dm_initialize_drm_device(struct amdgpu_device *adev)
> goto fail;
> }
>
> + if (adev->asic_type != CHIP_CARRIZO && adev->asic_type != CHIP_STONEY)
> + 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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
