lan743x: remove redundant assignment to variable rx_process_result
authorColin Ian King <colin.king@canonical.com>
Thu, 5 Sep 2019 14:01:35 +0000 (15:01 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 6 Sep 2019 14:47:07 +0000 (16:47 +0200)
The variable rx_process_result is being initialized with a value that
is never read and is being re-assigned immediately afterwards. The
assignment is redundant, so replace it with the return from function
lan743x_rx_process_packet.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/microchip/lan743x_main.c

index 15a8be6..a43140f 100644 (file)
@@ -2172,9 +2172,8 @@ static int lan743x_rx_napi_poll(struct napi_struct *napi, int weight)
        }
        count = 0;
        while (count < weight) {
-               int rx_process_result = -1;
+               int rx_process_result = lan743x_rx_process_packet(rx);
 
-               rx_process_result = lan743x_rx_process_packet(rx);
                if (rx_process_result == RX_PROCESS_RESULT_PACKET_RECEIVED) {
                        count++;
                } else if (rx_process_result ==