shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-06-05 02:21:52 +09:00
Code Issues Packages Projects Releases Wiki Activity

staging: comedi: addi_apci_3120: tidy up analog input subdevice init

Browse Source 
For aesthetics, add some white space to the analog input subdevice
initialization.

The async command support in this driver is pretty messed up. I'm not
really sure if it actually works yet. Luckly this driver was not
setting the SDF_CMD_READ subdev_flag so the command support was not
actually getting enabled.

The command support only works if an interrupt is available. Refactor
the subdevice init so that the command support is only hooked up if
we have an interrupt. For now always disable the command support. Once
the driver has been cleaned up the command will be enabled.

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>
This commit is contained in:
H Hartley Sweeten
2014-10-20 11:34:25 -07:00
committed by Greg Kroah-Hartman
parent 1bed58e3b6
commit 2a420f408a
1 changed files with 16 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
drivers/staging/comedi/drivers/addi_apci_3120.c
View File

@@ -168,23 +168,23 @@ static int apci3120_auto_attach(struct comedi_device *dev,
if (ret)
return ret;
/* Allocate and Initialise AI Subdevice Structures */
/* Analog Input subdevice */
s = &dev->subdevices[0];
dev->read_subdev = s;
s->type = COMEDI_SUBD_AI;
s->subdev_flags =
SDF_READABLE | SDF_COMMON | SDF_GROUND
| SDF_DIFF;
s->n_chan = 16;
s->maxdata = this_board->ai_is_16bit ? 0xffff : 0x0fff;
s->len_chanlist = s->n_chan;
s->range_table = &range_apci3120_ai;
s->insn_config = apci3120_ai_insn_config;
s->insn_read = apci3120_ai_insn_read;
s->do_cmdtest = apci3120_ai_cmdtest;
s->do_cmd = apci3120_ai_cmd;
s->cancel = apci3120_cancel;
s->type = COMEDI_SUBD_AI;
s->subdev_flags = SDF_READABLE | SDF_COMMON | SDF_GROUND | SDF_DIFF;
s->n_chan = 16;
s->maxdata = this_board->ai_is_16bit ? 0xffff : 0x0fff;
s->range_table = &range_apci3120_ai;
s->insn_config = apci3120_ai_insn_config;
s->insn_read = apci3120_ai_insn_read;
if (0 /* dev->irq */) {
dev->read_subdev = s;
s->subdev_flags |= SDF_CMD_READ;
s->len_chanlist = s->n_chan;
s->do_cmdtest = apci3120_ai_cmdtest;
s->do_cmd = apci3120_ai_cmd;
s->cancel = apci3120_cancel;
}
/* Analog Output subdevice */
s = &dev->subdevices[1];