gianfar: Use gfar_halt to stop DMA in gfar_probe
authorAndy Fleming <afleming@freescale.com>
Tue, 16 Dec 2008 23:25:45 +0000 (15:25 -0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 16 Dec 2008 23:25:45 +0000 (15:25 -0800)
gfar_halt does everything we want to do there, including disabling
TX/RX.  It also doesn't unnecessarily enable DMA if it's already
stopped.

Signed-off-by: Andy Fleming <afleming@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/gianfar.c

index 2635f5bed77f41b6a8242891229e8ca3d4816a75..55e319fa7fe6c902f24584c42c51c4d35ec23692 100644 (file)
@@ -225,19 +225,7 @@ static int gfar_probe(struct platform_device *pdev)
 
        /* Stop the DMA engine now, in case it was running before */
        /* (The firmware could have used it, and left it running). */
-       /* To do this, we write Graceful Receive Stop and Graceful */
-       /* Transmit Stop, and then wait until the corresponding bits */
-       /* in IEVENT indicate the stops have completed. */
-       tempval = gfar_read(&priv->regs->dmactrl);
-       tempval &= ~(DMACTRL_GRS | DMACTRL_GTS);
-       gfar_write(&priv->regs->dmactrl, tempval);
-
-       tempval = gfar_read(&priv->regs->dmactrl);
-       tempval |= (DMACTRL_GRS | DMACTRL_GTS);
-       gfar_write(&priv->regs->dmactrl, tempval);
-
-       while (!(gfar_read(&priv->regs->ievent) & (IEVENT_GRSC | IEVENT_GTSC)))
-               cpu_relax();
+       gfar_halt(dev);
 
        /* Reset MAC layer */
        gfar_write(&priv->regs->maccfg1, MACCFG1_SOFT_RESET);