Daniel Vetter wrote:
> On Fri, Apr 19, 2013 at 10:41:50AM +0200, "David M?ller (ELSOFT AG)" wrote:
>> +            /* GMBUS NAK handling seems to be unstable, hence let the
>> +             * transmitter detection run in bit banging mode for now.
>> +             */
>> +            intel_gmbus_force_bit(i2c, true);
>> +
>> +            dvoinit = dvo->dev_ops->init(&intel_dvo->dev, i2c);
>> +
>> +            intel_gmbus_force_bit(i2c, false);
> 
> Shouldn't we keep the dvo i2c line in bit-banging mode always, i.e.
> restore gmbus mode only when dvo init failed?
> 
> I suspect that for fickle hw not just init will fail ...

As far as i can tell, the critical part is the NAK handling.

Dave

Reply via email to