We know "ret" is zero here.  No need to check again.

Signed-off-by: Dan Carpenter <[email protected]>

diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c 
b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
index 28da412..2640004 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
@@ -1534,12 +1534,12 @@ static int i40e_add_del_fdir_ethtool(struct i40e_vsi 
*vsi,
                return ret;
        }
 
-       if (!ret && add)
+       if (add)
                i40e_update_ethtool_fdir_entry(vsi, input, fsp->location, NULL);
        else
                kfree(input);
 
-       return ret;
+       return 0;
 }
 
 /**

------------------------------------------------------------------------------
_______________________________________________
E1000-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit 
http://communities.intel.com/community/wired

Reply via email to