Commit 81b504b81437457614ce44ce0dac48038d35f859

Authored by stephen hemminger
Committed by David S. Miller
1 parent c3f52ae6a3

atl1[ce]: fix sparse warnings

The dmaw_block is an enum and max_pay_load is u32. Therefore
sparse gives warning about comparison of unsigned and signed value.
Resolve by using min_t to force cast.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

Showing 2 changed files with 4 additions and 4 deletions Side-by-side Diff

drivers/net/atl1c/atl1c_main.c
... ... @@ -1102,10 +1102,10 @@
1102 1102 AT_READ_REG(hw, REG_DEVICE_CTRL, &dev_ctrl_data);
1103 1103 max_pay_load = (dev_ctrl_data >> DEVICE_CTRL_MAX_PAYLOAD_SHIFT) &
1104 1104 DEVICE_CTRL_MAX_PAYLOAD_MASK;
1105   - hw->dmaw_block = min(max_pay_load, hw->dmaw_block);
  1105 + hw->dmaw_block = min_t(u32, max_pay_load, hw->dmaw_block);
1106 1106 max_pay_load = (dev_ctrl_data >> DEVICE_CTRL_MAX_RREQ_SZ_SHIFT) &
1107 1107 DEVICE_CTRL_MAX_RREQ_SZ_MASK;
1108   - hw->dmar_block = min(max_pay_load, hw->dmar_block);
  1108 + hw->dmar_block = min_t(u32, max_pay_load, hw->dmar_block);
1109 1109  
1110 1110 txq_ctrl_data = (hw->tpd_burst & TXQ_NUM_TPD_BURST_MASK) <<
1111 1111 TXQ_NUM_TPD_BURST_SHIFT;
drivers/net/atl1e/atl1e_main.c
... ... @@ -932,11 +932,11 @@
932 932 max_pay_load = ((dev_ctrl_data >> DEVICE_CTRL_MAX_PAYLOAD_SHIFT)) &
933 933 DEVICE_CTRL_MAX_PAYLOAD_MASK;
934 934  
935   - hw->dmaw_block = min(max_pay_load, hw->dmaw_block);
  935 + hw->dmaw_block = min_t(u32, max_pay_load, hw->dmaw_block);
936 936  
937 937 max_pay_load = ((dev_ctrl_data >> DEVICE_CTRL_MAX_RREQ_SZ_SHIFT)) &
938 938 DEVICE_CTRL_MAX_RREQ_SZ_MASK;
939   - hw->dmar_block = min(max_pay_load, hw->dmar_block);
  939 + hw->dmar_block = min_t(u32, max_pay_load, hw->dmar_block);
940 940  
941 941 if (hw->nic_type != athr_l2e_revB)
942 942 AT_WRITE_REGW(hw, REG_TXQ_CTRL + 2,