staging: comedi: mite: make mite_ack_linkc() handle mite_sync_dma()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 21 Apr 2016 19:04:44 +0000 (12:04 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 29 Apr 2016 05:19:49 +0000 (22:19 -0700)
The mite dma is always synced on a LINKC status. Some of the mite users
sync the dma regadless of the status.

Add a 'sync' parameter to mite_ack_linkc() to force a dma sync. Then do
the dma sync as needed.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/mite.c
drivers/staging/comedi/drivers/mite.h
drivers/staging/comedi/drivers/ni_mio_common.c
drivers/staging/comedi/drivers/ni_pcidio.c
drivers/staging/comedi/drivers/ni_tiocmd.c

index d97059217825cce44a7d85d4e1b4442d9fece4e0..b8cf898870065f2602c57d63097fc752f35c111c 100644 (file)
@@ -779,15 +779,21 @@ static unsigned int mite_get_status(struct mite_channel *mite_chan)
 }
 
 unsigned int mite_ack_linkc(struct mite_channel *mite_chan,
-                           struct comedi_subdevice *s)
+                           struct comedi_subdevice *s,
+                           bool sync)
 {
        struct mite_struct *mite = mite_chan->mite;
        unsigned int status;
 
        status = mite_get_status(mite_chan);
-       if (status & CHSR_LINKC)
+       if (status & CHSR_LINKC) {
                writel(CHOR_CLRLC,
                       mite->mite_io_addr + MITE_CHOR(mite_chan->channel));
+               sync = true;
+       }
+       if (sync)
+               mite_sync_dma(mite_chan, s);
+
        if (status & CHSR_XFERR) {
                dev_err(s->device->class_dev,
                        "mite: transfer error %08x\n", status);
index f8364edf8ce88e2951c92d6b4c21d6ea271ff396..bf6128abc464cb6d298f852c80a4168fa8e1d543 100644 (file)
@@ -93,8 +93,8 @@ void mite_dma_arm(struct mite_channel *mite_chan);
 void mite_dma_disarm(struct mite_channel *mite_chan);
 void mite_sync_dma(struct mite_channel *mite_chan, struct comedi_subdevice *s);
 u32 mite_bytes_in_transit(struct mite_channel *mite_chan);
-unsigned int mite_ack_linkc(struct mite_channel *mite_chan,
-                           struct comedi_subdevice *s);
+unsigned int mite_ack_linkc(struct mite_channel *, struct comedi_subdevice *,
+                           bool sync);
 int mite_done(struct mite_channel *mite_chan);
 
 void mite_prep_dma(struct mite_channel *mite_chan,
index a3b93c1adf344897d249a5737bcc34f9e614c333..dc024f27cc48417c3f4bef63ab809d90e87c48b2 100644 (file)
@@ -3632,10 +3632,8 @@ static void handle_cdio_interrupt(struct comedi_device *dev)
        unsigned long flags;
 
        spin_lock_irqsave(&devpriv->mite_channel_lock, flags);
-       if (devpriv->cdo_mite_chan) {
-               mite_ack_linkc(devpriv->cdo_mite_chan, s);
-               mite_sync_dma(devpriv->cdo_mite_chan, s);
-       }
+       if (devpriv->cdo_mite_chan)
+               mite_ack_linkc(devpriv->cdo_mite_chan, s, true);
        spin_unlock_irqrestore(&devpriv->mite_channel_lock, flags);
 
        cdio_status = ni_readl(dev, NI_M_CDIO_STATUS_REG);
@@ -5150,21 +5148,12 @@ static irqreturn_t ni_E_interrupt(int irq, void *d)
 #ifdef PCIDMA
        if (devpriv->mite) {
                unsigned long flags_too;
-               unsigned int m_status;
 
                spin_lock_irqsave(&devpriv->mite_channel_lock, flags_too);
-               if (s_ai && devpriv->ai_mite_chan) {
-                       m_status = mite_ack_linkc(devpriv->ai_mite_chan, s_ai);
-                       if (m_status & CHSR_LINKC)
-                               mite_sync_dma(devpriv->ai_mite_chan, s_ai);
-               }
-
-               if (s_ao && devpriv->ao_mite_chan) {
-                       m_status = mite_ack_linkc(devpriv->ao_mite_chan, s_ao);
-                       if (m_status & CHSR_LINKC)
-                               mite_sync_dma(devpriv->ao_mite_chan, s_ao);
-               }
-
+               if (s_ai && devpriv->ai_mite_chan)
+                       mite_ack_linkc(devpriv->ai_mite_chan, s_ai, false);
+               if (s_ao && devpriv->ao_mite_chan)
+                       mite_ack_linkc(devpriv->ao_mite_chan, s_ao, false);
                spin_unlock_irqrestore(&devpriv->mite_channel_lock, flags_too);
        }
 #endif
index b4c6bab57a26077701d8ff98b3ce04f46ae65b80..4a34da5140e0ab130f1c6a609cdc100f76a76df3 100644 (file)
@@ -400,13 +400,8 @@ static irqreturn_t nidio_interrupt(int irq, void *d)
 
        spin_lock(&devpriv->mite_channel_lock);
        if (devpriv->di_mite_chan) {
-               unsigned int m_status = mite_ack_linkc(devpriv->di_mite_chan,
-                                                      s);
-
-               if (m_status & CHSR_LINKC) {
-                       mite_sync_dma(devpriv->di_mite_chan, s);
-                       /* XXX need to byteswap */
-               }
+               mite_ack_linkc(devpriv->di_mite_chan, s, false);
+               /* XXX need to byteswap sync'ed dma */
        }
        spin_unlock(&devpriv->mite_channel_lock);
 
index 5228b9fc2a720ebdce329ad4304300e3c584daa8..9007c57544bf4f69d92c8dac8abfc0a0df4aedd6 100644 (file)
@@ -428,10 +428,8 @@ void ni_tio_handle_interrupt(struct ni_gpct *counter,
                break;
        }
        spin_lock_irqsave(&counter->lock, flags);
-       if (counter->mite_chan) {
-               mite_ack_linkc(counter->mite_chan, s);
-               mite_sync_dma(counter->mite_chan, s);
-       }
+       if (counter->mite_chan)
+               mite_ack_linkc(counter->mite_chan, s, true);
        spin_unlock_irqrestore(&counter->lock, flags);
 }
 EXPORT_SYMBOL_GPL(ni_tio_handle_interrupt);