Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d37b4c0a authored by Ivan Vecera's avatar Ivan Vecera Committed by David S. Miller
Browse files

be2net: remove local variable 'status' 



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: default avatarIvan Vecera <ivecera@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6fbaa570

drivers/net/ethernet/emulex/benet/be_ethtool.c

+2 −4
Original line number Diff line number Diff line
@@ -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;

}