diff options
| author |  Jesse Brandeburg <jesse.brandeburg@intel.com>
| 2021-03-25 17:38:31 -0700 |
|---|---|---|
| committer |  Tony Nguyen <anthony.l.nguyen@intel.com>
| 2021-05-26 09:11:41 -0700 |
| commit | de8447131d2b1923a91c4c30bf094422dfcc16bf (patch) | |
| tree | f926da0d068c234400539c21d7588f184f7227de /drivers/net/ethernet/intel/ixgbevf | |
| parent | igb: override two checker warnings (diff) | |
| download | linux-de8447131d2b1923a91c4c30bf094422dfcc16bf.tar.gz linux-de8447131d2b1923a91c4c30bf094422dfcc16bf.tar.bz2 linux-de8447131d2b1923a91c4c30bf094422dfcc16bf.zip | |
intel: call csum functions with well formatted arguments
The sparse build (C=2) found that there were two drivers
who had not been convered to call the csum_replace_by_diff() function
with sparse clean arguments. Most if not all drivers force the cast
like this patch does. So these drivers are now joining the party
(a bit late), but with no functional change.
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/ixgbevf')
| -rw-r--r-- | drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c index ba2ed8a43d2d..588c3aa50d94 100644 --- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c +++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c @@ -3814,7 +3814,7 @@ static int ixgbevf_tso(struct ixgbevf_ring *tx_ring, /* remove payload length from inner checksum */ paylen = skb->len - l4_offset; - csum_replace_by_diff(&l4.tcp->check, htonl(paylen)); + csum_replace_by_diff(&l4.tcp->check, (__force __wsum)htonl(paylen)); /* update gso size and bytecount with header size */ first->gso_segs = skb_shinfo(skb)->gso_segs; |