linux.git/drivers/staging/comedi/drivers.c, branch v2.6.34

Staging: comedi: drivers.c: Checkpatch cleanup

2010-03-04T00:43:07+00:00

WARNING: Use #include  instead of 
+#include 

WARNING: braces {} are not necessary for any arm of this statement
+	if (dev->driver) {
[...]
+	} else {
[...]

WARNING: braces {} are not necessary for single statement blocks
+	if (insn->insn == INSN_READ) {
+		data[0] = (new_data[1] >> (chan - base_bitfield_channel)) & 1;
+	}

WARNING: braces {} are not necessary for single statement blocks
+	if (async->prealloc_buf && async->prealloc_bufsz == new_size) {
+		return 0;
+	}

WARNING: braces {} are not necessary for single statement blocks
+				if (async->buf_page_list[i].virt_addr == NULL) {
+					break;
+				}

WARNING: braces {} are not necessary for single statement blocks
+	if ((int)(async->buf_write_alloc_count + nbytes - free_end) > 0) {
+		nbytes = free_end - async->buf_write_alloc_count;
+	}

WARNING: braces {} are not necessary for single statement blocks
+	if ((int)(async->buf_write_alloc_count + nbytes - free_end) > 0) {
+		nbytes = 0;
+	}

WARNING: braces {} are not necessary for single statement blocks
+	if (async->buf_write_ptr >= async->prealloc_bufsz) {
+		async->buf_write_ptr %= async->prealloc_bufsz;
+	}

Signed-off-by: Andrea Gelmini

Staging: comedi: drivers.c: checkpatch fix

2009-12-11T20:21:48+00:00

Fix a checkpatch.pl error. Fix struct * foo to struct *foo

Signed-off-by: Mithlesh Thukral 
Signed-off-by: Greg Kroah-Hartman

Staging: comedi: no need for checking vfree call

2009-09-15T19:01:33+00:00

vfree() does it's own NULL checking,so no need for check before
calling it.

'pages' is local variable argment,so in v2, it is no need assignment
 to NULL.

Signed-off-by: Figo.zhang 
Acked-by: Pekka Enberg 
Signed-off-by: Greg Kroah-Hartman

Staging: Comedi: Lindent changes to comdi driver in staging tree

2009-09-15T19:01:33+00:00

Lindent changes to comdi driver in staging tree.
This patch is followed by the checkpatch.pl error fixes.
Did not make them part of this patch as the patch size is already huge.

Signed-off-by: Mithlesh Thukral 
Signed-off-by: Greg Kroah-Hartman

Staging: comedi: remove comedi-specific wrappers

2009-06-19T18:00:33+00:00

There are a number of comedi "wrappers" for some RT functions that are
about to go away.  This patch removes all of the wrapper calls within
the comedi drivers and core in order to prepare for removing the RT
comedi code.

Cc: Ian Abbott 
Cc: Frank Mori Hess 
Signed-off-by: Greg Kroah-Hartman

Staging: comedi: make use of ARRAY_SIZE macro

2009-06-19T18:00:33+00:00

Replace instances of computing number of elements in an array with
sizeof(foo)/sizeof(struct footype) with the ARRAY_SIZE macro.

Signed-off-by: Bill Pemberton 
Signed-off-by: Greg Kroah-Hartman

Staging Comedi: fix spacing around parens

2009-06-19T18:00:31+00:00

Signed-off-by: Bill Pemberton 
Signed-off-by: Greg Kroah-Hartman

Staging: BUG to BUG_ON changes

2009-04-03T21:54:26+00:00

Signed-off-by: Stoyan Gaydarov 
Signed-off-by: Greg Kroah-Hartman

Staging: comedi: Remove comedi_devconfig typedef

2009-04-03T21:53:58+00:00

Signed-off-by: Bill Pemberton 
Signed-off-by: Greg Kroah-Hartman

Staging: comedi: Remove comedi_insn typedef

2009-04-03T21:53:57+00:00

Signed-off-by: Bill Pemberton 
Signed-off-by: Greg Kroah-Hartman