diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2018-08-09 09:28:52 -0400 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2018-08-23 12:24:27 -0400 |
commit | 6efa6239e7f8777dccddcebb643e6d9a0304bf43 (patch) | |
tree | cb7350facda20e73a1571f392511aa43fe3e300b /drivers/net/ethernet/intel/ice/ice_sched.c | |
parent | 4381147df9098706caa5cf9fda37e53b2fe4871f (diff) |
ice: Remove unnecessary node owner check
There is already a check for owner == ICE_SCHED_NODE_OWNER_LAN at the
beginning of ice_sched_update_vsi_child_nodes. Remove the additional
check to address the static analysis tool smatch issue "warn: we tested
'owner' before and it was 'false'".
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Anirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Tested-by: Tony Brelinski <tonyx.brelinski@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/ice/ice_sched.c')
-rw-r--r-- | drivers/net/ethernet/intel/ice/ice_sched.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/intel/ice/ice_sched.c b/drivers/net/ethernet/intel/ice/ice_sched.c index 2e6c1d92cc88..eeae199469b6 100644 --- a/drivers/net/ethernet/intel/ice/ice_sched.c +++ b/drivers/net/ethernet/intel/ice/ice_sched.c | |||
@@ -1576,8 +1576,7 @@ ice_sched_update_vsi_child_nodes(struct ice_port_info *pi, u16 vsi_id, u8 tc, | |||
1576 | return status; | 1576 | return status; |
1577 | } | 1577 | } |
1578 | 1578 | ||
1579 | if (owner == ICE_SCHED_NODE_OWNER_LAN) | 1579 | vsi->max_lanq[tc] = new_numqs; |
1580 | vsi->max_lanq[tc] = new_numqs; | ||
1581 | 1580 | ||
1582 | return status; | 1581 | return status; |
1583 | } | 1582 | } |