On 12-05-2015 22:19, Wolfram Sang wrote: > On Thu, Apr 23, 2015 at 10:27:50PM +0200, Svenning Sørensen wrote: >> According to I2C spec, max SCL rate is 100 kHz, but SCx200/CS5536 >> controller is currently driving it at 214 kHz according to my math. >> >> SCL is derived from an input clock of 48 MHz, which must be divided >> by 480 (240 cycles for each SCL high/low state) to be within spec. >> >> Signed-off-by: Svenning Soerensen <sss@xxxxxxxxxxx> > > So, did you test/measure the actual frequency? Or is this patch based on > specs only? Thanks for caring about this old driver, though! > Unfortunately I don't have any scope, so it is based on datasheet only. I noticed sporadic i2c errors reading the temperature on some of my PC Engines Alix.2 boards (with LM86 sensor chip) so started digging the datasheets and driver for an explanation. I haven't noticed any problems after this change. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html