On 2024-08-22 11:58, Hamza Mahfooz wrote:
> Since, drm_crtc_vblank_on_config() was recently introduced, we want to
> apply laxer vblanking policies across the board, since:
> 
> 1. It will allow us to shut down interrupts faster for better power
>    savings.
> 
> 2. As a payment towards tech debt, since drm_vblank_offdelay (which
>    defaults to 5s) isn't necessary on AMD hardware.
> 
> Also, patches 2-3 in this series provide even laxer policies and they
> are intentionally separated from patch #1. This way, if issues with them
> are encountered in the wild, it should be easy to bisect the issue to
> those patches.
> 

With an updated description for patch 1 that calls out the new
behavior this series is
Reivewed-by: Harry Wentland <[email protected]>

Harry

> 
> Hamza Mahfooz (3):
>   drm/amd/display: use drm_crtc_vblank_on_config()
>   drm/amd/display: use a more lax vblank enable policy for DCN35+
>   drm/amd/display: use a more lax vblank enable policy for older ASICs
> 
>  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 39 +++++++++++++------
>  1 file changed, 28 insertions(+), 11 deletions(-)
> 

Reply via email to