e1000e: Remove redundant statement
authorluo penghao <cgel.zte@gmail.com>
Mon, 18 Oct 2021 08:51:54 +0000 (08:51 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 19 Oct 2021 11:48:40 +0000 (12:48 +0100)
This assignment statement is meaningless, because the statement
will execute to the tag "set_itr_now".

The clang_analyzer complains as follows:

drivers/net/ethernet/intel/e1000e/netdev.c:2552:3 warning:

Value stored to 'current_itr' is never read.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: luo penghao <luo.penghao@zte.com.cn>
Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/intel/e1000e/netdev.c

index ff8672ae2b6ce69c15f2b56c5d425d433c704340..21ec7169b6a16c485f990281b953c1ca85778b00 100644 (file)
@@ -2549,7 +2549,6 @@ static void e1000_set_itr(struct e1000_adapter *adapter)
 
        /* for non-gigabit speeds, just fix the interrupt rate at 4000 */
        if (adapter->link_speed != SPEED_1000) {
-               current_itr = 0;
                new_itr = 4000;
                goto set_itr_now;
        }