be2net: remove local variable 'status'
authorIvan Vecera <ivecera@redhat.com>
Wed, 18 Nov 2015 13:06:34 +0000 (14:06 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Nov 2015 20:21:41 +0000 (15:21 -0500)
The lancer_cmd_get_file_len() uses lancer_cmd_read_object() to get
the current size of registers for ethtool registers dump. Returned status
value is stored but not checked. The check itself is not necessary as
the data_read output variable is initialized to 0 and status variable
can be removed.

Signed-off-by: Ivan Vecera <ivecera@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/emulex/benet/be_ethtool.c

index 734f655c99c113805e270a0846e934553fbca9b7..d2a5baf019ab806d0e224984c157e84f0f77184c 100644 (file)
@@ -241,13 +241,11 @@ static u32 lancer_cmd_get_file_len(struct be_adapter *adapter, u8 *file_name)
        u32 data_read = 0, eof;
        u8 addn_status;
        struct be_dma_mem data_len_cmd;
-       int status;
 
        memset(&data_len_cmd, 0, sizeof(data_len_cmd));
        /* data_offset and data_size should be 0 to get reg len */
-       status = lancer_cmd_read_object(adapter, &data_len_cmd, 0, 0,
-                                       file_name, &data_read, &eof,
-                                       &addn_status);
+       lancer_cmd_read_object(adapter, &data_len_cmd, 0, 0, file_name,
+                              &data_read, &eof, &addn_status);
 
        return data_read;
 }