Re: [PATCH v2 07/24] drm/i915: Remove useless checks from primary enable/disable

Chris Wilson
2014-04-30T11:28:30Z

On Mon, Apr 28, 2014 at 03:53:25PM +0300, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> We won't be calling intel_enable_primary_plane() or
> intel_disable_primary_plane() with the primary plane in the
> wrong state. So remove the useless DISPLAY_PLANE_ENABLE checks.

Oh, really? That sounds very, very confident. However,
haswell_get_pipe_config() disagrees with you.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre