Re: [Intel-gfx] [PATCH 5/5] drm/i915: disable the cpu edp port after the cpu pipe

2012-09-13 Thread Daniel Vetter
On Thu, Sep 13, 2012 at 04:43:57PM -0300, Paulo Zanoni wrote: > 2012/9/13 Daniel Vetter : > > On Thu, Sep 13, 2012 at 04:11:20PM -0300, Paulo Zanoni wrote: > >> Hi > >> > >> 2012/9/6 Daniel Vetter : > >> > See bspec, Vol3 Part2, Section 1.1.3 "Display Mode Set Sequence". This > >> > applies to all

Re: [Intel-gfx] [PATCH 5/5] drm/i915: disable the cpu edp port after the cpu pipe

2012-09-13 Thread Paulo Zanoni
2012/9/13 Daniel Vetter : > On Thu, Sep 13, 2012 at 04:11:20PM -0300, Paulo Zanoni wrote: >> Hi >> >> 2012/9/6 Daniel Vetter : >> > See bspec, Vol3 Part2, Section 1.1.3 "Display Mode Set Sequence". This >> > applies to all platforms where we currently support eDP on, i.e. ilk, >> > snb & ivb. >> >

Re: [Intel-gfx] [PATCH 5/5] drm/i915: disable the cpu edp port after the cpu pipe

2012-09-13 Thread Daniel Vetter
On Thu, Sep 13, 2012 at 04:11:20PM -0300, Paulo Zanoni wrote: > Hi > > 2012/9/6 Daniel Vetter : > > See bspec, Vol3 Part2, Section 1.1.3 "Display Mode Set Sequence". This > > applies to all platforms where we currently support eDP on, i.e. ilk, > > snb & ivb. > > > > Ok, so I looked at BSpec and

Re: [Intel-gfx] [PATCH 5/5] drm/i915: disable the cpu edp port after the cpu pipe

2012-09-13 Thread Paulo Zanoni
Hi 2012/9/6 Daniel Vetter : > See bspec, Vol3 Part2, Section 1.1.3 "Display Mode Set Sequence". This > applies to all platforms where we currently support eDP on, i.e. ilk, > snb & ivb. > Ok, so I looked at BSpec and the conclusion is: shouldn't we do this for eDP _and_ DP instead of just eDP? Th

[Intel-gfx] [PATCH 5/5] drm/i915: disable the cpu edp port after the cpu pipe

2012-09-06 Thread Daniel Vetter
See bspec, Vol3 Part2, Section 1.1.3 "Display Mode Set Sequence". This applies to all platforms where we currently support eDP on, i.e. ilk, snb & ivb. Without this change we fail to light up the eDP port on previously unused crtcs (likely because something is stuck on the old pipe), and we also f