diff options
| author |   Axel Lin <axel.lin@gmail.com> | 2010-10-22 08:30:26 +0800 |
|---|---|---|
| committer |   Samuel Ortiz <sameo@linux.intel.com> | 2010-10-29 00:30:37 +0200 |
| commit | 25d950714781f34e7ae1292f38889cf39983cb38 (patch) | |
| tree | e89653e3a0d630e8ae171b67f509d451c0e95ea7 /drivers | |
| parent | mfd: Add free max8998->ono irq in max8998_irq_exit() (diff) | |
| download | linux-dev-25d950714781f34e7ae1292f38889cf39983cb38.tar.xz linux-dev-25d950714781f34e7ae1292f38889cf39983cb38.zip | |
mfd: Remove unneeded ret value checking for max8998 register updates
i2c_smbus_write_byte_data() returns zero or negative value,
therefore no need to check if ret is greater than zero or not.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mfd/max8998.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/mfd/max8998.c b/drivers/mfd/max8998.c index a720f412cd15..06ddf741d4c7 100644 --- a/drivers/mfd/max8998.c +++ b/drivers/mfd/max8998.c @@ -110,8 +110,6 @@ int max8998_update_reg(struct i2c_client *i2c, u8 reg, u8 val, u8 mask) u8 old_val = ret & 0xff; u8 new_val = (val & mask) | (old_val & (~mask)); ret = i2c_smbus_write_byte_data(i2c, reg, new_val); - if (ret >= 0) - ret = 0; } mutex_unlock(&max8998->iolock); return ret; |