mfd: max14577: Remove redundant initialization of variable current_bits
authorColin Ian King <colin.king@canonical.com>
Thu, 23 Jul 2020 16:15:41 +0000 (17:15 +0100)
committerLee Jones <lee.jones@linaro.org>
Thu, 13 Aug 2020 06:50:04 +0000 (07:50 +0100)
The variable current_bits is being initialized with a value that is
never read and it is being updated later with a new value. The
initialization is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/max14577.c

index fd8864cafd25c33a26637dc2ada915c5532334e4..be185e9d5f16b13ec7d62a1e060a8c1ecdb9eb34 100644 (file)
@@ -61,7 +61,7 @@ EXPORT_SYMBOL_GPL(maxim_charger_currents);
 int maxim_charger_calc_reg_current(const struct maxim_charger_current *limits,
                unsigned int min_ua, unsigned int max_ua, u8 *dst)
 {
-       unsigned int current_bits = 0xf;
+       unsigned int current_bits;
 
        if (min_ua > max_ua)
                return -EINVAL;