commit 63abf3edaf42d0b9f278df90fe41c7ed4796b6b1
Author: Chris Wilson <[email protected]>
Date:   Wed Dec 8 16:48:21 2010 +0000

    drm/i915/sdvo: Only use the SDVO pin if it is in the valid range

added a default fallback if BIOS provides an invalid pin mapping, but
failed to force GPIO bit-banging on it. Finish the job, and also clean up
the function a bit. With bit-banging, setting the gmbus speed has no
effect, so drop it.

Signed-off-by: Jani Nikula <[email protected]>
---
 drivers/gpu/drm/i915/intel_sdvo.c |   16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_sdvo.c 
b/drivers/gpu/drm/i915/intel_sdvo.c
index 0007a4d..eab1277 100644
--- a/drivers/gpu/drm/i915/intel_sdvo.c
+++ b/drivers/gpu/drm/i915/intel_sdvo.c
@@ -2042,17 +2042,15 @@ intel_sdvo_select_i2c_bus(struct drm_i915_private 
*dev_priv,
        else
                mapping = &dev_priv->sdvo_mappings[1];
 
-       pin = GMBUS_PORT_DPB;
-       if (mapping->initialized)
+       if (mapping->initialized && intel_gmbus_is_port_valid(mapping->i2c_pin))
                pin = mapping->i2c_pin;
+       else
+               pin = GMBUS_PORT_DPB;
 
-       if (intel_gmbus_is_port_valid(pin)) {
-               sdvo->i2c = intel_gmbus_get_adapter(dev_priv, pin);
-               intel_gmbus_set_speed(sdvo->i2c, GMBUS_RATE_1MHZ);
-               intel_gmbus_force_bit(sdvo->i2c, true);
-       } else {
-               sdvo->i2c = intel_gmbus_get_adapter(dev_priv, GMBUS_PORT_DPB);
-       }
+       sdvo->i2c = intel_gmbus_get_adapter(dev_priv, pin);
+
+       /* gmbus is not supported with sdvo yet */
+       intel_gmbus_force_bit(sdvo->i2c, true);
 }
 
 static bool
-- 
1.7.9.5

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to