staging: comedi: ni_660x: remove inline mite_ring()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 22 Mar 2016 18:10:48 +0000 (11:10 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
This fuction just returns a pointer from the private data. The name
might provide some confusion since it appears to be an exported
function from the mite driver.

Just remove it and get the pointer directly where 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/ni_660x.c

index cf25892d92b24717912e4f556ae8075174aa98a7..1cca9ea5cdd2f8d370abc441625b838ade179df4 100644 (file)
@@ -465,16 +465,6 @@ static unsigned int ni_660x_gpct_read(struct ni_gpct *counter,
        return ni_660x_read(dev, counter->chip_index, ni_660x_register);
 }
 
-static inline struct mite_dma_descriptor_ring *mite_ring(struct ni_660x_private
-                                                        *priv,
-                                                        struct ni_gpct
-                                                        *counter)
-{
-       unsigned int chip = counter->chip_index;
-
-       return priv->mite_rings[chip][counter->counter_index];
-}
-
 static inline void ni_660x_set_dma_channel(struct comedi_device *dev,
                                           unsigned int mite_channel,
                                           struct ni_gpct *counter)
@@ -515,12 +505,13 @@ static int ni_660x_request_mite_channel(struct comedi_device *dev,
                                        enum comedi_io_direction direction)
 {
        struct ni_660x_private *devpriv = dev->private;
-       unsigned long flags;
+       struct mite_dma_descriptor_ring *ring;
        struct mite_channel *mite_chan;
+       unsigned long flags;
 
        spin_lock_irqsave(&devpriv->mite_channel_lock, flags);
-       mite_chan = mite_request_channel(devpriv->mite,
-                                        mite_ring(devpriv, counter));
+       ring = devpriv->mite_rings[counter->chip_index][counter->counter_index];
+       mite_chan = mite_request_channel(devpriv->mite, ring);
        if (!mite_chan) {
                spin_unlock_irqrestore(&devpriv->mite_channel_lock, flags);
                dev_err(dev->class_dev,
@@ -645,9 +636,11 @@ static int ni_660x_buf_change(struct comedi_device *dev,
 {
        struct ni_660x_private *devpriv = dev->private;
        struct ni_gpct *counter = s->private;
+       struct mite_dma_descriptor_ring *ring;
        int ret;
 
-       ret = mite_buf_change(mite_ring(devpriv, counter), s);
+       ring = devpriv->mite_rings[counter->chip_index][counter->counter_index];
+       ret = mite_buf_change(ring, s);
        if (ret < 0)
                return ret;