diff options
| author | Lee Jones <lee.jones@linaro.org> | 2020-06-24 13:31:31 +0100 |
|---|---|---|
| committer | Lee Jones <lee.jones@linaro.org> | 2020-07-06 08:32:02 +0100 |
| commit | 2fbd583443905d6f8c6d87aebdef2717ae039f0b (patch) | |
| tree | ef4ee010fc25c76eec039c2b6d4e7ccfd1c2d4a9 /drivers/mfd | |
| parent | 20d60f850d2d8b6f9536c1a499fc6d6a5c724a55 (diff) | |
| download | lwn-2fbd583443905d6f8c6d87aebdef2717ae039f0b.tar.gz lwn-2fbd583443905d6f8c6d87aebdef2717ae039f0b.zip | |
mfd: tps65010: Remove delcared and set, but never used variable 'status'
'status' hasn't been checked since 2008.
It's probably safe to remove it.
Fixes W=1 warning:
drivers/mfd/tps65010.c:407:7: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
407 | int status;
| ^~~~~~
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd')
| -rw-r--r-- | drivers/mfd/tps65010.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/mfd/tps65010.c b/drivers/mfd/tps65010.c index 65fcc58c02da..7e7dbee58ca9 100644 --- a/drivers/mfd/tps65010.c +++ b/drivers/mfd/tps65010.c @@ -404,7 +404,6 @@ static void tps65010_work(struct work_struct *work) tps65010_interrupt(tps); if (test_and_clear_bit(FLAG_VBUS_CHANGED, &tps->flags)) { - int status; u8 chgconfig, tmp; chgconfig = i2c_smbus_read_byte_data(tps->client, @@ -415,8 +414,8 @@ static void tps65010_work(struct work_struct *work) else if (tps->vbus >= 100) chgconfig |= TPS_VBUS_CHARGING; - status = i2c_smbus_write_byte_data(tps->client, - TPS_CHGCONFIG, chgconfig); + i2c_smbus_write_byte_data(tps->client, + TPS_CHGCONFIG, chgconfig); /* vbus update fails unless VBUS is connected! */ tmp = i2c_smbus_read_byte_data(tps->client, TPS_CHGCONFIG); |