From 99284f252bb1f9df82dbf6a01bef2b34c19b5083 Mon Sep 17 00:00:00 2001 From: Bernd Porr Date: Thu, 24 Jul 2014 14:34:32 +0100 Subject: [PATCH] staging: comedi: usbduxfast: adding missing break in case statement Added a missing "break" which forced the board to acquire 16 channels even when only 3 had been requested. Thanks for Hartley Sweeten to spot this bug. Signed-off-by: Bernd Porr Reported-by: H Hartley Sweeten Reviewed-by: Ian Abbott Reviewed-by: H Hartley Sweeten Signed-off-by: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/usbduxfast.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/comedi/drivers/usbduxfast.c b/drivers/staging/comedi/drivers/usbduxfast.c index 2490e88e7f32..f85818dd5e11 100644 --- a/drivers/staging/comedi/drivers/usbduxfast.c +++ b/drivers/staging/comedi/drivers/usbduxfast.c @@ -745,6 +745,7 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev, 0x00, (0xff - 0x02) & rngmask, 0x00); usbduxfast_cmd_data(dev, 6, 0x01, 0x00, rngmask, 0x00); + break; case 16: if (CR_RANGE(cmd->chanlist[0]) > 0) -- 2.34.1