openkylin
/
linux
mirror of https://gitee.com/openkylin/linux.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

staging: comedi: ni_660x: clean up pfi routing 

Cleans up the pfi routing code to make it easier to follow, read, and also
to prepare to use this cleaned up code for enabling the device-global
routing interface for ni_660x devices.

Signed-off-by: Spencer E. Olson <olsonse@umich.edu>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Spencer E. Olson 2018-10-03 14:56:11 -06:00 committed by Greg Kroah-Hartman
parent 51c4ba6407
commit a0c5e84600
1 changed files with 56 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
drivers/staging/comedi/drivers/ni_660x.c
View File

@ -596,6 +596,37 @@ static void ni_660x_select_pfi_output(struct comedi_device *dev,
ni_660x_write(dev, active_chip, bits, NI660X_IO_CFG(chan)); ni_660x_write(dev, active_chip, bits, NI660X_IO_CFG(chan));
} }
static void ni_660x_set_pfi_direction(struct comedi_device *dev,
unsigned int chan,
unsigned int direction)
{
struct ni_660x_private *devpriv = dev->private;
u64 bit;
bit = 1ULL << chan;
if (direction == COMEDI_OUTPUT) {
devpriv->io_dir |= bit;
/* reset the output to currently assigned output value */
ni_660x_select_pfi_output(dev, chan, devpriv->io_cfg[chan]);
} else {
devpriv->io_dir &= ~bit;
/* set pin to high-z; do not change currently assigned route */
ni_660x_select_pfi_output(dev, chan, 0);
}
}
static unsigned int ni_660x_get_pfi_direction(struct comedi_device *dev,
unsigned int chan)
{
struct ni_660x_private *devpriv = dev->private;
u64 bit;
bit = 1ULL << chan;
return (devpriv->io_dir & bit) ? COMEDI_OUTPUT : COMEDI_INPUT;
}
static int ni_660x_set_pfi_routing(struct comedi_device *dev, static int ni_660x_set_pfi_routing(struct comedi_device *dev,
unsigned int chan, unsigned int source) unsigned int chan, unsigned int source)
{ {
@ -614,36 +645,48 @@ static int ni_660x_set_pfi_routing(struct comedi_device *dev,
} }
devpriv->io_cfg[chan] = source; devpriv->io_cfg[chan] = source;
if (devpriv->io_dir & (1ULL << chan)) if (ni_660x_get_pfi_direction(dev, chan) == COMEDI_OUTPUT)
ni_660x_select_pfi_output(dev, chan, devpriv->io_cfg[chan]); ni_660x_select_pfi_output(dev, chan, devpriv->io_cfg[chan]);
return 0; return 0;
} }
static int ni_660x_get_pfi_routing(struct comedi_device *dev, unsigned int chan)
{
struct ni_660x_private *devpriv = dev->private;
return devpriv->io_cfg[chan];
}
static void ni_660x_set_pfi_filter(struct comedi_device *dev,
unsigned int chan, unsigned int value)
{
unsigned int val;
val = ni_660x_read(dev, 0, NI660X_IO_CFG(chan));
val &= ~NI660X_IO_CFG_IN_SEL_MASK(chan);
val |= NI660X_IO_CFG_IN_SEL(chan, value);
ni_660x_write(dev, 0, val, NI660X_IO_CFG(chan));
}
static int ni_660x_dio_insn_config(struct comedi_device *dev, static int ni_660x_dio_insn_config(struct comedi_device *dev,
struct comedi_subdevice *s, struct comedi_subdevice *s,
struct comedi_insn *insn, struct comedi_insn *insn,
unsigned int *data) unsigned int *data)
{ {
struct ni_660x_private *devpriv = dev->private;
unsigned int chan = CR_CHAN(insn->chanspec); unsigned int chan = CR_CHAN(insn->chanspec);
u64 bit = 1ULL << chan;
unsigned int val;
int ret; int ret;
switch (data[0]) { switch (data[0]) {
case INSN_CONFIG_DIO_OUTPUT: case INSN_CONFIG_DIO_OUTPUT:
devpriv->io_dir |= bit; ni_660x_set_pfi_direction(dev, chan, COMEDI_OUTPUT);
ni_660x_select_pfi_output(dev, chan, devpriv->io_cfg[chan]);
break; break;
case INSN_CONFIG_DIO_INPUT: case INSN_CONFIG_DIO_INPUT:
devpriv->io_dir &= ~bit; ni_660x_set_pfi_direction(dev, chan, COMEDI_INPUT);
ni_660x_select_pfi_output(dev, chan, 0); /* high-z */
break; break;
case INSN_CONFIG_DIO_QUERY: case INSN_CONFIG_DIO_QUERY:
data[1] = (devpriv->io_dir & bit) ? COMEDI_OUTPUT data[1] = ni_660x_get_pfi_direction(dev, chan);
: COMEDI_INPUT;
break; break;
case INSN_CONFIG_SET_ROUTING: case INSN_CONFIG_SET_ROUTING:
@ -653,14 +696,11 @@ static int ni_660x_dio_insn_config(struct comedi_device *dev,
break; break;
case INSN_CONFIG_GET_ROUTING: case INSN_CONFIG_GET_ROUTING:
data[1] = devpriv->io_cfg[chan]; data[1] = ni_660x_get_pfi_routing(dev, chan);
break; break;
case INSN_CONFIG_FILTER: case INSN_CONFIG_FILTER:
val = ni_660x_read(dev, 0, NI660X_IO_CFG(chan)); ni_660x_set_pfi_filter(dev, chan, data[1]);
val &= ~NI660X_IO_CFG_IN_SEL_MASK(chan);
val |= NI660X_IO_CFG_IN_SEL(chan, data[1]);
ni_660x_write(dev, 0, val, NI660X_IO_CFG(chan));
break; break;
default: default:
@ -840,7 +880,7 @@ static int ni_660x_auto_attach(struct comedi_device *dev,
: NI_660X_PFI_OUTPUT_COUNTER; : NI_660X_PFI_OUTPUT_COUNTER;
ni_660x_set_pfi_routing(dev, i, source); ni_660x_set_pfi_routing(dev, i, source);
ni_660x_select_pfi_output(dev, i, 0); /* high-z */ ni_660x_set_pfi_direction(dev, i, COMEDI_INPUT);/* high-z */
} }
/* Counter subdevices (4 NI TIO General Purpose Counters per chip) */ /* Counter subdevices (4 NI TIO General Purpose Counters per chip) */