[PATCH 07/40] staging: comedi: me4000: remove struct me4000_ao_info

H Hartley Sweeten hartleys at visionengravers.com
Sat Sep 8 00:40:25 UTC 2012


The me4000_ao_info in the boardinfo struct is used to indicate
the number of analog output channels and a couple other details
about them. Remove the extra struct and absorb the data into the
boardinfo struct.

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>
---
 drivers/staging/comedi/drivers/me4000.c | 48 +++++++++++----------------------
 drivers/staging/comedi/drivers/me4000.h |  8 ++----
 2 files changed, 18 insertions(+), 38 deletions(-)

diff --git a/drivers/staging/comedi/drivers/me4000.c b/drivers/staging/comedi/drivers/me4000.c
index cd001c3..3ffaa39 100644
--- a/drivers/staging/comedi/drivers/me4000.c
+++ b/drivers/staging/comedi/drivers/me4000.c
@@ -103,9 +103,7 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4670",
 		.device_id	= 0x4670,
-		.ao		= {
-			.count		= 4,
-		},
+		.ao_nchan	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ex_trig_analog	= 1,
@@ -114,9 +112,7 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4670i",
 		.device_id	= 0x4671,
-		.ao		= {
-			.count		= 4,
-		},
+		.ao_nchan	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ex_trig_analog	= 1,
@@ -125,9 +121,7 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4670s",
 		.device_id	= 0x4672,
-		.ao		= {
-			.count		= 4,
-		},
+		.ao_nchan	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ai_sh_nchan	= 8,
@@ -137,9 +131,7 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4670is",
 		.device_id	= 0x4673,
-		.ao		= {
-			.count		= 4,
-		},
+		.ao_nchan	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ai_sh_nchan	= 8,
@@ -149,10 +141,8 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4680",
 		.device_id	= 0x4680,
-		.ao		= {
-			.count		= 4,
-			.fifo_count	= 4,
-		},
+		.ao_nchan	= 4,
+		.ao_fifo	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ex_trig_analog	= 1,
@@ -161,10 +151,8 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4680i",
 		.device_id	= 0x4681,
-		.ao		= {
-			.count		= 4,
-			.fifo_count	= 4,
-		},
+		.ao_nchan	= 4,
+		.ao_fifo	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ex_trig_analog	= 1,
@@ -173,10 +161,8 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4680s",
 		.device_id	= 0x4682,
-		.ao		= {
-			.count		= 4,
-			.fifo_count	= 4,
-		},
+		.ao_nchan	= 4,
+		.ao_fifo	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ai_sh_nchan	= 8,
@@ -186,10 +172,8 @@ static const struct me4000_board me4000_boards[] = {
 	}, {
 		.name		= "ME-4680is",
 		.device_id	= 0x4683,
-		.ao		= {
-			.count		= 4,
-			.fifo_count	= 4,
-		},
+		.ao_nchan	= 4,
+		.ao_fifo	= 4,
 		.ai_nchan	= 32,
 		.ai_diff_nchan	= 16,
 		.ai_sh_nchan	= 8,
@@ -463,7 +447,7 @@ static int init_ao_context(struct comedi_device *dev)
 {
 	int i;
 
-	for (i = 0; i < thisboard->ao.count; i++) {
+	for (i = 0; i < thisboard->ao_nchan; i++) {
 		/* spin_lock_init(&info->ao_context[i].use_lock); */
 		info->ao_context[i].irq = info->irq;
 
@@ -1689,7 +1673,7 @@ static int me4000_ao_insn_write(struct comedi_device *dev,
 		return -EINVAL;
 	}
 
-	if (chan >= thisboard->ao.count) {
+	if (chan >= thisboard->ao_nchan) {
 		printk(KERN_ERR
 		       "comedi%d: me4000: me4000_ao_insn_write(): "
 		       "Invalid channel %d\n", dev->minor, insn->n);
@@ -2169,10 +2153,10 @@ static int me4000_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 
 	s = &dev->subdevices[1];
 
-	if (thisboard->ao.count) {
+	if (thisboard->ao_nchan) {
 		s->type = COMEDI_SUBD_AO;
 		s->subdev_flags = SDF_WRITEABLE | SDF_COMMON | SDF_GROUND;
-		s->n_chan = thisboard->ao.count;
+		s->n_chan = thisboard->ao_nchan;
 		s->maxdata = 0xFFFF;	/*  16 bit DAC */
 		s->range_table = &me4000_ao_range;
 		s->insn_write = me4000_ao_insn_write;
diff --git a/drivers/staging/comedi/drivers/me4000.h b/drivers/staging/comedi/drivers/me4000.h
index cbf8c6b..ab6094a 100644
--- a/drivers/staging/comedi/drivers/me4000.h
+++ b/drivers/staging/comedi/drivers/me4000.h
@@ -256,15 +256,11 @@
   Information about the hardware capabilities
   ===========================================================================*/
 
-struct me4000_ao_info {
-	int count;
-	int fifo_count;
-};
-
 struct me4000_board {
 	const char *name;
 	unsigned short device_id;
-	struct me4000_ao_info ao;
+	int ao_nchan;
+	int ao_fifo;
 	int ai_nchan;
 	int ai_diff_nchan;
 	int ai_sh_nchan;
-- 
1.7.11




More information about the devel mailing list