Messages in this thread Patch in this message | | | From | Amit Kucheria <> | Subject | [PATCH] mfd: twl4030: clarify the return value for read and write | Date | Wed, 2 Dec 2009 15:31:18 +0200 |
| |
We should be checking if all the messages were tranferred or not. Currently we return success even if none of messages were transferred successfully.
Signed-off-by: Amit Kucheria <amit.kucheria@verdurent.com> --- drivers/mfd/twl4030-core.c | 21 +++++++++++++-------- 1 files changed, 13 insertions(+), 8 deletions(-)
diff --git a/drivers/mfd/twl4030-core.c b/drivers/mfd/twl4030-core.c index cf462b9..6ace378 100644 --- a/drivers/mfd/twl4030-core.c +++ b/drivers/mfd/twl4030-core.c @@ -298,10 +298,12 @@ int twl4030_i2c_write(u8 mod_no, u8 *value, u8 reg, unsigned num_bytes) ret = i2c_transfer(twl->client->adapter, twl->xfer_msg, 1); mutex_unlock(&twl->xfer_lock); - /* i2cTransfer returns num messages.translate it pls.. */ - if (ret >= 0) - ret = 0; - return ret; + /* i2c_transfer returns number of messages transferred */ + if (ret != 1) { + pr_err("%s: twl4030_i2c_write failed to transfer all messages\n", DRIVER_NAME); + return ret; + } else + return 0; } EXPORT_SYMBOL(twl4030_i2c_write); @@ -350,10 +352,13 @@ int twl4030_i2c_read(u8 mod_no, u8 *value, u8 reg, unsigned num_bytes) ret = i2c_transfer(twl->client->adapter, twl->xfer_msg, 2); mutex_unlock(&twl->xfer_lock); - /* i2cTransfer returns num messages.translate it pls.. */ - if (ret >= 0) - ret = 0; - return ret; + /* i2c_transfer returns number of messages transferred */ + if (ret != 2) { + pr_err("%s: twl4030_i2c_read failed to transfer all messages\n", DRIVER_NAME); + return ret; + } + else + return 0; } EXPORT_SYMBOL(twl4030_i2c_read); -- 1.6.3.3
| |