On 2/19/2019 1:24 PM, Andrius Sirvys wrote: > Shifting signed 32-bit values by 31-bits has the potential for > unexpected outcomes as compiler can overwrite a bit. > Specified that values are unsigned. > > Errors are observed from running cppcheck. > > Bugzilla ID: 58 > Fixes: 69e209be5464 ("net/axgbe: add register map and related macros") > Fixes: b5bf7719221d ("bnx2x: driver support routines") > Fixes: ed2ced6fe927 ("net/bnxt: check initialization before accessing stats") > Fixes: 6fda3f0ddda9 ("net/cxgbe: add API to program hardware MPS table") > Fixes: bdb244b96920 ("e1000: whitespace changes") > Fixes: 5a32a257f957 ("e1000: more NICs in base driver") > Fixes: 2fe669f4bcd2 ("net/nfp: support MAC address change") > Fixes: defb9a5dd156 ("nfp: introduce driver initialization") > Fixes: ec94dbc57362 ("qede: add base driver") > Fixes: d2e7d931d0ad ("net/qede/base: formatting changes") > Fixes: cdc07e83bb24 ("net/tap: add eBPF program file") > Cc: sta...@dpdk.org > > Signed-off-by: Andrius Sirvys <andrius.sir...@intel.com>
Reviewed-by: Ferruh Yigit <ferruh.yi...@intel.com>