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

From: Chris Wilson
Date: Tue Jan 07 2014 - 08:14:19 EST

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 associated with i915
> > after a switcheroo event.
> Okay, so I can see the following get_edid functions:
> intel_lvds_get_edid()
> intel_crt_get_edid()
> intel_sdvo_get_edid()
> intel_dp_get_edid()
> In the hdmi case, drm_get_edid() is called in two places:
> intel_hdmi_detect() and intel_hdmi_detect_audio(); should I factor out
> something into a corresponding intel_hdmi_get_edid() function?
> Now, in i915_switcheroo_reprobe(), I need to call all these functions,
> right? They all accept a a generic drm_connector and an specific
> i2c_adapter. But how do I do that without exporting each of these
> functions and their adapters?

If you look at struct intel_connector, you will see space for adding a
callback that each of the LVDS/CRT/SDVO/DP connectors can hook into if
they want to reprobe after a switcheroo.

struct intel_connector *connector;
base.head) {

(or something similar) would do in the switcheroo callback.


Chris Wilson, Intel Open Source Technology Centre
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at