From: Paulo Zanoni <[email protected]>

The first one is helpless against a single variable read. The second
one is not needed anymore now that we have fbc.lock.

Signed-off-by: Paulo Zanoni <[email protected]>
---
 drivers/gpu/drm/i915/i915_debugfs.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c 
b/drivers/gpu/drm/i915/i915_debugfs.c
index 5248ed9..4de083b 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -1646,9 +1646,7 @@ static int i915_fbc_fc_get(void *data, u64 *val)
        if (INTEL_INFO(dev)->gen < 7 || !HAS_FBC(dev))
                return -ENODEV;
 
-       drm_modeset_lock_all(dev);
        *val = dev_priv->fbc.false_color;
-       drm_modeset_unlock_all(dev);
 
        return 0;
 }
@@ -1662,7 +1660,6 @@ static int i915_fbc_fc_set(void *data, u64 val)
        if (INTEL_INFO(dev)->gen < 7 || !HAS_FBC(dev))
                return -ENODEV;
 
-       drm_modeset_lock_all(dev);
        mutex_lock(&dev_priv->fbc.lock);
 
        reg = I915_READ(ILK_DPFC_CONTROL);
@@ -1673,7 +1670,6 @@ static int i915_fbc_fc_set(void *data, u64 val)
                   (reg & ~FBC_CTL_FALSE_COLOR));
 
        mutex_unlock(&dev_priv->fbc.lock);
-       drm_modeset_unlock_all(dev);
        return 0;
 }
 
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to