Message ID | 1389689394-22369-6-git-send-email-aaron.f.brown@intel.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Hello. On 14-01-2014 12:49, Aaron Brown wrote: > From: Jesse Brandeburg <jesse.brandeburg@intel.com> > Remove some un-necessary parenthesis. > Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> > Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> > Signed-off-by: Aaron Brown <aaron.f.brown@intel.com> > --- > drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > diff --git a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c > index 101ed41..d5d98fe 100644 > --- a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c > +++ b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c > @@ -485,8 +485,7 @@ i40e_status i40e_configure_lan_hmc(struct i40e_hw *hw, > /* Make one big object, a single SD */ > info.count = 1; > ret_code = i40e_create_lan_hmc_object(hw, &info); > - if ((ret_code) && > - (model == I40E_HMC_MODEL_DIRECT_PREFERRED)) > + if (ret_code && (model == I40E_HMC_MODEL_DIRECT_PREFERRED)) Parens around == are also unnecessary, if you're at it. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c index 101ed41..d5d98fe 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c +++ b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c @@ -485,8 +485,7 @@ i40e_status i40e_configure_lan_hmc(struct i40e_hw *hw, /* Make one big object, a single SD */ info.count = 1; ret_code = i40e_create_lan_hmc_object(hw, &info); - if ((ret_code) && - (model == I40E_HMC_MODEL_DIRECT_PREFERRED)) + if (ret_code && (model == I40E_HMC_MODEL_DIRECT_PREFERRED)) goto try_type_paged; else if (ret_code) goto configure_lan_hmc_out;