diff options
| author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2023-06-06 22:14:59 +0300 |
|---|---|---|
| committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2023-09-27 18:46:08 +0300 |
| commit | 7678e089bd18b4dd61478ce728f01fd5239e97ff (patch) | |
| tree | 4f074ce5efa9dc35ce2c4c841c711d29799e4a15 /drivers/gpu/drm/i915 | |
| parent | 25ea3411bd23c5f0043881e2c6710423eb411784 (diff) | |
| download | linux-7678e089bd18b4dd61478ce728f01fd5239e97ff.tar.xz | |
drm/i915/dsb: Evade transcoder undelayed vblank when using DSB
We want to start the DSB execution from the transcoder's undelayed
vblank, so in order to guarantee atomicity with the all the other
mmio register writes we need to evade both vblanks.
Note that currently we don't add any vblank delay, so this is
effectively a nop. But in the future when we start to program
double buffered registers from the DSB we'll need to delay the
pipe's vblank to provide the register programming "window2"
for the DSB.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230606191504.18099-15-ville.syrjala@linux.intel.com
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915')
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_crtc.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_crtc.c b/drivers/gpu/drm/i915/display/intel_crtc.c index 492347bd0e9d..e233ec9d1ba8 100644 --- a/drivers/gpu/drm/i915/display/intel_crtc.c +++ b/drivers/gpu/drm/i915/display/intel_crtc.c @@ -516,8 +516,11 @@ static void intel_crtc_vblank_evade_scanlines(struct intel_atomic_state *state, * M/N and TRANS_VTOTAL are double buffered on the transcoder's * undelayed vblank, so with seamless M/N and LRR we must evade * both vblanks. + * + * DSB execution waits for the transcoder's undelayed vblank, + * hence we must kick off the commit before that. */ - if (new_crtc_state->update_m_n || new_crtc_state->update_lrr) + if (new_crtc_state->dsb || new_crtc_state->update_m_n || new_crtc_state->update_lrr) *min -= adjusted_mode->crtc_vblank_start - adjusted_mode->crtc_vdisplay; } |