diff options
| author |   Heiner Kallweit <hkallweit1@gmail.com> | 2019-06-06 07:49:17 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-06-06 10:23:31 -0700 |
| commit | 355f948adf3aedb1b2f1644ce5bbefe39177e28a (patch) | |
| tree | 51a637c74aaf204ad6b5843ef0bd44edccfb2524 /drivers | |
| parent | Merge branch 'tcp-flowlabel' (diff) | |
| download | linux-dev-355f948adf3aedb1b2f1644ce5bbefe39177e28a.tar.xz linux-dev-355f948adf3aedb1b2f1644ce5bbefe39177e28a.zip | |
r8169: silence sparse warning in rtl8169_start_xmit
The opts[] array is of type u32. Therefore remove the wrong
cpu_to_le32(). The opts[] array members are converted to little endian
later when being assigned to the respective descriptor fields.
This is not a new issue, it just popped up due to r8169.c having
been renamed and more thoroughly checked. Due to the renaming
this patch applies to net-next only.
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/realtek/r8169_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c index d34cc855f7db..8b7d45ff1d03 100644 --- a/drivers/net/ethernet/realtek/r8169_main.c +++ b/drivers/net/ethernet/realtek/r8169_main.c @@ -5621,7 +5621,7 @@ static netdev_tx_t rtl8169_start_xmit(struct sk_buff *skb, if (unlikely(le32_to_cpu(txd->opts1) & DescOwn)) goto err_stop_0; - opts[1] = cpu_to_le32(rtl8169_tx_vlan_tag(skb)); + opts[1] = rtl8169_tx_vlan_tag(skb); opts[0] = DescOwn; if (rtl_chip_supports_csum_v2(tp)) { |