For aesthetic reasons, cleanup this function a bit.
Change the (async == NULL) test to simply (!async).
Cleanup the multi-line comment about the barrier.
Signed-off-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx>
Cc: Ian Abbott <abbobbi@xxxxxxxxx>
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
drivers/staging/comedi/comedi_buf.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/comedi/comedi_buf.c b/drivers/staging/comedi/comedi_buf.c
index 827295f..a12b70b 100644
--- a/drivers/staging/comedi/comedi_buf.c
+++ b/drivers/staging/comedi/comedi_buf.c
@@ -275,14 +275,17 @@ unsigned int comedi_buf_read_n_available(struct comedi_async *async)
{
unsigned num_bytes;
- if (async == NULL)
+ if (!async)
return 0;
+
num_bytes = async->munge_count - async->buf_read_count;
- /* barrier insures the read of munge_count in this
- query occurs before any following reads of the buffer which
- might be based on the return value from this query.
+
+ /*
+ * barrier insures the read of munge_count above occurs
+ * before any following reads out of the buffer
*/
smp_rmb();
+
return num_bytes;
}
EXPORT_SYMBOL(comedi_buf_read_n_available);
--
1.8.0
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
