From: H Hartley Sweeten Date: Tue, 27 May 2014 17:30:47 +0000 (-0700) Subject: staging: comedi: amplc_dio200_common: use comedi_cmd pointer X-Git-Tag: next-20140530~19^2~49 X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=71ba7506f05e6512e4b25df2233fa238025399d5;p=karo-tx-linux.git staging: comedi: amplc_dio200_common: use comedi_cmd pointer Use the local variable to access the comedi_cmd as a pointer in dio200_read_scan_intr() instead of getting to it from the comedi_subdevice pointer. Remove the unnecessary local variable 'len', this is just the cmd->chanlist_len. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/comedi/drivers/amplc_dio200_common.c b/drivers/staging/comedi/drivers/amplc_dio200_common.c index 1a580c374ca8..3edaa4028da2 100644 --- a/drivers/staging/comedi/drivers/amplc_dio200_common.c +++ b/drivers/staging/comedi/drivers/amplc_dio200_common.c @@ -312,12 +312,11 @@ static void dio200_read_scan_intr(struct comedi_device *dev, struct dio200_subdev_intr *subpriv = s->private; struct comedi_cmd *cmd = &s->async->cmd; unsigned short val; - unsigned int n, ch, len; + unsigned int n, ch; val = 0; - len = s->async->cmd.chanlist_len; - for (n = 0; n < len; n++) { - ch = CR_CHAN(s->async->cmd.chanlist[n]); + for (n = 0; n < cmd->chanlist_len; n++) { + ch = CR_CHAN(cmd->chanlist[n]); if (triggered & (1U << ch)) val |= (1U << n); }