On Thu, Jan 17, 2013 at 10:45:53AM +0100, Guennadi Liakhovetski wrote: > Currently the i2c-sh_mobile.c driver generates STOP conditions after each > message. This makes any multi-message transfers, e.g. reading, impossible > with the AS3711 PMIC on kzm9g. This patch series fixes this by only > generating the STOP condition after the last message or if the I2C_M_STOP > flag is set. > > Guennadi Liakhovetski (4): > i2c: i2c-sh_mobile.c: cosmetic: trivially simplify 2 functions > i2c: sh_mobile: fix timeout error handling > i2c: sh_mobile: eliminate an open-coded "goto" loop > i2c: sh_mobile: don't send a stop condition by default inside > transfers Looks good to me. I have to mention usleep_range, though. I leave it to you if you want to fix it or not. Other than that, I would apply the series. -- 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