Searched hist:"1 bc2962e530527de829bf4b1eb99f24dc25d1828" (Results 1 – 1 of 1) sorted by relevance
/linux/drivers/i2c/busses/ |
H A D | i2c-s3c2410.c | diff 1bc2962e530527de829bf4b1eb99f24dc25d1828 Fri Apr 02 15:15:09 CEST 2010 Mark Brown <broonie@opensource.wolfsonmicro.com> i2c-s3c2410: Remove unconditional 1ms delay on each transfer
The S3C I2C controller indicates completion of I2C transfers before the bus has a stop condition on it. In order to ensure that we do not attempt to start a new transfer before the bus is idle the driver currently inserts a 1ms delay. This is vastly larger than is generally required and has a visible effect on performance under load, such as when bringing up audio CODECs or reading back status information with non-bulk I2C reads.
Replace the sleep with a spin on the IIC status register for up to 1ms. This will busy wait but testing on my SMDK6410 system indicates that the overwhelming majority of transactions complete on the first spin, with maximum latencies of less than 10 spins so the absolute overhead of busy waiting should be at worst comprable to msleep(), and the overall system performance is dramatically improved.
The main risk is poor interaction with multimaster systems where we may miss the bus going idle before the next transaction. Defend against this by falling back to the original 1ms delay after 20 spins.
The overall effect in my testing is an approximately 20% improvement in kernel startup time.
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com> Signed-off-by: Ben Dooks <ben-linux@fluff.org>
|