* [PATCH net-next] be2net: remove local variable 'status'
@ 2015-11-18 13:06 Ivan Vecera
2015-11-18 20:22 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Ivan Vecera @ 2015-11-18 13:06 UTC (permalink / raw)
To: netdev
Cc: sriharsha.basavapatna, sathya.perla, ajit.khaparde, padmanabh.ratnakar
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>
---
drivers/net/ethernet/emulex/benet/be_ethtool.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/emulex/benet/be_ethtool.c b/drivers/net/ethernet/emulex/benet/be_ethtool.c
index 734f655..d2a5baf 100644
--- a/drivers/net/ethernet/emulex/benet/be_ethtool.c
+++ b/drivers/net/ethernet/emulex/benet/be_ethtool.c
@@ -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;
}
--
2.4.10
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] be2net: remove local variable 'status'
2015-11-18 13:06 [PATCH net-next] be2net: remove local variable 'status' Ivan Vecera
@ 2015-11-18 20:22 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2015-11-18 20:22 UTC (permalink / raw)
To: ivecera
Cc: netdev, sriharsha.basavapatna, sathya.perla, ajit.khaparde,
padmanabh.ratnakar
From: Ivan Vecera <ivecera@redhat.com>
Date: Wed, 18 Nov 2015 14:06:34 +0100
> 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>
Applied.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-11-18 20:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-18 13:06 [PATCH net-next] be2net: remove local variable 'status' Ivan Vecera
2015-11-18 20:22 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.