Commit 95ebcea6 authored by Joachim Eastwood's avatar Joachim Eastwood Committed by David S. Miller

net/macb: fix truncate warnings

When building macb on x86_64 the following warnings show up:
  drivers/net/ethernet/cadence/macb.c: In function macb_interrupt:
  drivers/net/ethernet/cadence/macb.c:556:4: warning: large integer implicitly truncated to unsigned type [-Woverflow]
  drivers/net/ethernet/cadence/macb.c: In function macb_reset_hw:
  drivers/net/ethernet/cadence/macb.c:792:2: warning: large integer implicitly truncated to unsigned type [-Woverflow]
  drivers/net/ethernet/cadence/macb.c:793:2: warning: large integer implicitly truncated to unsigned type [-Woverflow]
  drivers/net/ethernet/cadence/macb.c:796:2: warning: large integer implicitly truncated to unsigned type [-Woverflow]

Use -1 insted of ~0UL, as done in other places in the driver,
to silence these warnings.
Signed-off-by: default avatarJoachim Eastwood <manabian@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 51ebd318
...@@ -553,7 +553,7 @@ static irqreturn_t macb_interrupt(int irq, void *dev_id) ...@@ -553,7 +553,7 @@ static irqreturn_t macb_interrupt(int irq, void *dev_id)
while (status) { while (status) {
/* close possible race with dev_close */ /* close possible race with dev_close */
if (unlikely(!netif_running(dev))) { if (unlikely(!netif_running(dev))) {
macb_writel(bp, IDR, ~0UL); macb_writel(bp, IDR, -1);
break; break;
} }
...@@ -789,11 +789,11 @@ static void macb_reset_hw(struct macb *bp) ...@@ -789,11 +789,11 @@ static void macb_reset_hw(struct macb *bp)
macb_writel(bp, NCR, MACB_BIT(CLRSTAT)); macb_writel(bp, NCR, MACB_BIT(CLRSTAT));
/* Clear all status flags */ /* Clear all status flags */
macb_writel(bp, TSR, ~0UL); macb_writel(bp, TSR, -1);
macb_writel(bp, RSR, ~0UL); macb_writel(bp, RSR, -1);
/* Disable all interrupts */ /* Disable all interrupts */
macb_writel(bp, IDR, ~0UL); macb_writel(bp, IDR, -1);
macb_readl(bp, ISR); macb_readl(bp, ISR);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment