public-mirrors/linux
1
0
Fork 0
mirror of git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git synced 2025-08-04 16:25:34 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

staging: comedi: comedi_buf: make comedi_buf_read_samples() always return full samples

Browse source 
A number of drivers currently use comedi_buf_get() to read single samples
from the the async buffer. This works but the drivers have to handle the
COMEDI_CB_BLOCK and COMEDI_CB_EOS events. Converting those drivers to use
comedi_buf_read_samples() moves the handling of those events into the core.

Modify comedi_buf_read_samples() so that the async buffer is checked for
the number of full samples available. Use that to clamp the number of
samples that will be read. This makes sure that only full samples are
read from the async buffer making comedi_buf_read_samples() mimic the
action of comedi_buf_get() with the added benifit of handling the events.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
H Hartley Sweeten 2014-10-22 14:36:42 -07:00 • committed by Greg Kroah-Hartman
parent 109bf06e2a
commit 8de27e7010
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
drivers/staging/comedi/comedi_buf.c
View file

@ -566,15 +566,15 @@ unsigned int comedi_buf_read_samples(struct comedi_subdevice *s,
unsigned int max_samples;
unsigned int nbytes;
max_samples = s->async->prealloc_bufsz / bytes_per_sample(s);
/* clamp nsamples to the number of full samples available */
max_samples = comedi_buf_read_n_available(s) / bytes_per_sample(s);
if (nsamples > max_samples)
nsamples = max_samples;
nbytes = nsamples * bytes_per_sample(s);
if (nbytes == 0)
if (nsamples == 0)
return 0;
nbytes = comedi_buf_read_alloc(s, nbytes);
nbytes = comedi_buf_read_alloc(s, nsamples * bytes_per_sample(s));
comedi_buf_memcpy_from(s, 0, data, nbytes);
comedi_buf_read_free(s, nbytes);
comedi_inc_scan_progress(s, nbytes);