[PATCH v2] drm/amd/display: Honor pplib stutter mask for all ASICs in DC

Alex Deucher alexdeucher at gmail.com
Thu Jul 19 19:51:07 UTC 2018


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


More information about the amd-gfx mailing list