On Sat, 10 Nov 2012, Chris Wilson <[email protected]> wrote:
> @@ -583,7 +583,10 @@ void intel_gmbus_force_bit(struct i2c_adapter *adapter, 
> bool force_bit)
>  {
>       struct intel_gmbus *bus = to_intel_gmbus(adapter);
>  
> -     bus->force_bit = force_bit;
> +     bus->force_bit += force_bit ? 1 : -1;
> +     DRM_DEBUG_KMS("%sabling bit-banging on %s. force bit now %d\n",
> +                   force_bit ? "dis" : "en", adapter->name,
> +                   bus->force_bit);

Reading a dmesg from Chris, I noticed force_bit ? "dis" : "en" is
backwards.

BR,
Jani.

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

Reply via email to