[i2c] [PATCH] i2c: improve lost-arbitration handling in i2c-algo-pcf

Daniel Smolik marvin at mydatex.cz
Mon Aug 13 13:10:54 CEST 2007


Jean Delvare napsal(a):
> Hi Daniel,
> 
> On Sun, 12 Aug 2007 21:18:58 +0200, Daniel Smolik wrote:
>>  >Attached is a patch that improves multi-master support for PCF8584.
>>  >The current implementation of multi-master support (mine) in
>>  >i2c-algo-pcf does not properly handle a collision detected during 
>>  >wait_for_bb()
>>  >and thus causes an error that is only recoverable by reloading the
>>  >relevant device drivers.  I'd appreciate your consideration of this
>>  >patch.
>>
>>  >The only affected driver is i2c-envctrl.c-- currently out-of-kernel,
>>  >in development; you
>>  >can find it here if you are interested:
>>
>>     http://www.david-web.co.uk/index.php?p=envctrl
>>
>>  >This change has been tested by a few individuals (on Sun Microsystems
>>  >E250 servers) over the course of several months, and rectifies
>>  >occasional errors seen without the patch applied.
>>
>>  >I'll be happy to provide further clarification if you are interested.
>>  >Please cc me on responses-- I am not a member of this list.
>>
>> Thanks, I test this patch one week and my E250 works wihout problem.
>>
>> (Please cc me too on responses-- I am not a member of this list.)
>> Acked-by: Daniel Smolik <marvin at mydatex.cz>
> 
> Thanks for your testing and ack. I've reviewed the patch now and asked
> Eric for an update:
> http://lists.lm-sensors.org/pipermail/i2c/2007-August/001690.html
> 
> I would be grateful if you could test again when the new patch is
> available.
> 
> Thanks,
No problem Jean,
please let me info and I test again.


                  Regards
				Dan




More information about the i2c mailing list