drm/msm: Change dpu_crtc_get_vblank_counter to use vsync count.

Browse source

[why]
vsync_cnt atomic counter increments for every hw vsync. On the other
hand, frame count is a register that increments when the frame gets
actually pushed out. We cannnot read this register whenever the timing
engine is off, but vblank counter should still return a valid number.
This behavior also matches the downstream driver.

[How]
Read the encoder vsync count instead of the dpu_encoder_phys frame
count.

Suggested-by: Abhinav Kumar <abhinavk@codeaurora.org>
CC: Rob Clark <robdclark@chromium.org>
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
Reviewed-by: Abhinav Kumar <abhinavk@codeaurora.org>
Link: https://lore.kernel.org/r/20210830181359.124267-1-markyacoub@chromium.org
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Rob Clark <robdclark@chromium.org>

...

This commit is contained in:

Mark Yacoub 2021-08-30 14:13:59 -04:00 • committed by Rob Clark

parent f25f656608

commit 885455d6bf

3 changed files with 9 additions and 19 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

8

drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c

View file

16

drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

View file

4

drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h

View file