]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
drm/i915: Remove pointless planes_changed=true assignment
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 8 Jul 2019 12:53:09 +0000 (15:53 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 11 Sep 2019 14:04:27 +0000 (17:04 +0300)
i915 doesn't use the crtc_state->plane_changed flag for anything,
so setting it is pointless.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190708125325.16576-4-ville.syrjala@linux.intel.com
Reviewed-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
drivers/gpu/drm/i915/display/intel_atomic.c

index d3fb75bb9eb1d1b4c87a58bb8ca108cef42a7aa9..698802da07b7318122acb3b87288272d32a2e719 100644 (file)
@@ -378,13 +378,6 @@ int intel_atomic_setup_scalers(struct drm_i915_private *dev_priv,
                                                plane->base.id);
                                        return PTR_ERR(state);
                                }
-
-                               /*
-                                * the plane is added after plane checks are run,
-                                * but since this plane is unchanged just do the
-                                * minimum required validation.
-                                */
-                               crtc_state->base.planes_changed = true;
                        }
 
                        intel_plane = to_intel_plane(plane);