Commit 57fab5a2 authored by Ian Abbott's avatar Ian Abbott Committed by Greg Kroah-Hartman

staging: comedi: rtd520: remove #include "comedi_fc.h"

As preparation for removal of "comedi_fc.h", replace calls to the
`cfc_check_trigger_...` functions from "comedi_fc.h" with the
replacement `comedi_check_trigger_...` functions from "../comedidev.h"
and remove the inclusion of "comedi_fc.h".
Signed-off-by: default avatarIan Abbott <abbotti@mev.co.uk>
Reviewed-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 75a12586
...@@ -99,7 +99,6 @@ ...@@ -99,7 +99,6 @@
#include "../comedi_pci.h" #include "../comedi_pci.h"
#include "comedi_fc.h"
#include "plx9080.h" #include "plx9080.h"
/* /*
...@@ -733,21 +732,22 @@ static int rtd_ai_cmdtest(struct comedi_device *dev, ...@@ -733,21 +732,22 @@ static int rtd_ai_cmdtest(struct comedi_device *dev,
/* Step 1 : check if triggers are trivially valid */ /* Step 1 : check if triggers are trivially valid */
err |= cfc_check_trigger_src(&cmd->start_src, TRIG_NOW); err |= comedi_check_trigger_src(&cmd->start_src, TRIG_NOW);
err |= cfc_check_trigger_src(&cmd->scan_begin_src, err |= comedi_check_trigger_src(&cmd->scan_begin_src,
TRIG_TIMER | TRIG_EXT); TRIG_TIMER | TRIG_EXT);
err |= cfc_check_trigger_src(&cmd->convert_src, TRIG_TIMER | TRIG_EXT); err |= comedi_check_trigger_src(&cmd->convert_src,
err |= cfc_check_trigger_src(&cmd->scan_end_src, TRIG_COUNT); TRIG_TIMER | TRIG_EXT);
err |= cfc_check_trigger_src(&cmd->stop_src, TRIG_COUNT | TRIG_NONE); err |= comedi_check_trigger_src(&cmd->scan_end_src, TRIG_COUNT);
err |= comedi_check_trigger_src(&cmd->stop_src, TRIG_COUNT | TRIG_NONE);
if (err) if (err)
return 1; return 1;
/* Step 2a : make sure trigger sources are unique */ /* Step 2a : make sure trigger sources are unique */
err |= cfc_check_trigger_is_unique(cmd->scan_begin_src); err |= comedi_check_trigger_is_unique(cmd->scan_begin_src);
err |= cfc_check_trigger_is_unique(cmd->convert_src); err |= comedi_check_trigger_is_unique(cmd->convert_src);
err |= cfc_check_trigger_is_unique(cmd->stop_src); err |= comedi_check_trigger_is_unique(cmd->stop_src);
/* Step 2b : and mutually compatible */ /* Step 2b : and mutually compatible */
...@@ -756,32 +756,32 @@ static int rtd_ai_cmdtest(struct comedi_device *dev, ...@@ -756,32 +756,32 @@ static int rtd_ai_cmdtest(struct comedi_device *dev,
/* Step 3: check if arguments are trivially valid */ /* Step 3: check if arguments are trivially valid */
err |= cfc_check_trigger_arg_is(&cmd->start_arg, 0); err |= comedi_check_trigger_arg_is(&cmd->start_arg, 0);
if (cmd->scan_begin_src == TRIG_TIMER) { if (cmd->scan_begin_src == TRIG_TIMER) {
/* Note: these are time periods, not actual rates */ /* Note: these are time periods, not actual rates */
if (1 == cmd->chanlist_len) { /* no scanning */ if (1 == cmd->chanlist_len) { /* no scanning */
if (cfc_check_trigger_arg_min(&cmd->scan_begin_arg, if (comedi_check_trigger_arg_min(&cmd->scan_begin_arg,
RTD_MAX_SPEED_1)) { RTD_MAX_SPEED_1)) {
rtd_ns_to_timer(&cmd->scan_begin_arg, rtd_ns_to_timer(&cmd->scan_begin_arg,
CMDF_ROUND_UP); CMDF_ROUND_UP);
err |= -EINVAL; err |= -EINVAL;
} }
if (cfc_check_trigger_arg_max(&cmd->scan_begin_arg, if (comedi_check_trigger_arg_max(&cmd->scan_begin_arg,
RTD_MIN_SPEED_1)) { RTD_MIN_SPEED_1)) {
rtd_ns_to_timer(&cmd->scan_begin_arg, rtd_ns_to_timer(&cmd->scan_begin_arg,
CMDF_ROUND_DOWN); CMDF_ROUND_DOWN);
err |= -EINVAL; err |= -EINVAL;
} }
} else { } else {
if (cfc_check_trigger_arg_min(&cmd->scan_begin_arg, if (comedi_check_trigger_arg_min(&cmd->scan_begin_arg,
RTD_MAX_SPEED)) { RTD_MAX_SPEED)) {
rtd_ns_to_timer(&cmd->scan_begin_arg, rtd_ns_to_timer(&cmd->scan_begin_arg,
CMDF_ROUND_UP); CMDF_ROUND_UP);
err |= -EINVAL; err |= -EINVAL;
} }
if (cfc_check_trigger_arg_max(&cmd->scan_begin_arg, if (comedi_check_trigger_arg_max(&cmd->scan_begin_arg,
RTD_MIN_SPEED)) { RTD_MIN_SPEED)) {
rtd_ns_to_timer(&cmd->scan_begin_arg, rtd_ns_to_timer(&cmd->scan_begin_arg,
CMDF_ROUND_DOWN); CMDF_ROUND_DOWN);
err |= -EINVAL; err |= -EINVAL;
...@@ -791,32 +791,32 @@ static int rtd_ai_cmdtest(struct comedi_device *dev, ...@@ -791,32 +791,32 @@ static int rtd_ai_cmdtest(struct comedi_device *dev,
/* external trigger */ /* external trigger */
/* should be level/edge, hi/lo specification here */ /* should be level/edge, hi/lo specification here */
/* should specify multiple external triggers */ /* should specify multiple external triggers */
err |= cfc_check_trigger_arg_max(&cmd->scan_begin_arg, 9); err |= comedi_check_trigger_arg_max(&cmd->scan_begin_arg, 9);
} }
if (cmd->convert_src == TRIG_TIMER) { if (cmd->convert_src == TRIG_TIMER) {
if (1 == cmd->chanlist_len) { /* no scanning */ if (1 == cmd->chanlist_len) { /* no scanning */
if (cfc_check_trigger_arg_min(&cmd->convert_arg, if (comedi_check_trigger_arg_min(&cmd->convert_arg,
RTD_MAX_SPEED_1)) { RTD_MAX_SPEED_1)) {
rtd_ns_to_timer(&cmd->convert_arg, rtd_ns_to_timer(&cmd->convert_arg,
CMDF_ROUND_UP); CMDF_ROUND_UP);
err |= -EINVAL; err |= -EINVAL;
} }
if (cfc_check_trigger_arg_max(&cmd->convert_arg, if (comedi_check_trigger_arg_max(&cmd->convert_arg,
RTD_MIN_SPEED_1)) { RTD_MIN_SPEED_1)) {
rtd_ns_to_timer(&cmd->convert_arg, rtd_ns_to_timer(&cmd->convert_arg,
CMDF_ROUND_DOWN); CMDF_ROUND_DOWN);
err |= -EINVAL; err |= -EINVAL;
} }
} else { } else {
if (cfc_check_trigger_arg_min(&cmd->convert_arg, if (comedi_check_trigger_arg_min(&cmd->convert_arg,
RTD_MAX_SPEED)) { RTD_MAX_SPEED)) {
rtd_ns_to_timer(&cmd->convert_arg, rtd_ns_to_timer(&cmd->convert_arg,
CMDF_ROUND_UP); CMDF_ROUND_UP);
err |= -EINVAL; err |= -EINVAL;
} }
if (cfc_check_trigger_arg_max(&cmd->convert_arg, if (comedi_check_trigger_arg_max(&cmd->convert_arg,
RTD_MIN_SPEED)) { RTD_MIN_SPEED)) {
rtd_ns_to_timer(&cmd->convert_arg, rtd_ns_to_timer(&cmd->convert_arg,
CMDF_ROUND_DOWN); CMDF_ROUND_DOWN);
err |= -EINVAL; err |= -EINVAL;
...@@ -825,15 +825,16 @@ static int rtd_ai_cmdtest(struct comedi_device *dev, ...@@ -825,15 +825,16 @@ static int rtd_ai_cmdtest(struct comedi_device *dev,
} else { } else {
/* external trigger */ /* external trigger */
/* see above */ /* see above */
err |= cfc_check_trigger_arg_max(&cmd->convert_arg, 9); err |= comedi_check_trigger_arg_max(&cmd->convert_arg, 9);
} }
err |= cfc_check_trigger_arg_is(&cmd->scan_end_arg, cmd->chanlist_len); err |= comedi_check_trigger_arg_is(&cmd->scan_end_arg,
cmd->chanlist_len);
if (cmd->stop_src == TRIG_COUNT) if (cmd->stop_src == TRIG_COUNT)
err |= cfc_check_trigger_arg_min(&cmd->stop_arg, 1); err |= comedi_check_trigger_arg_min(&cmd->stop_arg, 1);
else /* TRIG_NONE */ else /* TRIG_NONE */
err |= cfc_check_trigger_arg_is(&cmd->stop_arg, 0); err |= comedi_check_trigger_arg_is(&cmd->stop_arg, 0);
if (err) if (err)
return 3; return 3;
...@@ -843,18 +844,19 @@ static int rtd_ai_cmdtest(struct comedi_device *dev, ...@@ -843,18 +844,19 @@ static int rtd_ai_cmdtest(struct comedi_device *dev,
if (cmd->scan_begin_src == TRIG_TIMER) { if (cmd->scan_begin_src == TRIG_TIMER) {
arg = cmd->scan_begin_arg; arg = cmd->scan_begin_arg;
rtd_ns_to_timer(&arg, cmd->flags); rtd_ns_to_timer(&arg, cmd->flags);
err |= cfc_check_trigger_arg_is(&cmd->scan_begin_arg, arg); err |= comedi_check_trigger_arg_is(&cmd->scan_begin_arg, arg);
} }
if (cmd->convert_src == TRIG_TIMER) { if (cmd->convert_src == TRIG_TIMER) {
arg = cmd->convert_arg; arg = cmd->convert_arg;
rtd_ns_to_timer(&arg, cmd->flags); rtd_ns_to_timer(&arg, cmd->flags);
err |= cfc_check_trigger_arg_is(&cmd->convert_arg, arg); err |= comedi_check_trigger_arg_is(&cmd->convert_arg, arg);
if (cmd->scan_begin_src == TRIG_TIMER) { if (cmd->scan_begin_src == TRIG_TIMER) {
arg = cmd->convert_arg * cmd->scan_end_arg; arg = cmd->convert_arg * cmd->scan_end_arg;
err |= cfc_check_trigger_arg_min(&cmd->scan_begin_arg, err |= comedi_check_trigger_arg_min(&cmd->
arg); scan_begin_arg,
arg);
} }
} }
......
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