[PATCH 074/108] staging: comedi: addi_apci_3120: remove private data 'b_InterruptMode'
H Hartley Sweeten
hsweeten at visionengravers.com
Tue Nov 4 17:54:58 UTC 2014
This member of the private data is not really needed. The interrupt mode
can be determined by checking the devpriv->mode value to see if the
interrupt is enabled. Remove the unnecessary member.
Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
Cc: Ian Abbott <abbotti at mev.co.uk>
Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
---
.../staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 18 +++---------------
drivers/staging/comedi/drivers/addi_apci_3120.c | 1 -
2 files changed, 3 insertions(+), 16 deletions(-)
diff --git a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c
index c7f6130..84600d4 100644
--- a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c
+++ b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c
@@ -74,10 +74,6 @@ This program is distributed in the hope that it will be useful, but WITHOUT ANY
#define APCI3120_START 1
#define APCI3120_STOP 0
-#define APCI3120_EOC_MODE 1
-#define APCI3120_EOS_MODE 2
-#define APCI3120_DMA_MODE 3
-
#define APCI3120_RD_FIFO 0x00
/* software trigger dummy register */
@@ -97,8 +93,6 @@ static int apci3120_reset(struct comedi_device *dev)
{
struct apci3120_private *devpriv = dev->private;
- devpriv->b_InterruptMode = APCI3120_EOC_MODE;
-
/* variables used in timer subdevice */
devpriv->b_Timer2Mode = 0;
devpriv->b_Timer2Interrupt = 0;
@@ -145,8 +139,6 @@ static int apci3120_cancel(struct comedi_device *dev,
inw(dev->iobase + APCI3120_STATUS_REG);
devpriv->ui_DmaActualBuffer = 0;
- devpriv->b_InterruptMode = APCI3120_EOC_MODE;
-
return 0;
}
@@ -413,11 +405,8 @@ static int apci3120_ai_cmd(struct comedi_device *dev,
apci3120_timer_write(dev, 0, divisor);
if (devpriv->us_UseDma) {
- devpriv->b_InterruptMode = APCI3120_DMA_MODE;
apci3120_setup_dma(dev, s);
} else {
- devpriv->b_InterruptMode = APCI3120_EOS_MODE;
-
devpriv->mode |= APCI3120_MODE_EOS_IRQ_ENA;
if (cmd->stop_src == TRIG_COUNT) {
@@ -615,12 +604,12 @@ static irqreturn_t apci3120_interrupt(int irq, void *d)
dev_err(dev->class_dev, "AMCC IRQ - TARGET DMA ABORT!\n");
if ((status & APCI3120_STATUS_EOC_INT) == 0 &&
- devpriv->b_InterruptMode == APCI3120_EOC_MODE) {
+ (devpriv->mode & APCI3120_MODE_EOC_IRQ_ENA)) {
/* nothing to do... EOC mode is not currently used */
}
if ((status & APCI3120_STATUS_EOS_INT) &&
- devpriv->b_InterruptMode == APCI3120_EOS_MODE) {
+ (devpriv->mode & APCI3120_MODE_EOS_IRQ_ENA)) {
unsigned short val;
int i;
@@ -663,8 +652,7 @@ static irqreturn_t apci3120_interrupt(int irq, void *d)
apci3120_clr_timer2_interrupt(dev);
}
- if ((status & APCI3120_STATUS_AMCC_INT) &&
- devpriv->b_InterruptMode == APCI3120_DMA_MODE) {
+ if (status & APCI3120_STATUS_AMCC_INT) {
/* Clear Timer Write TC int */
outl(APCI3120_CLEAR_WRITE_TC_INT,
devpriv->amcc + APCI3120_AMCC_OP_REG_INTCSR);
diff --git a/drivers/staging/comedi/drivers/addi_apci_3120.c b/drivers/staging/comedi/drivers/addi_apci_3120.c
index ec66233..a6ae42f 100644
--- a/drivers/staging/comedi/drivers/addi_apci_3120.c
+++ b/drivers/staging/comedi/drivers/addi_apci_3120.c
@@ -133,7 +133,6 @@ struct apci3120_private {
unsigned short ctrl;
unsigned char b_Timer2Mode;
unsigned char b_Timer2Interrupt;
- unsigned char b_InterruptMode;
unsigned char b_ExttrigEnable;
struct task_struct *tsk_Current;
};
--
2.0.3
More information about the devel
mailing list