From: Bernd Porr Date: Sat, 21 Oct 2006 22:54:30 +0000 (+0000) Subject: Fixed two small bugs Ian Abbott pointed out. Ian Wrote: X-Git-Tag: r0_7_74~183 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=9b9fd22087d64942cacc895ea024e87c54597bc1;p=comedi.git Fixed two small bugs Ian Abbott pointed out. Ian Wrote: The first one is just a tiny mistake in usbduxfast_ai_cmdtest(): /* step 3: make sure arguments are trivially compatible */ if(cmd->start_arg == TRIG_NOW && cmd->start_arg != 0) should be: if(cmd->start_src == TRIG_NOW && cmd->start_arg != 0) The other one is a mismatch between cmdtest and cmd. cmdtest checks start_src==TRIG_NOW or start_src==TRIG_EXT, but cmd checks start_src==TRIG_NOW and otherwise assumes start_src==TRIG_INT. --- diff --git a/comedi/drivers/usbduxfast.c b/comedi/drivers/usbduxfast.c index 2513a8ed..730f6c4a 100644 --- a/comedi/drivers/usbduxfast.c +++ b/comedi/drivers/usbduxfast.c @@ -1,4 +1,4 @@ -#define DRIVER_VERSION "v0.94" +#define DRIVER_VERSION "v0.99" #define DRIVER_AUTHOR "Bernd Porr, BerndPorr@f2s.com" #define DRIVER_DESC "USB-DUXfast, BerndPorr@f2s.com" /* @@ -25,7 +25,7 @@ Driver: usbduxfast.c Description: ITL USB-DUXfast Devices: [ITL] USB-DUX (usbduxfast.o) Author: Bernd Porr -Updated: 12 Jun 2005 +Updated: 21 Oct 2006 Status: testing */ @@ -43,6 +43,7 @@ Status: testing * Buffer overflows in the FX2 are handed over to comedi. * 0.92: Dropping now 4 packets. The quad buffer has to be emptied. * Added insn command basically for testing. Sample rate is 1MHz/16ch=62.5kHz + * 0.99: Ian Abbott pointed out a bug which has been corrected. Thanks! */ @@ -672,7 +673,7 @@ static int usbduxfast_ai_cmdtest(comedi_device *dev, /* step 1: make sure trigger sources are trivially valid */ tmp = cmd->start_src; - cmd->start_src &= TRIG_NOW | TRIG_EXT; + cmd->start_src &= TRIG_NOW; if(!cmd->start_src || tmp != cmd->start_src) err++; tmp = cmd->scan_begin_src; @@ -715,7 +716,7 @@ static int usbduxfast_ai_cmdtest(comedi_device *dev, /* step 3: make sure arguments are trivially compatible */ - if(cmd->start_arg == TRIG_NOW && cmd->start_arg != 0) + if(cmd->start_src == TRIG_NOW && cmd->start_arg != 0) { cmd->start_arg = 0; err++;