Commit 68ef50f6 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: comedi_fc: tidy up cfc_bytes_per_scan()

This function returns the total number of bytes for a full scan of
an async command chanlist. The returned value is an unsigned int.

Change all the local variables to unsigned int and use a local
variable for the s->async->cmd.chanlist_len.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a3065bc9
...@@ -47,19 +47,20 @@ unsigned int cfc_handle_events(struct comedi_device *, ...@@ -47,19 +47,20 @@ unsigned int cfc_handle_events(struct comedi_device *,
static inline unsigned int cfc_bytes_per_scan(struct comedi_subdevice *s) static inline unsigned int cfc_bytes_per_scan(struct comedi_subdevice *s)
{ {
int num_samples; unsigned int chanlist_len = s->async->cmd.chanlist_len;
int bits_per_sample; unsigned int num_samples;
unsigned int bits_per_sample;
switch (s->type) { switch (s->type) {
case COMEDI_SUBD_DI: case COMEDI_SUBD_DI:
case COMEDI_SUBD_DO: case COMEDI_SUBD_DO:
case COMEDI_SUBD_DIO: case COMEDI_SUBD_DIO:
bits_per_sample = 8 * bytes_per_sample(s); bits_per_sample = 8 * bytes_per_sample(s);
num_samples = (s->async->cmd.chanlist_len + num_samples = (chanlist_len + bits_per_sample - 1) /
bits_per_sample - 1) / bits_per_sample; bits_per_sample;
break; break;
default: default:
num_samples = s->async->cmd.chanlist_len; num_samples = chanlist_len;
break; break;
} }
return num_samples * bytes_per_sample(s); return num_samples * bytes_per_sample(s);
......
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