On Sat, Nov 29, 2014 at 01:58:42PM -0800, Danielle Costantino wrote:

i2c: mpc: add register documentation to Freescale I2C driver

This should be in one patch.

return -ETIMEDOUT for all time-out error conditions

This should be in a seperate patch.

and warn on arbitration lost.

And this should be dropped. "Arbitration lost" lost is not an error, it
is specified behaviour of I2C.

     if (!(cmd_err & CSR_MCF)) {
-        dev_dbg(i2c->dev, "unfinished\n");
+        dev_warn(i2c->dev, "unfinished\n");

Are you sure this helps a regular user?