On 2018-05-18 06:33 AM, Michel DÃĪnzer wrote:
> From: Michel DÃĪnzer <michel.daenzer at amd.com>
>> Leo pointed out that drmmode_do_crtc_dpms wasn't getting called when
> turning off an output with
>> xrandr --output <output> --off
>> This meant that the vblank sequence number and timestamp wouldn't be
> saved before turning off the CRTC in this case.
>> Reported-by: Leo (Sunpeng) Li <sunpeng.li at amd.com>
> Signed-off-by: Michel DÃĪnzer <michel.daenzer at amd.com>
Series is
Reviewed-by: Leo (Sunpeng) Li <sunpeng.li at amd.com>
> ---
> src/drmmode_display.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>> diff --git a/src/drmmode_display.c b/src/drmmode_display.c
> index 0ee89961a..5d4627810 100644
> --- a/src/drmmode_display.c
> +++ b/src/drmmode_display.c
> @@ -342,8 +342,7 @@ drmmode_crtc_dpms(xf86CrtcPtr crtc, int mode)
>> /* Disable unused CRTCs and enable/disable active CRTCs */
> if (!crtc->enabled || mode != DPMSModeOn) {
> - drmmode_crtc_wait_pending_event(drmmode_crtc, pAMDGPUEnt->fd,
> - drmmode_crtc->flip_pending);
> + drmmode_do_crtc_dpms(crtc, DPMSModeOff);
> drmModeSetCrtc(pAMDGPUEnt->fd, drmmode_crtc->mode_crtc->crtc_id,
> 0, 0, 0, NULL, 0, NULL);
> drmmode_fb_reference(pAMDGPUEnt->fd, &drmmode_crtc->fb, NULL);
>
