i40e: remove dead code

This looks like a cut and paste error. The code makes no sense where
it is, and accomplishes nothing. Since we've removed the goto, we can
also get rid of the extraneous brackets.

Change-ID: I9315e3eafeee0a5713c94b0dc57b58b60a849124
Signed-off-by: Mitch Williams <mitch.a.williams@intel.com>
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Tested-by: Sibai Li <sibai.li@intel.com>
Signed-off-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
index b9d1c1c8ca5a69f0d4d801f3cba0e4bc960ab46a..299372b852ce5eac0cb4cec301c2cd4b74314029 100644 (file)
--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
@@ -408,18 +408,10 @@ static int i40e_alloc_vsi_res(struct i40e_vf *vf, enum i40e_vsi_type type)
                                 "Could not allocate VF broadcast filter\n");
        }
 
-       if (!f) {
-               dev_err(&pf->pdev->dev, "Unable to add ucast filter\n");
-               ret = -ENOMEM;
-               goto error_alloc_vsi_res;
-       }
-
        /* program mac filter */
        ret = i40e_sync_vsi_filters(vsi);
-       if (ret) {
+       if (ret)
                dev_err(&pf->pdev->dev, "Unable to program ucast filters\n");
-               goto error_alloc_vsi_res;
-       }
 
 error_alloc_vsi_res:
        return ret;