Commit 8df5dd57 authored by Paulo Zanoni's avatar Paulo Zanoni Committed by Daniel Vetter

drm/i915: move set_no_fbc_reason() call out of intel_fbc_find_crtc()

So now all the calls are inside __intel_fbc_update(). Consistency!
Signed-off-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 232fd934
...@@ -509,10 +509,8 @@ static struct drm_crtc *intel_fbc_find_crtc(struct drm_i915_private *dev_priv) ...@@ -509,10 +509,8 @@ static struct drm_crtc *intel_fbc_find_crtc(struct drm_i915_private *dev_priv)
break; break;
} }
if (!crtc || crtc->primary->fb == NULL) { if (!crtc || crtc->primary->fb == NULL)
set_no_fbc_reason(dev_priv, FBC_NO_OUTPUT);
return NULL; return NULL;
}
return crtc; return crtc;
} }
...@@ -720,8 +718,10 @@ static void __intel_fbc_update(struct drm_i915_private *dev_priv) ...@@ -720,8 +718,10 @@ static void __intel_fbc_update(struct drm_i915_private *dev_priv)
* - going to an unsupported config (interlace, pixel multiply, etc.) * - going to an unsupported config (interlace, pixel multiply, etc.)
*/ */
crtc = intel_fbc_find_crtc(dev_priv); crtc = intel_fbc_find_crtc(dev_priv);
if (!crtc) if (!crtc) {
set_no_fbc_reason(dev_priv, FBC_NO_OUTPUT);
goto out_disable; goto out_disable;
}
if (!multiple_pipes_ok(dev_priv)) { if (!multiple_pipes_ok(dev_priv)) {
set_no_fbc_reason(dev_priv, FBC_MULTIPLE_PIPES); set_no_fbc_reason(dev_priv, FBC_MULTIPLE_PIPES);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment