Commit 104618b3 authored by Paulo Zanoni's avatar Paulo Zanoni Committed by Daniel Vetter

drm/i915: don't keep reassigning FBC_UNSUPPORTED

This may save a few picoseconds on !HAS_FBC platforms. And it also
satisfies my OCD.
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 7cc65746
...@@ -502,10 +502,8 @@ void intel_fbc_update(struct drm_device *dev) ...@@ -502,10 +502,8 @@ void intel_fbc_update(struct drm_device *dev)
const struct drm_display_mode *adjusted_mode; const struct drm_display_mode *adjusted_mode;
unsigned int max_width, max_height; unsigned int max_width, max_height;
if (!HAS_FBC(dev)) { if (!HAS_FBC(dev))
set_no_fbc_reason(dev_priv, FBC_UNSUPPORTED);
return; return;
}
if (i915.enable_fbc < 0) { if (i915.enable_fbc < 0) {
if (set_no_fbc_reason(dev_priv, FBC_CHIP_DEFAULT)) if (set_no_fbc_reason(dev_priv, FBC_CHIP_DEFAULT))
...@@ -670,6 +668,7 @@ void intel_fbc_init(struct drm_i915_private *dev_priv) ...@@ -670,6 +668,7 @@ void intel_fbc_init(struct drm_i915_private *dev_priv)
{ {
if (!HAS_FBC(dev_priv)) { if (!HAS_FBC(dev_priv)) {
dev_priv->fbc.enabled = false; dev_priv->fbc.enabled = false;
dev_priv->fbc.no_fbc_reason = FBC_UNSUPPORTED;
return; return;
} }
......
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