The chanlist_len is validated by the comedi core before calling
the do_cmdtest function. Remove these redundant checks.

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: Ian Abbott <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
---
 drivers/staging/comedi/drivers/adl_pci9111.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/drivers/staging/comedi/drivers/adl_pci9111.c 
b/drivers/staging/comedi/drivers/adl_pci9111.c
index 349ca0e..ba8d794 100644
--- a/drivers/staging/comedi/drivers/adl_pci9111.c
+++ b/drivers/staging/comedi/drivers/adl_pci9111.c
@@ -514,16 +514,6 @@ pci9111_ai_do_cmd_test(struct comedi_device *dev,
 
        /*  Step 3 : make sure arguments are trivialy compatible */
 
-       if (cmd->chanlist_len < 1) {
-               cmd->chanlist_len = 1;
-               error++;
-       }
-
-       if (cmd->chanlist_len > board->ai_channel_nbr) {
-               cmd->chanlist_len = board->ai_channel_nbr;
-               error++;
-       }
-
        if ((cmd->start_src == TRIG_NOW) && (cmd->start_arg != 0)) {
                cmd->start_arg = 0;
                error++;
-- 
1.7.11

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to