Re: [Intel-gfx] [PATCH] drm/i915: Rip out SUPPORTS_EDP

2013-09-24 Thread Daniel Vetter
On Tue, Sep 24, 2013 at 3:05 PM, Paulo Zanoni wrote: >> I haven't seen any of those. What has been observed in the wild is lvds + >> sdvo lvds, but I've never seen edp + some other panel anywhere ... > > Now that I think more about it, we only have 1 set of panel power > sequencing registers. How

Re: [Intel-gfx] [PATCH] drm/i915: Rip out SUPPORTS_EDP

2013-09-24 Thread Paulo Zanoni
2013/9/24 Daniel Vetter : > On Mon, Sep 23, 2013 at 02:04:43PM -0300, Paulo Zanoni wrote: >> 2013/9/21 Daniel Vetter : >> > It only controls the setting of the vbt.edp_support variable, which in >> > turn only controls one debug output plus can also force-disable the >> > lvds output. >> > >> > Sin

Re: [Intel-gfx] [PATCH] drm/i915: Rip out SUPPORTS_EDP

2013-09-24 Thread Daniel Vetter
On Mon, Sep 23, 2013 at 02:04:43PM -0300, Paulo Zanoni wrote: > 2013/9/21 Daniel Vetter : > > It only controls the setting of the vbt.edp_support variable, which in > > turn only controls one debug output plus can also force-disable the > > lvds output. > > > > Since the value only restricted this

Re: [Intel-gfx] [PATCH] drm/i915: Rip out SUPPORTS_EDP

2013-09-23 Thread Paulo Zanoni
2013/9/21 Daniel Vetter : > It only controls the setting of the vbt.edp_support variable, which in > turn only controls one debug output plus can also force-disable the > lvds output. > > Since the value only restricted this logic to mobile ilk there's the > slight risk that this will break lvds on

[Intel-gfx] [PATCH] drm/i915: Rip out SUPPORTS_EDP

2013-09-21 Thread Daniel Vetter
It only controls the setting of the vbt.edp_support variable, which in turn only controls one debug output plus can also force-disable the lvds output. Since the value only restricted this logic to mobile ilk there's the slight risk that this will break lvds on desktop ilk or on snb/ivb platforms.