staging: net: wimax: i2400m: fw: remove redundant initialization of variable result
authorColin Ian King <colin.king@canonical.com>
Thu, 28 Jan 2021 17:37:02 +0000 (17:37 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 29 Jan 2021 15:45:57 +0000 (16:45 +0100)
The variable result 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.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Addresses-Coverity: ("Unused value")
Link: https://lore.kernel.org/r/20210128173703.645328-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wimax/i2400m/fw.c

index edb5eba0898b0001d7080b636876e000f6ab380b..b2fd4bd2c5f908161d5eaa29ed112affa944b8e6 100644 (file)
@@ -583,7 +583,7 @@ ssize_t i2400m_bm_cmd(struct i2400m *i2400m,
                      struct i2400m_bootrom_header *ack, size_t ack_size,
                      int flags)
 {
-       ssize_t result = -ENOMEM, rx_bytes;
+       ssize_t result, rx_bytes;
        struct device *dev = i2400m_dev(i2400m);
        int opcode = cmd == NULL ? -1 : i2400m_brh_get_opcode(cmd);