diff options
| author | Ville Syrjälä | 2016-04-29 16:31:18 +0200 |
|---|---|---|
| committer | Ville Syrjälä | 2016-05-09 18:14:57 +0200 |
| commit | f818ffea87bb7ee4eb8813eb6258778c5460db2e (patch) | |
| tree | f152bc7a36f338aef7ac40ebd37281fe3500315b /drivers/gpu/drm/i915/intel_display.c | |
| parent | drm/i915: Calculate IPS linetime watermark based on future cdclk (diff) | |
| download | kernel-qcow2-linux-f818ffea87bb7ee4eb8813eb6258778c5460db2e.tar.gz kernel-qcow2-linux-f818ffea87bb7ee4eb8813eb6258778c5460db2e.tar.xz kernel-qcow2-linux-f818ffea87bb7ee4eb8813eb6258778c5460db2e.zip | |
drm/i915: Add a FIXME about crtc !active vs. watermarks
When the crtc is enabled but !active, we should still compute the
watermarks as if the planes were visible. That would make it more
likely that the we can later transition to active without errors.
Add a FIXME to remind people that we're doing the wrong thing now.
We should perhaps just move the wm computation for each individual plane
into the .check_plane hook, and later we'd just combine the results from
all active planes.
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1461940278-17122-2-git-send-email-ville.syrjala@linux.intel.com
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_display.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index a8f5d035941b..d3f6e58c199e 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -11823,6 +11823,11 @@ int intel_plane_atomic_calc_changes(struct drm_crtc_state *crtc_state, * Visibility is calculated as if the crtc was on, but * after scaler setup everything depends on it being off * when the crtc isn't active. + * + * FIXME this is wrong for watermarks. Watermarks should also + * be computed as if the pipe would be active. Perhaps move + * per-plane wm computation to the .check_plane() hook, and + * only combine the results from all planes in the current place? */ if (!is_crtc_enabled) to_intel_plane_state(plane_state)->visible = visible = false; |