d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

kni: remove redundant parenthesis

Browse Source 
Don't need parens on simple if expression

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
This commit is contained in:
Stephen Hemminger 2017-12-14 15:32:18 -08:00 committed by Thomas Monjalon
parent eb23e8c9af
commit 4d4cc68f0d
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c
View File

@ -3009,7 +3009,7 @@ static int __devinit igb_probe(struct pci_dev *pdev,
case e1000_i211: case e1000_i211:
/* Enable EEE for internal copper PHY devices */ /* Enable EEE for internal copper PHY devices */
err = e1000_set_eee_i350(hw); err = e1000_set_eee_i350(hw);
if ((!err) && if (!err &&
(adapter->flags & IGB_FLAG_EEE)) (adapter->flags & IGB_FLAG_EEE))
adapter->eee_advert = adapter->eee_advert =
MDIO_EEE_100TX | MDIO_EEE_1000T; MDIO_EEE_100TX | MDIO_EEE_1000T;
@ -5022,7 +5022,7 @@ static void igb_update_itr(struct igb_q_vector *q_vector,
itrval = bulk_latency; itrval = bulk_latency;
} else if ((packets < 10) || ((bytes/packets) > 1200)) { } else if ((packets < 10) || ((bytes/packets) > 1200)) {
itrval = bulk_latency; itrval = bulk_latency;
} else if ((packets > 35)) { } else if (packets > 35) {
itrval = lowest_latency; itrval = lowest_latency;
} }
} else if (bytes/packets > 2000) { } else if (bytes/packets > 2000) {

6
lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_common.c
View File

@ -2415,7 +2415,7 @@ s32 ixgbe_fc_enable_generic(struct ixgbe_hw *hw)
static s32 ixgbe_negotiate_fc(struct ixgbe_hw *hw, u32 adv_reg, u32 lp_reg, static s32 ixgbe_negotiate_fc(struct ixgbe_hw *hw, u32 adv_reg, u32 lp_reg,
u32 adv_sym, u32 adv_asm, u32 lp_sym, u32 lp_asm) u32 adv_sym, u32 adv_asm, u32 lp_sym, u32 lp_asm)
{ {
if ((!(adv_reg)) || (!(lp_reg))) if (!adv_reg || !lp_reg)
return IXGBE_ERR_FC_NOT_NEGOTIATED; return IXGBE_ERR_FC_NOT_NEGOTIATED;
if ((adv_reg & adv_sym) && (lp_reg & lp_sym)) { if ((adv_reg & adv_sym) && (lp_reg & lp_sym)) {
@ -3279,7 +3279,7 @@ s32 ixgbe_set_vfta_generic(struct ixgbe_hw *hw, u32 vlan, u32 vind,
vfta_changed = true; vfta_changed = true;
} }
} else { } else {
if ((vfta & targetbit)) { if (vfta & targetbit) {
vfta &= ~targetbit; vfta &= ~targetbit;
vfta_changed = true; vfta_changed = true;
} }
@ -3392,7 +3392,7 @@ s32 ixgbe_set_vlvf_generic(struct ixgbe_hw *hw, u32 vlan, u32 vind,
if (bits) { if (bits) {
IXGBE_WRITE_REG(hw, IXGBE_VLVF(vlvf_index), IXGBE_WRITE_REG(hw, IXGBE_VLVF(vlvf_index),
(IXGBE_VLVF_VIEN | vlan)); (IXGBE_VLVF_VIEN | vlan));
if ((!vlan_on) && (vfta_changed != NULL)) { if (!vlan_on && (vfta_changed != NULL)) {
/* someone wants to clear the vfta entry /* someone wants to clear the vfta entry
* but some pools/VFs are still using it. * but some pools/VFs are still using it.
* Ignore it. */ * Ignore it. */