From 1ddd22c0dfdc1f03e22acae71a39b6be347568be Mon Sep 17 00:00:00 2001 From: H Hartley Sweeten Date: Mon, 17 Feb 2014 14:26:59 -0700 Subject: [PATCH] staging: comedi: pcl816: all board types have 16 analog inputs All the boards supported by this driver have 16 analog input channels. Remove the 'n_aichan' member from the boardinfo and refactor pcl816_attach(). Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcl816.c | 35 ++++++++++--------------- 1 file changed, 14 insertions(+), 21 deletions(-) diff --git a/drivers/staging/comedi/drivers/pcl816.c b/drivers/staging/comedi/drivers/pcl816.c index d1ca4b426de..d64ed5e2685 100644 --- a/drivers/staging/comedi/drivers/pcl816.c +++ b/drivers/staging/comedi/drivers/pcl816.c @@ -93,7 +93,6 @@ static const struct comedi_lrange range_pcl816 = { struct pcl816_board { const char *name; int n_ranges; - int n_aichan; unsigned int ai_ns_min; int n_aochan; const struct comedi_lrange *ai_range_type; @@ -108,7 +107,6 @@ static const struct pcl816_board boardtypes[] = { { .name = "pcl816", .n_ranges = 8, - .n_aichan = 16, .ai_ns_min = 10000, .n_aochan = 1, .ai_range_type = &range_pcl816, @@ -120,7 +118,6 @@ static const struct pcl816_board boardtypes[] = { }, { .name = "pcl814b", .n_ranges = 8, - .n_aichan = 16, .ai_ns_min = 10000, .n_aochan = 1, .ai_range_type = &range_pcl816, @@ -927,24 +924,20 @@ static int pcl816_attach(struct comedi_device *dev, struct comedi_devconfig *it) return ret; s = &dev->subdevices[0]; - if (board->n_aichan > 0) { - s->type = COMEDI_SUBD_AI; - s->subdev_flags = SDF_CMD_READ | SDF_DIFF; - s->n_chan = board->n_aichan; - s->maxdata = board->ai_maxdata; - s->range_table = board->ai_range_type; - s->insn_read = pcl816_ai_insn_read; - if (dev->irq) { - dev->read_subdev = s; - s->subdev_flags |= SDF_CMD_READ; - s->len_chanlist = board->ai_chanlist; - s->do_cmdtest = pcl816_ai_cmdtest; - s->do_cmd = pcl816_ai_cmd; - s->poll = pcl816_ai_poll; - s->cancel = pcl816_ai_cancel; - } - } else { - s->type = COMEDI_SUBD_UNUSED; + s->type = COMEDI_SUBD_AI; + s->subdev_flags = SDF_CMD_READ | SDF_DIFF; + s->n_chan = 16; + s->maxdata = board->ai_maxdata; + s->range_table = board->ai_range_type; + s->insn_read = pcl816_ai_insn_read; + if (dev->irq) { + dev->read_subdev = s; + s->subdev_flags |= SDF_CMD_READ; + s->len_chanlist = board->ai_chanlist; + s->do_cmdtest = pcl816_ai_cmdtest; + s->do_cmd = pcl816_ai_cmd; + s->poll = pcl816_ai_poll; + s->cancel = pcl816_ai_cancel; } #if 0 -- 2.46.0