diff options
author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-06-18 16:26:05 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-06-24 18:51:04 -0400 |
commit | b07839c09b5235caae99ba2f8c24968a9115707c (patch) | |
tree | 31599c9afe3428cb492bb15c7292a03736eb907d | |
parent | 1fdd0fc5fe28cd11ecf64e77892a1d9a1fd89fbf (diff) |
staging: comedi: pcmuio: cleanup remaining #define's
Remove all the unused #define's and add namespace to the ones that
are used.
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>
-rw-r--r-- | drivers/staging/comedi/drivers/pcmuio.c | 37 |
1 files changed, 12 insertions, 25 deletions
diff --git a/drivers/staging/comedi/drivers/pcmuio.c b/drivers/staging/comedi/drivers/pcmuio.c index d1f5fc024089..b88b24d290e8 100644 --- a/drivers/staging/comedi/drivers/pcmuio.c +++ b/drivers/staging/comedi/drivers/pcmuio.c | |||
@@ -108,21 +108,8 @@ | |||
108 | #define PCMUIO_PAGE_INT_ID 3 | 108 | #define PCMUIO_PAGE_INT_ID 3 |
109 | #define PCMUIO_PAGE_REG(x) (0x08 + (x)) | 109 | #define PCMUIO_PAGE_REG(x) (0x08 + (x)) |
110 | 110 | ||
111 | #define CHANS_PER_PORT 8 | 111 | #define PCMUIO_ASIC_IOSIZE 0x10 |
112 | #define PORTS_PER_ASIC 6 | 112 | #define PCMUIO_MAX_ASICS 2 |
113 | /* number of channels per comedi subdevice */ | ||
114 | #define MAX_CHANS_PER_SUBDEV 24 | ||
115 | #define PORTS_PER_SUBDEV (MAX_CHANS_PER_SUBDEV / CHANS_PER_PORT) | ||
116 | #define CHANS_PER_ASIC (CHANS_PER_PORT * PORTS_PER_ASIC) | ||
117 | #define INTR_CHANS_PER_ASIC 24 | ||
118 | #define INTR_PORTS_PER_SUBDEV (INTR_CHANS_PER_ASIC / CHANS_PER_PORT) | ||
119 | #define MAX_DIO_CHANS (PORTS_PER_ASIC * 2 * CHANS_PER_PORT) | ||
120 | #define MAX_ASICS (MAX_DIO_CHANS / CHANS_PER_ASIC) | ||
121 | |||
122 | /* IO Memory sizes */ | ||
123 | #define ASIC_IOSIZE 0x10 | ||
124 | #define PCMUIO48_IOSIZE ASIC_IOSIZE | ||
125 | #define PCMUIO96_IOSIZE (ASIC_IOSIZE * 2) | ||
126 | 113 | ||
127 | struct pcmuio_board { | 114 | struct pcmuio_board { |
128 | const char *name; | 115 | const char *name; |
@@ -160,14 +147,14 @@ struct pcmuio_private { | |||
160 | struct { | 147 | struct { |
161 | unsigned int irq; | 148 | unsigned int irq; |
162 | spinlock_t spinlock; | 149 | spinlock_t spinlock; |
163 | } asics[MAX_ASICS]; | 150 | } asics[PCMUIO_MAX_ASICS]; |
164 | struct pcmuio_subdev_private *sprivs; | 151 | struct pcmuio_subdev_private *sprivs; |
165 | }; | 152 | }; |
166 | 153 | ||
167 | static void pcmuio_write(struct comedi_device *dev, unsigned int val, | 154 | static void pcmuio_write(struct comedi_device *dev, unsigned int val, |
168 | int asic, int page, int port) | 155 | int asic, int page, int port) |
169 | { | 156 | { |
170 | unsigned long iobase = dev->iobase + (asic * ASIC_IOSIZE); | 157 | unsigned long iobase = dev->iobase + (asic * PCMUIO_ASIC_IOSIZE); |
171 | 158 | ||
172 | if (page == 0) { | 159 | if (page == 0) { |
173 | /* Port registers are valid for any page */ | 160 | /* Port registers are valid for any page */ |
@@ -185,7 +172,7 @@ static void pcmuio_write(struct comedi_device *dev, unsigned int val, | |||
185 | static unsigned int pcmuio_read(struct comedi_device *dev, | 172 | static unsigned int pcmuio_read(struct comedi_device *dev, |
186 | int asic, int page, int port) | 173 | int asic, int page, int port) |
187 | { | 174 | { |
188 | unsigned long iobase = dev->iobase + (asic * ASIC_IOSIZE); | 175 | unsigned long iobase = dev->iobase + (asic * PCMUIO_ASIC_IOSIZE); |
189 | unsigned int val; | 176 | unsigned int val; |
190 | 177 | ||
191 | if (page == 0) { | 178 | if (page == 0) { |
@@ -368,7 +355,7 @@ static int pcmuio_handle_asic_interrupt(struct comedi_device *dev, int asic) | |||
368 | { | 355 | { |
369 | struct pcmuio_private *devpriv = dev->private; | 356 | struct pcmuio_private *devpriv = dev->private; |
370 | struct pcmuio_subdev_private *subpriv; | 357 | struct pcmuio_subdev_private *subpriv; |
371 | unsigned long iobase = dev->iobase + (asic * ASIC_IOSIZE); | 358 | unsigned long iobase = dev->iobase + (asic * PCMUIO_ASIC_IOSIZE); |
372 | unsigned int triggered = 0; | 359 | unsigned int triggered = 0; |
373 | int got1 = 0; | 360 | int got1 = 0; |
374 | unsigned long flags; | 361 | unsigned long flags; |
@@ -413,7 +400,7 @@ static irqreturn_t interrupt_pcmuio(int irq, void *d) | |||
413 | int got1 = 0; | 400 | int got1 = 0; |
414 | int asic; | 401 | int asic; |
415 | 402 | ||
416 | for (asic = 0; asic < MAX_ASICS; ++asic) { | 403 | for (asic = 0; asic < PCMUIO_MAX_ASICS; ++asic) { |
417 | if (irq == devpriv->asics[asic].irq) { | 404 | if (irq == devpriv->asics[asic].irq) { |
418 | /* it is an interrupt for ASIC #asic */ | 405 | /* it is an interrupt for ASIC #asic */ |
419 | if (pcmuio_handle_asic_interrupt(dev, asic)) | 406 | if (pcmuio_handle_asic_interrupt(dev, asic)) |
@@ -611,14 +598,14 @@ static int pcmuio_attach(struct comedi_device *dev, struct comedi_devconfig *it) | |||
611 | struct pcmuio_private *devpriv; | 598 | struct pcmuio_private *devpriv; |
612 | struct pcmuio_subdev_private *subpriv; | 599 | struct pcmuio_subdev_private *subpriv; |
613 | int sdev_no, n_subdevs, asic; | 600 | int sdev_no, n_subdevs, asic; |
614 | unsigned int irq[MAX_ASICS]; | 601 | unsigned int irq[PCMUIO_MAX_ASICS]; |
615 | int ret; | 602 | int ret; |
616 | 603 | ||
617 | irq[0] = it->options[1]; | 604 | irq[0] = it->options[1]; |
618 | irq[1] = it->options[2]; | 605 | irq[1] = it->options[2]; |
619 | 606 | ||
620 | ret = comedi_request_region(dev, it->options[0], | 607 | ret = comedi_request_region(dev, it->options[0], |
621 | board->num_asics * ASIC_IOSIZE); | 608 | board->num_asics * PCMUIO_ASIC_IOSIZE); |
622 | if (ret) | 609 | if (ret) |
623 | return ret; | 610 | return ret; |
624 | 611 | ||
@@ -627,7 +614,7 @@ static int pcmuio_attach(struct comedi_device *dev, struct comedi_devconfig *it) | |||
627 | return -ENOMEM; | 614 | return -ENOMEM; |
628 | dev->private = devpriv; | 615 | dev->private = devpriv; |
629 | 616 | ||
630 | for (asic = 0; asic < MAX_ASICS; ++asic) | 617 | for (asic = 0; asic < PCMUIO_MAX_ASICS; ++asic) |
631 | spin_lock_init(&devpriv->asics[asic].spinlock); | 618 | spin_lock_init(&devpriv->asics[asic].spinlock); |
632 | 619 | ||
633 | n_subdevs = board->num_asics * 2; | 620 | n_subdevs = board->num_asics * 2; |
@@ -672,7 +659,7 @@ static int pcmuio_attach(struct comedi_device *dev, struct comedi_devconfig *it) | |||
672 | 659 | ||
673 | init_asics(dev); /* clear out all the registers, basically */ | 660 | init_asics(dev); /* clear out all the registers, basically */ |
674 | 661 | ||
675 | for (asic = 0; irq[0] && asic < MAX_ASICS; ++asic) { | 662 | for (asic = 0; irq[0] && asic < PCMUIO_MAX_ASICS; ++asic) { |
676 | if (irq[asic] | 663 | if (irq[asic] |
677 | && request_irq(irq[asic], interrupt_pcmuio, | 664 | && request_irq(irq[asic], interrupt_pcmuio, |
678 | IRQF_SHARED, board->name, dev)) { | 665 | IRQF_SHARED, board->name, dev)) { |
@@ -705,7 +692,7 @@ static void pcmuio_detach(struct comedi_device *dev) | |||
705 | struct pcmuio_private *devpriv = dev->private; | 692 | struct pcmuio_private *devpriv = dev->private; |
706 | int i; | 693 | int i; |
707 | 694 | ||
708 | for (i = 0; i < MAX_ASICS; ++i) { | 695 | for (i = 0; i < PCMUIO_MAX_ASICS; ++i) { |
709 | if (devpriv->asics[i].irq) | 696 | if (devpriv->asics[i].irq) |
710 | free_irq(devpriv->asics[i].irq, dev); | 697 | free_irq(devpriv->asics[i].irq, dev); |
711 | } | 698 | } |