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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 30, 2014 at 02:40:10PM +0300, Ville Syrjälä wrote:
> On Wed, Apr 30, 2014 at 12:28:25PM +0100, Chris Wilson wrote:
> > On Mon, Apr 28, 2014 at 03:53:25PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote:
> > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> > > 
> > > 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.
> 
> Are you saying you have a machine where the BIOS enables the pipe
> but not the plane? Otherwise I don't know what get_pipe_config() has to
> do with this.

Yes.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux