From d21af4cbfb7fccbccedc1d9e143a14aee6b1c9ad Mon Sep 17 00:00:00 2001 From: H Hartley Sweeten Date: Wed, 9 Jan 2013 13:29:53 -0700 Subject: [PATCH] staging: comedi: comedi_buf: clarify comedi_buf_write_free() Use the helper comedi_buf_write_n_allocated() to clarify the check to make sure the number of bytes to free is not more than the number of bytes allocated. Signed-off-by: H Hartley Sweeten Signed-off-by: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- drivers/staging/comedi/comedi_buf.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/comedi_buf.c b/drivers/staging/comedi/comedi_buf.c index 95cbe365ded2..fd2a92e29fb0 100644 --- a/drivers/staging/comedi/comedi_buf.c +++ b/drivers/staging/comedi/comedi_buf.c @@ -252,11 +252,12 @@ static unsigned int comedi_buf_munge(struct comedi_async *async, /* transfers a chunk from writer to filled buffer space */ unsigned comedi_buf_write_free(struct comedi_async *async, unsigned int nbytes) { - if ((int)(async->buf_write_count + nbytes - - async->buf_write_alloc_count) > 0) { + unsigned int allocated = comedi_buf_write_n_allocated(async); + + if (nbytes > allocated) { dev_info(async->subdevice->device->class_dev, "attempted to write-free more bytes than have been write-allocated.\n"); - nbytes = async->buf_write_alloc_count - async->buf_write_count; + nbytes = allocated; } async->buf_write_count += nbytes; async->buf_write_ptr += nbytes; -- 2.39.5