Re: [PATCH 4/4] drm/i915: check LVDS for EDID on GPU switches

2014-01-07 Thread Chris Wilson
On Tue, Jan 07, 2014 at 06:35:26PM +0530, Ramkumar Ramachandra wrote: > Chris Wilson wrote: > > Rather than special casing lvds (especially when there are other panel > > connectors that can also be muxed), extend the connector interface to > > support a reprobe and walk over all connectors associa

Re: [PATCH 4/4] drm/i915: check LVDS for EDID on GPU switches

2014-01-07 Thread Ramkumar Ramachandra
Chris Wilson wrote: > Rather than special casing lvds (especially when there are other panel > connectors that can also be muxed), extend the connector interface to > support a reprobe and walk over all connectors associated with i915 > after a switcheroo event. Okay, so I can see the following ge

Re: [PATCH 4/4] drm/i915: check LVDS for EDID on GPU switches

2014-01-07 Thread Chris Wilson
On Tue, Jan 07, 2014 at 04:42:12PM +0530, Ramkumar Ramachandra wrote: > Chris Wilson wrote: > > On Tue, Jan 07, 2014 at 03:28:43PM +0530, Ramkumar Ramachandra wrote: > >> If the LVDS panel wasn't connected at boot then we won't have an EDID > >> for it. To fix this, call intel_lvds_get_edid() from

Re: [PATCH 4/4] drm/i915: check LVDS for EDID on GPU switches

2014-01-07 Thread Ramkumar Ramachandra
Chris Wilson wrote: > On Tue, Jan 07, 2014 at 03:28:43PM +0530, Ramkumar Ramachandra wrote: >> If the LVDS panel wasn't connected at boot then we won't have an EDID >> for it. To fix this, call intel_lvds_get_edid() from the vga_switcheroo >> reprobe callback. > > I would rather have an iterator ov

Re: [PATCH 4/4] drm/i915: check LVDS for EDID on GPU switches

2014-01-07 Thread Chris Wilson
On Tue, Jan 07, 2014 at 03:28:43PM +0530, Ramkumar Ramachandra wrote: > If the LVDS panel wasn't connected at boot then we won't have an EDID > for it. To fix this, call intel_lvds_get_edid() from the vga_switcheroo > reprobe callback. I would rather have an iterator over all our connectors (or pe

[PATCH 4/4] drm/i915: check LVDS for EDID on GPU switches

2014-01-07 Thread Ramkumar Ramachandra
If the LVDS panel wasn't connected at boot then we won't have an EDID for it. To fix this, call intel_lvds_get_edid() from the vga_switcheroo reprobe callback. Cc: Andreas Heider Cc: Seth Forshee Original-patch-by: Seth Forshee Signed-off-by: Ramkumar Ramachandra --- drivers/gpu/drm/i915/i915