Commit 3443d1c6 authored by Ian Abbott's avatar Ian Abbott Committed by Greg Kroah-Hartman

staging: comedi: ni_65xx: support INSN_CONFIG_DIGITAL_TRIG

The "edge detection interrupt" subdevice supports the
`INSN_CONFIG_CHANGE_NOTIFY` comedi instruction which is only supported
by one other driver.  It is limited to the first 32 channels, but boards
supported by this driver support edge detection on all digital I/O,
digital input and digital output channels.  The
`INSN_CONFIG_DIGITAL_TRIG` comedi instruction is more flexible as it
supports more than 32 channels (with multiple instructions).  It can
also support level detection, but the hardware does not support that.

Add partial support for the `INSN_CONFIG_DIGITAL_TRIG` instruction, but
only for edge detection.

Since `INSN_CONFIG_CHANGE_NOTIFY` can only deal with 32 channels, make
it disable edge detection for any remaining channels.
Signed-off-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a5e7d93d
...@@ -365,6 +365,16 @@ static void ni_65xx_update_edge_detection(struct comedi_device *dev, ...@@ -365,6 +365,16 @@ static void ni_65xx_update_edge_detection(struct comedi_device *dev,
} }
} }
static void ni_65xx_disable_edge_detection(struct comedi_device *dev)
{
/* clear edge detection for channels 0 to 31 */
ni_65xx_update_edge_detection(dev, 0, 0, 0);
/* clear edge detection for channels 32 to 63 */
ni_65xx_update_edge_detection(dev, 32, 0, 0);
/* clear edge detection for channels 64 to 95 */
ni_65xx_update_edge_detection(dev, 64, 0, 0);
}
static int ni_65xx_dio_insn_config(struct comedi_device *dev, static int ni_65xx_dio_insn_config(struct comedi_device *dev,
struct comedi_subdevice *s, struct comedi_subdevice *s,
struct comedi_insn *insn, struct comedi_insn *insn,
...@@ -599,10 +609,33 @@ static int ni_65xx_intr_insn_config(struct comedi_device *dev, ...@@ -599,10 +609,33 @@ static int ni_65xx_intr_insn_config(struct comedi_device *dev,
if (insn->n != 3) if (insn->n != 3)
return -EINVAL; return -EINVAL;
/* /* update edge detection for channels 0 to 31 */
* This only works for the first 4 ports (32 channels)!
*/
ni_65xx_update_edge_detection(dev, 0, data[1], data[2]); ni_65xx_update_edge_detection(dev, 0, data[1], data[2]);
/* clear edge detection for channels 32 to 63 */
ni_65xx_update_edge_detection(dev, 32, 0, 0);
/* clear edge detection for channels 64 to 95 */
ni_65xx_update_edge_detection(dev, 64, 0, 0);
break;
case INSN_CONFIG_DIGITAL_TRIG:
/* check trigger number */
if (data[1] != 0)
return -EINVAL;
/* check digital trigger operation */
switch (data[2]) {
case COMEDI_DIGITAL_TRIG_DISABLE:
ni_65xx_disable_edge_detection(dev);
break;
case COMEDI_DIGITAL_TRIG_ENABLE_EDGES:
/*
* update edge detection for channels data[3]
* to (data[3] + 31)
*/
ni_65xx_update_edge_detection(dev, data[3],
data[4], data[5]);
break;
default:
return -EINVAL;
}
break; break;
default: default:
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment