This is a patch to the adl_pci6208.c file that fixes up a space before
tabs warning found by checkpatch.pl tool.
Signed-off-by: Jaakko Niemelä <mato@kyborgi.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch in hfa834x_usb.c to fix typedef declarations and long lines.
Signed-off-by: Edgardo Hames <ehames@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The code is trying to say that if the offset is higher than the max it
should be set to the max, but there is an off by one bug and it sets it
one passed the end of the array.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch removes unecessary whitespace before quoted newlines
in two files.
One warning in each file remains unresolved.
Signed-off-by: Michael Tate <michael.tate@wanadoo.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
removed extern keyword from rtl819x_watchdog_wqcallback.
removed static declaration from variable check_reset_cnt.
changed NumRxOkInPeriod and NumTxOkInPeriod comparison
in function rtl819x_watchdog_wqcallback to 666; we're not on windows.
plus cosmetics.
Signed-off-by: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
variable i is declared integer in rtl8192SU_ConfigAdapterInfo8192SForAutoLoadFail.
a few lines down it is re-declared u8.
removed the re-declaration.
plus cosmetics
Signed-off-by: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
this is basically a port of Larry Fingers Patch for rtl8187se to rtl8192su.
also removed some dead code.
plus cosmetics.
Signed-off-by: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This revised patch fixes 2 brace coding style issues reported by checkpatch.pl
One warning line > 80 chars not resolved on maintainers advice.
Signed-off-by: Michael Tate <michael.tate@wanadoo.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fix all but one of the errors reported by checkpatch.pl tool.
Errors included wayward braces, white space issues(trailing
and prohibited), C99 issues, and foo* issues
Signed-off-by: John Sheehan <john.d.sheehan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Some confusion was caused by the ___iio_init_ring_buffer and equivalent
in ring_sw handling both init of spin locks etc and allocation and
of the actual buffer. This resulted in ring->use_lock being held
before it was initialized and actually during the initialization.
Some of the recent cleanups in the spin lock code seem to have triggered
the bug actually causing traceable crashes.
The following patch should fix this but hasn't been extensively tested
as of yet and there may well be some side effects I haven't thought of.
Just wanted to get this out there before anyone else runs into it!
Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Removed kerneldoc /** from functions that should not have them.
Added proper kerneldoc headers to functions that should have them.
This includes fixes as pointed out by Randy Dunlap and Joe Perches.
Cc: Joe Perches <joe@perches.com>
Acked-by: Randy Dunlap <rdunlap@xenotime.net>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Remove Ringbuffer work line item from TODO file.
The ring buffer in the Hyper-V Linux drivers is used to communicate with
the parent partition running Windows Server 2008 Hyper-V. The ring
buffer functionality on the Hyper-V Linux drivers is written to be
functionally compatible with the ring buffer functionality on the
Hyper-V Server. Consequently, it is not possible to make any changes
that might break the compatibility with server side ring buffer
implementation.
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Remove staging/poch.
Reasons for removal are -- The driver has serious cache
issues, that I couldn't fix. The card vendor is working
on a better replacement for the driver. The driver has
been delayed a lot and development has come to a stand
still.
Signed-off-by: Vijay Kumar B. <vijaykumar@bravegnu.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the aio_aio12_8.c that fixes up print(k) warnings found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a Patch to the adv_pci1723.c file that fixes up brace, print(k) and over 80 character warnings found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the adl_pci9118.c file that fixes WARNING: printk() should include KERN_facility level found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Fixes all over 80 character warnings in the adl_pci9118.c file found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Patch to the adl_pci9118.c that fixes, ERROR: code indent should use tabs where possible, found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Obviously still needs serious attention, but it compiles.
Original author: Rick Dobbs
Add driver to support wanPMC-CxT1E1 card.
This card provides 1-4 ports of T1E1 in PMC form factor.
Note, Rick doesn't want his email showing up as the "From:" author, but
has given his blessing to have the code included in the kernel tree.
Signed-off-by: Bob Beers <bob.beers@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the skel.c file that fixes the 80 characters limit and
printk warnings found by the checkpatch.pl tool
Signed-off-by: Ellwyn Cole <ellwync@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/batman-adv/send.c:137: CHECK: multiple assignments should be avoided
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/otus/hal/hpreg.c:33: WARNING: space prohibited between function name and open parenthesis '('
drivers/staging/otus/hal/hpreg.c:33: WARNING: space prohibited between function name and open parenthesis '('
drivers/staging/otus/hal/hpreg.c:38: CHECK: if this code is redundant consider removing it
drivers/staging/otus/hal/hpreg.c:81: ERROR: need consistent spacing around '|' (ctx:VxW)
drivers/staging/otus/hal/hpreg.c:96: ERROR: Macros with multiple statements should be enclosed in a do - while loop
drivers/staging/otus/hal/hpreg.c:98: CHECK: if this code is redundant consider removing it
drivers/staging/otus/hal/hpreg.c:275: ERROR: need consistent spacing around '|' (ctx:WxV)
drivers/staging/otus/hal/hpreg.c:304: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:362: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:376: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:377: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:402: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:402: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:413: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:413: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:434: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:496: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:497: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:501: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:526: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:529: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:531: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:539: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:539: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:544: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:663: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1335: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1336: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1345: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1345: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1346: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1346: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1347: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1347: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1355: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1356: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1357: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1357: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1365: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1366: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1367: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1367: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1375: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1376: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1377: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1377: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1378: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1378: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1387: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1387: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1388: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1388: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1395: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1396: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1397: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1397: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1398: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1398: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1398: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1398: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1405: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1406: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1407: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1407: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1408: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1408: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1415: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1415: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1415: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1415: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1416: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1416: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1416: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1416: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1416: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1416: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1432: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1432: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1432: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1435: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1435: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1435: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1436: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1436: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1436: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1442: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1442: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1442: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1445: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1445: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1445: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1446: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1446: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1446: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1455: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1455: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1455: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1455: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1456: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1456: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1456: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1456: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1456: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1456: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1465: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1465: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1465: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1465: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1466: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1466: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1466: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1466: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1466: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1466: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1475: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1475: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1475: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1475: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1476: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1485: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1485: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1485: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1485: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1485: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1486: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1486: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1486: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1486: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1486: ERROR: space required before that '-' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1557: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1558: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1560: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1560: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1560: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1560: ERROR: spaces required around that '<' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1560: ERROR: spaces required around that '&&' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1562: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1568: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1574: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1579: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1590: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1590: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1590: ERROR: space required after that ';' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1590: ERROR: spaces required around that '<' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1592: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1594: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1613: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1613: ERROR: spaces required around that '<' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1635: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:1639: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1639: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1640: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:1643: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1647: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1647: WARNING: suspect code indent for conditional statements (8, 8)
drivers/staging/otus/hal/hpreg.c:1649: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1652: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1652: WARNING: suspect code indent for conditional statements (8, 8)
drivers/staging/otus/hal/hpreg.c:1654: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1657: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1659: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1660: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1662: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1662: ERROR: else should follow close brace '}'
drivers/staging/otus/hal/hpreg.c:1664: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1671: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1674: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1675: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1676: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1676: ERROR: space required after that ',' (ctx:VxO)
drivers/staging/otus/hal/hpreg.c:1676: ERROR: space required before that '*' (ctx:OxV)
drivers/staging/otus/hal/hpreg.c:1676: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1678: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1681: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1682: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1682: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1683: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1684: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1685: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1686: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1687: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1690: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1692: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1693: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1694: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1695: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1695: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1696: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1696: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1697: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1697: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1698: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1698: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1699: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1700: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1700: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1701: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1702: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1702: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1703: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1706: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1707: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1708: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1710: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1711: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1712: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1713: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1714: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1714: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1717: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1719: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1720: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1721: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1722: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1722: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1723: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1723: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1724: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1724: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1725: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1725: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1726: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1726: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1727: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1728: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1728: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1729: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1730: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1730: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1731: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1734: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1735: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1735: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1736: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1737: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1738: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1739: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1740: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1743: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1744: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1744: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1745: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1746: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1747: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1748: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1749: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1752: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1755: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1755: WARNING: suspect code indent for conditional statements (16, 20)
drivers/staging/otus/hal/hpreg.c:1757: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1759: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1760: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1760: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1761: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1762: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1762: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1763: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1764: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1766: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1766: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1767: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1767: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1767: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:1767: ERROR: space prohibited before that close parenthesis ')'
drivers/staging/otus/hal/hpreg.c:1768: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1769: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1770: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1772: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1772: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1772: ERROR: space required after that ';' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1772: ERROR: spaces required around that '<' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1774: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1774: ERROR: space required after that ',' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1778: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1779: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1780: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1781: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1781: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1786: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1786: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1787: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1787: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1787: ERROR: spaces required around that '==' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1788: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1789: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1789: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1789: ERROR: space prohibited after that open parenthesis '('
drivers/staging/otus/hal/hpreg.c:1789: ERROR: space prohibited before that close parenthesis ')'
drivers/staging/otus/hal/hpreg.c:1789: ERROR: space required before the open parenthesis '('
drivers/staging/otus/hal/hpreg.c:1790: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1791: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1792: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1793: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1794: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1795: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1796: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1797: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1798: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1799: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1799: WARNING: suspect code indent for conditional statements (40, 44)
drivers/staging/otus/hal/hpreg.c:1805: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1807: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1809: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1811: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1814: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1816: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1817: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1820: ERROR: space required before the open parenthesis '('
drivers/staging/otus/hal/hpreg.c:1821: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1826: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1837: CHECK: if this code is redundant consider removing it
drivers/staging/otus/hal/hpreg.c:1839: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1840: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1841: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1842: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1843: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1845: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1845: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1845: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:1845: ERROR: spaces required around that '=' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1845: ERROR: spaces required around that '<' (ctx:VxV)
drivers/staging/otus/hal/hpreg.c:1846: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1847: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1848: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1849: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1850: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1851: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1852: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1853: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1854: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1861: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:1863: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1871: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:1874: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1881: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1883: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1883: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1883: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:1884: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1885: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1887: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1887: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1888: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1888: ERROR: "(foo*)" should be "(foo *)"
drivers/staging/otus/hal/hpreg.c:1890: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1890: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1892: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1892: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1893: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1894: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1894: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1896: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1897: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1898: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1899: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1904: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:1908: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1909: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1915: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1916: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1917: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1917: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1919: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1920: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1922: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1922: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1923: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1923: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1923: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:1924: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1925: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1925: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1926: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1926: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1928: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1930: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1930: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1931: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1932: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1933: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1935: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1935: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1936: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1936: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1937: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1937: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1938: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1938: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1939: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1939: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1940: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1940: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1941: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1941: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1942: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1942: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1943: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1943: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1944: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1944: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1945: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1945: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1946: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1946: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1947: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1947: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1948: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1948: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1949: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1949: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1950: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1950: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1951: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1951: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1952: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1952: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1953: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1953: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1954: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1954: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1955: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1955: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1956: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1956: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1957: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1957: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1958: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1958: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1959: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1959: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1960: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1960: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1961: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1961: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1962: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1962: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1963: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1963: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1964: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1964: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1965: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1965: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1966: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1966: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1967: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1967: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1968: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1968: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1969: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1969: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1970: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1970: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1971: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1971: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1973: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1974: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1976: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:1980: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:1984: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1986: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1986: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:1986: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:1987: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1988: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1989: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:1995: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2003: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2005: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2005: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2005: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2006: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2007: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2008: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2009: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2032: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2035: ERROR: space required after that close brace '}'
drivers/staging/otus/hal/hpreg.c:2039: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2041: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2041: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2042: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2045: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2047: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2048: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2050: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2050: ERROR: else should follow close brace '}'
drivers/staging/otus/hal/hpreg.c:2052: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2055: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2057: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2057: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2057: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2058: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2059: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2060: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2061: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2061: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2062: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2063: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2064: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2065: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2066: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2067: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2073: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2079: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2081: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2081: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2082: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2085: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2085: ERROR: space prohibited after that open parenthesis '('
drivers/staging/otus/hal/hpreg.c:2087: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2087: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2087: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2088: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2089: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2090: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2091: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2093: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2093: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2094: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2094: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2096: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2098: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2098: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2098: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2099: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2100: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2101: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2102: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2103: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2105: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2111: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2117: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2119: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2119: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2120: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2120: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2120: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2121: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2122: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2123: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2124: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2127: ERROR: return is not a function, parentheses are not required
drivers/staging/otus/hal/hpreg.c:2130: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2140: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2142: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2142: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2143: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2143: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2143: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2144: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2145: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2146: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2147: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2152: ERROR: return is not a function, parentheses are not required
drivers/staging/otus/hal/hpreg.c:2155: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2160: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2162: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2162: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2162: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2163: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2164: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2165: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2171: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2181: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2183: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2183: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2183: WARNING: suspect code indent for conditional statements (8, 12)
drivers/staging/otus/hal/hpreg.c:2184: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2185: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2185: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2186: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2187: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2187: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2187: WARNING: suspect code indent for conditional statements (16, 20)
drivers/staging/otus/hal/hpreg.c:2188: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2189: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2190: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2191: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2192: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2193: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2193: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2193: ERROR: else should follow close brace '}'
drivers/staging/otus/hal/hpreg.c:2194: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2195: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2195: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2195: WARNING: suspect code indent for conditional statements (16, 20)
drivers/staging/otus/hal/hpreg.c:2196: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2197: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2198: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2199: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2200: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2201: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2212: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2216: ERROR: that open brace { should be on the previous line
drivers/staging/otus/hal/hpreg.c:2218: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2219: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2220: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2221: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2222: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2223: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2224: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2225: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2226: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2227: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2228: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2229: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2230: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2231: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2232: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2233: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2234: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2235: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2235: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2236: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2238: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2239: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2239: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2240: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2242: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2243: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2244: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2245: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2246: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2247: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2248: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2249: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2250: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2251: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2252: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2253: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2254: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2255: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2256: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2257: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2258: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2259: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2260: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2260: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2261: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2263: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2264: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2265: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2266: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2267: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2268: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2269: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2270: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2271: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2272: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2273: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2274: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2275: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2276: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2277: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2278: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2279: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2280: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2281: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2282: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2283: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2284: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2285: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2286: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2287: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2288: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2289: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2290: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2291: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2292: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2293: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2294: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2295: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2296: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2297: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2298: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2299: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2300: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2301: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2302: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2303: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2304: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2305: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2306: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2307: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2308: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2309: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2310: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2311: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2312: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2313: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2314: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2315: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2316: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2317: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2318: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2319: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2320: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2321: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2322: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2323: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2324: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2325: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2326: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2327: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2328: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2329: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2330: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2331: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2332: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2332: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2333: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2335: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2336: ERROR: code indent should use tabs where possible
drivers/staging/otus/hal/hpreg.c:2338: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpreg.c:2343: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2345: ERROR: "foo* bar" should be "foo *bar"
drivers/staging/otus/hal/hpreg.c:2348: ERROR: spaces required around that '=' (ctx:VxV)
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/otus/hal/hpani.h:102: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpani.h:356: ERROR: do not use C99 // comments
drivers/staging/otus/hal/hpani.h:395: ERROR: do not use C99 // comments
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/asus_oled/asus_oled.c:774: ERROR: code indent should use tabs where possible
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/rt2870/common/rtusb_bulk.c:45: ERROR: that open brace { should be on the previous line
drivers/staging/rt2870/common/rtusb_bulk.c:175: ERROR: space required before the open parenthesis '('
drivers/staging/rt2870/common/rtusb_bulk.c:179: ERROR: space required before the open parenthesis '('
drivers/staging/rt2870/common/rtusb_bulk.c:190: ERROR: "foo * bar" should be "foo *bar"
drivers/staging/rt2870/common/rtusb_bulk.c:276: ERROR: space prohibited after that '&' (ctx:VxW)
drivers/staging/rt2870/common/rtusb_bulk.c:278: ERROR: space prohibited after that '&' (ctx:WxW)
drivers/staging/rt2870/common/rtusb_bulk.c:313: ERROR: space prohibited before that close parenthesis ')'
drivers/staging/rt2870/common/rtusb_bulk.c:329: WARNING: __func__ should be used instead of gcc specific __FUNCTION__
drivers/staging/rt2870/common/rtusb_bulk.c:337: ERROR: space prohibited after that '&' (ctx:VxW)
drivers/staging/rt2870/common/rtusb_bulk.c:404: WARNING: braces {} are not necessary for single statement blocks
drivers/staging/rt2870/common/rtusb_bulk.c:952: WARNING: braces {} are not necessary for single statement blocks
drivers/staging/rt2870/common/rtusb_bulk.c:1017: WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/rt2870/common/rtusb_io.c:27: ERROR: code indent should use tabs where possible
drivers/staging/rt2870/common/rtusb_io.c:404: ERROR: while should follow close brace '}'
drivers/staging/rt2870/common/rtusb_io.c:459: ERROR: while should follow close brace '}'
drivers/staging/rt2870/common/rtusb_io.c:655: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:659: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:669: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:701: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:729: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:739: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:770: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_io.c:1412: ERROR: that open brace { should be on the previous line
drivers/staging/rt2870/common/rtusb_io.c:1434: ERROR: that open brace { should be on the previous line
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/rt2870/common/rtusb_data.c:127: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_data.c:141: ERROR: return is not a function, parentheses are not required
drivers/staging/rt2870/common/rtusb_data.c:154: ERROR: space prohibited before that close parenthesis ')'
drivers/staging/rt2870/common/rtusb_data.c:204: ERROR: space prohibited after that '&' (ctx:VxW)
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the usbduxfast.c file that fixes comments
over 80 warnings found by the checkpatch.pl tool
Signed-off-by: Stewart Robertson <stewart_r@aliencamel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the 8253.h file to fix 80 character line warning
found by checkpatch.pl tool
Signed-off-by: Graham M Howe <gman.1352@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: do not use C99 // comments
+// bit type
ERROR: do not use C99 // comments
+// 802.11 frame related, defined as 802.11 spec
ERROR: do not use C99 // comments
+//#define WLAN_RATES_MAXLEN 255
ERROR: do not use C99 // comments
+//#define WLAN_WEPMAX_KEYLEN 29
ERROR: do not use C99 // comments
+// Frame Type
ERROR: do not use C99 // comments
+// Frame Subtypes
ERROR: do not use C99 // comments
+// Control
ERROR: do not use C99 // comments
+// Data
ERROR: do not use C99 // comments
+// GET & SET Frame Control bit
ERROR: do not use C99 // comments
+// Sequence Field bit
ERROR: do not use C99 // comments
+// Capability Field bit
ERROR: do not use C99 // comments
+// GET & SET Frame Control bit
ERROR: do not use C99 // comments
+// Sequence Field bit
ERROR: do not use C99 // comments
+// Capability Field bit
ERROR: do not use C99 // comments
+#endif //#ifdef __BIG_ENDIAN
ERROR: do not use C99 // comments
+// ERP Field bit
ERROR: do not use C99 // comments
+// Support & Basic Rates field
ERROR: do not use C99 // comments
+// TIM field
ERROR: do not use C99 // comments
+// 3-Addr & 4-Addr
ERROR: do not use C99 // comments
+// IEEE ADDR
ERROR: do not use C99 // comments
+// 802.11 Header Format
ERROR: space required after that close brace '}'
+}__attribute__ ((__packed__))
ERROR: space required after that close brace '}'
+}__attribute__ ((__packed__))
ERROR: space required after that close brace '}'
+}__attribute__ ((__packed__))
ERROR: do not use C99 // comments
+#endif // __80211HDR_H__
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
When the dt3155 driver is built-in (not as a loadable module),
these build errors happen:
drivers/staging/dt3155/dt3155_drv.c:1047: error: implicit declaration of function 'request_irq'
drivers/staging/dt3155/dt3155_drv.c:1048: error: 'IRQF_SHARED' undeclared (first use in this function)
drivers/staging/dt3155/dt3155_drv.c:1048: error: 'IRQF_DISABLED' undeclared (first use in this function)
drivers/staging/dt3155/dt3155_drv.c:1091: error: implicit declaration of function 'free_irq'
so remove the #ifdef MODULE check since it's not needed. Also remove
the CONFIG_PCI check since the Kconfig file already requires that.
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Scott Smedley <ss@aao.gov.au>
Tested-by: Jan III Sobieski <jan3sobi3ski@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
percpu.h is included by sched.h and module.h and thus ends up being
included when building most .c files. percpu.h includes slab.h which
in turn includes gfp.h making everything defined by the two files
universally available and complicating inclusion dependencies.
percpu.h -> slab.h dependency is about to be removed. Prepare for
this change by updating users of gfp and slab facilities include those
headers directly instead of assuming availability. As this conversion
needs to touch large number of source files, the following script is
used as the basis of conversion.
http://userweb.kernel.org/~tj/misc/slabh-sweep.py
The script does the followings.
* Scan files for gfp and slab usages and update includes such that
only the necessary includes are there. ie. if only gfp is used,
gfp.h, if slab is used, slab.h.
* When the script inserts a new include, it looks at the include
blocks and try to put the new include such that its order conforms
to its surrounding. It's put in the include block which contains
core kernel includes, in the same order that the rest are ordered -
alphabetical, Christmas tree, rev-Xmas-tree or at the end if there
doesn't seem to be any matching order.
* If the script can't find a place to put a new include (mostly
because the file doesn't have fitting include block), it prints out
an error message indicating which .h file needs to be added to the
file.
The conversion was done in the following steps.
1. The initial automatic conversion of all .c files updated slightly
over 4000 files, deleting around 700 includes and adding ~480 gfp.h
and ~3000 slab.h inclusions. The script emitted errors for ~400
files.
2. Each error was manually checked. Some didn't need the inclusion,
some needed manual addition while adding it to implementation .h or
embedding .c file was more appropriate for others. This step added
inclusions to around 150 files.
3. The script was run again and the output was compared to the edits
from #2 to make sure no file was left behind.
4. Several build tests were done and a couple of problems were fixed.
e.g. lib/decompress_*.c used malloc/free() wrappers around slab
APIs requiring slab.h to be added manually.
5. The script was run on all .h files but without automatically
editing them as sprinkling gfp.h and slab.h inclusions around .h
files could easily lead to inclusion dependency hell. Most gfp.h
inclusion directives were ignored as stuff from gfp.h was usually
wildly available and often used in preprocessor macros. Each
slab.h inclusion directive was examined and added manually as
necessary.
6. percpu.h was updated not to include slab.h.
7. Build test were done on the following configurations and failures
were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my
distributed build env didn't work with gcov compiles) and a few
more options had to be turned off depending on archs to make things
build (like ipr on powerpc/64 which failed due to missing writeq).
* x86 and x86_64 UP and SMP allmodconfig and a custom test config.
* powerpc and powerpc64 SMP allmodconfig
* sparc and sparc64 SMP allmodconfig
* ia64 SMP allmodconfig
* s390 SMP allmodconfig
* alpha SMP allmodconfig
* um on x86_64 SMP allmodconfig
8. percpu.h modifications were reverted so that it could be applied as
a separate patch and serve as bisection point.
Given the fact that I had only a couple of failures from tests on step
6, I'm fairly confident about the coverage of this conversion patch.
If there is a breakage, it's likely to be something in one of the arch
headers which should be easily discoverable easily on most builds of
the specific arch.
Signed-off-by: Tejun Heo <tj@kernel.org>
Guess-its-ok-by: Christoph Lameter <cl@linux-foundation.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
FC disable is bit 3 of the txmac ctl register, but commit 6720949d55
("Staging: et131x: Kil the txmac type") accidentally changed the code to
set bit 2 instead.
Signed-off-by: Nick Bowler <nbowler@draconx.ca>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Values such as max_brightness should be set before backlights are
registered, but the current API doesn't allow that. Add a parameter to
backlight_device_register and update drivers to ensure that they
set this correctly.
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
Convert some drivers who export a single string as class attribute
to the new class_attr_string functions. This removes redundant
code all over.
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Passing the attribute to the low level IO functions allows all kinds
of cleanups, by sharing low level IO code without requiring
an own function for every piece of data.
Also drivers can extend the attributes with own data fields
and use that in the low level function.
This makes the class attributes the same as sysdev_class attributes
and plain attributes.
This will allow further cleanups in drivers.
Full tree sweep converting all users.
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6: (33 commits)
quota: stop using QUOTA_OK / NO_QUOTA
dquot: cleanup dquot initialize routine
dquot: move dquot initialization responsibility into the filesystem
dquot: cleanup dquot drop routine
dquot: move dquot drop responsibility into the filesystem
dquot: cleanup dquot transfer routine
dquot: move dquot transfer responsibility into the filesystem
dquot: cleanup inode allocation / freeing routines
dquot: cleanup space allocation / freeing routines
ext3: add writepage sanity checks
ext3: Truncate allocated blocks if direct IO write fails to update i_size
quota: Properly invalidate caches even for filesystems with blocksize < pagesize
quota: generalize quota transfer interface
quota: sb_quota state flags cleanup
jbd: Delay discarding buffers in journal_unmap_buffer
ext3: quota_write cross block boundary behaviour
quota: drop permission checks from xfs_fs_set_xstate/xfs_fs_set_xquota
quota: split out compat_sys_quotactl support from quota.c
quota: split out netlink notification support from quota.c
quota: remove invalid optimization from quota_sync_all
...
Fixed trivial conflicts in fs/namei.c and fs/ufs/inode.c
Get rid of the transfer dquot operation - it is now always called from
the filesystem and if a filesystem really needs it's own (which none
currently does) it can just call into it's own routine directly.
Rename the now static low-level dquot_transfer helper to __dquot_transfer
and vfs_dq_transfer to dquot_transfer to have a consistent namespace,
and make the new dquot_transfer return a normal negative errno value
which all callers expect.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jan Kara <jack@suse.cz>
The wireless core made a number of api changes that caused
the winbond driver to break. This patch fixes those errors by
making the needed changes to the driver.
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
There were a number of patches that went into Linus's
tree already that conflicted with other changes in the
staging branch. This merge resolves those merge conflicts.
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Turns out that multiple people sent pretty much the same patch
for the same staging drivers. Commit these in two different
branches and merge them together to get a more complete coverage
of the cleanup and properly credit everyone for the work that they
did.
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes some code style issues about assignments in if conditions.
Signed-off-by: Chihau Chau <chihau@gmail.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes some code style issues like to add one space after a while or
switch statement and before a open parenthesis '(', and to include KERN_
facility level in the printk() functions.
Signed-off-by: Chihau Chau <chihau@gmail.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: do not initialise externals to 0 or NULL
+int comedi_num_legacy_minors = 0;
WARNING: braces {} are not necessary for single statement blocks
+ if (s->subdev_flags & SDF_CMD_READ) {
+ kill_fasync(&dev->async_queue, SIGIO, POLL_IN);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (s->subdev_flags & SDF_CMD_WRITE) {
+ kill_fasync(&dev->async_queue, SIGIO, POLL_OUT);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_labpc.c file that fixes suspect code indent for
conditional statements found by the checkpatch.pl tool
Signed-off-by: Stewart Robertson <stewart_r@aliencamel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_labpc.c file that fixes up all the brace
warnings found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_labpc.c file that fixes up a brace warning
found by the checkpatch.pl tool
Signed-off-by: Maurice Dawson <mauricedawson2699@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_labpc.c file that fixes up a brace warning
found by the checkpatch.pl tool.
Signed-off-by: Maurice Dawson <maurice2699@btinternet.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes some coding style issues like include KERN_ facility levels
in some printk() and one trailing whitespace error.
Signed-off-by: Chihau Chau <chihau@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes some code style issues like else staments after the close
braces '}' and to use __func__ instead of __FUNCTION__.
Signed-off-by: Chihau Chau <chihau@gmail.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch fixes all the errors and the majority of the warnings found with
checkpatch.pl script in hfa384x.h, following Gábor Stefanik hints
Signed-off-by: Alessandro Ghedini <al3xbio@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes some coding style issues like to use __func__ instead
__FUNCTION__, "foo *bar" instead "foo* bar" and a initial comment with
"/* */" instead "//"
Signed-off-by: Chihau Chau <chihau@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_660x.c file that fixes up the brace and 80
character issues found by the checkpatch tool
Signed-off-by: Graham M Howe <gmhowe@btopenworld.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_65xx.c file that fixes up a brace
warning found by the checkpatch.pl tool
Signed-off-by: Graham M Howe <gmhowe@btopenworld.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the pcmad comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes a line over 80 characters and a brace warnings.
Signed-off-by: Chihau Chau <chihau@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Commit 9c1390a923ddb6fba1cf9d7440743369140c6d8a replaced
all u_int's with u32 and u_long's with u64. Unfortunately, a u_long
is still only 32-bits so they should have been replaced with u32 also.
This can be verified by the register definitions in dt3155_io.h. It
specifically states that the memory mapped registers are 32-bit.
Fix this by changing all the u64 to u32.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Greg Kroah-Hartman <greg@kroah.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: Use #include <linux/io.h> instead of <asm/io.h>
+#include <asm/io.h>
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 <andrea.gelmini@gelma.net
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: do not use C99 // comments
+ volatile u_char radioModule;// shows EEPROM, can be overridden at 0x111
ERROR: do not use C99 // comments
+ volatile u_char defaultChannelSet; // shows EEProm, can be overriiden at 0x10A
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: space prohibited between function name and open parenthesis '('
+void MD4Init (MD4_CTX *);
WARNING: space prohibited between function name and open parenthesis '('
+void MD4Update (MD4_CTX *, u8 *, UINT);
WARNING: space prohibited between function name and open parenthesis '('
+void MD4Final (u8 [16], MD4_CTX *);
ERROR: do not use C99 // comments
+#endif //__MD4_H__
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: trailing whitespace
+ * Copyright (C) 1998 Elmer.Joandi@ut.ee, +37-255-13500^I$
WARNING: Use #include <linux/io.h> instead of <asm/io.h>
+#include <asm/io.h>
ERROR: do not use C99 // comments
+//#define ARLAN_DEBUGGING 1
ERROR: "foo * bar" should be "foo *bar"
+extern int arlan_command(struct net_device * dev, int command);
ERROR: trailing whitespace
+ $
ERROR: trailing whitespace
+#define IFDEBUG( L ) if ( (L) & arlan_debug ) $
ERROR: space prohibited after that open parenthesis '('
+#define IFDEBUG( L ) if ( (L) & arlan_debug )
ERROR: space prohibited before that close parenthesis ')'
+#define IFDEBUG( L ) if ( (L) & arlan_debug )
ERROR: trailing whitespace
+#define ARLAN_FAKE_HDR_LEN 12 $
ERROR: space required after that ',' (ctx:VxV)
+ #define ARLAN_DEBUG(a,b) printk(KERN_DEBUG a, b)
^
ERROR: trailing whitespace
+^I#define ARLAN_DEBUG(a,b) $
ERROR: space required after that ',' (ctx:VxV)
+ #define ARLAN_DEBUG(a,b)
^
ERROR: open brace '{' following struct go on the same line
+struct arlan_shmem
+{
ERROR: trailing whitespace
+ /* Header Signature */ $
ERROR: trailing whitespace
+ $
ERROR: trailing whitespace
+ $
ERROR: trailing whitespace
+ $
ERROR: trailing whitespace
+ $
ERROR: trailing whitespace
+ int systemId; $
ERROR: open brace '{' following struct go on the same line
+struct TxParam
+{
ERROR: "foo * bar" should be "foo *bar"
+ struct arlan_shmem __iomem * card;
ERROR: "foo * bar" should be "foo *bar"
+ struct arlan_shmem * conf;
ERROR: trailing whitespace
+ struct arlan_conf_stru * Conf;^I $
ERROR: "foo * bar" should be "foo *bar"
+ struct arlan_conf_stru * Conf;
ERROR: trailing whitespace
+ $
ERROR: space required after that ',' (ctx:VxV)
+#define READSHM(to,from,atype) {\
^
ERROR: space required after that ',' (ctx:VxV)
+#define READSHM(to,from,atype) {\
^
ERROR: space required after that ',' (ctx:VxO)
+ memcpy_fromio(&(tmp),&(from),sizeof(atype));\
^
ERROR: space required before that '&' (ctx:OxV)
+ memcpy_fromio(&(tmp),&(from),sizeof(atype));\
^
ERROR: space required after that ',' (ctx:VxV)
+ memcpy_fromio(&(tmp),&(from),sizeof(atype));\
^
ERROR: space required after that ',' (ctx:VxV)
+#define READSHMEM(from,atype)\
^
ERROR: space required after that ',' (ctx:VxV)
+#define WRITESHM(to,from,atype) \
^
ERROR: space required after that ',' (ctx:VxV)
+#define WRITESHM(to,from,atype) \
^
ERROR: space required after that ',' (ctx:VxO)
+ memcpy_toio(&(to),&tmpSHM,sizeof(atype));\
^
ERROR: space required before that '&' (ctx:OxV)
+ memcpy_toio(&(to),&tmpSHM,sizeof(atype));\
^
ERROR: space required after that ',' (ctx:VxV)
+ memcpy_toio(&(to),&tmpSHM,sizeof(atype));\
^
ERROR: space required after that ',' (ctx:VxV)
+#define DEBUGSHM(levelSHM,stringSHM,stuff,atype) \
^
ERROR: space required after that ',' (ctx:VxV)
+#define DEBUGSHM(levelSHM,stringSHM,stuff,atype) \
^
ERROR: space required after that ',' (ctx:VxV)
+#define DEBUGSHM(levelSHM,stringSHM,stuff,atype) \
^
ERROR: space required after that ',' (ctx:VxO)
+ memcpy_fromio(&tmpSHM,&(stuff),sizeof(atype));\
^
ERROR: space required before that '&' (ctx:OxV)
+ memcpy_fromio(&tmpSHM,&(stuff),sizeof(atype));\
^
ERROR: space required after that ',' (ctx:VxV)
+ memcpy_fromio(&tmpSHM,&(stuff),sizeof(atype));\
^
ERROR: space required after that ',' (ctx:VxV)
+ IFDEBUG(levelSHM) printk(stringSHM,tmpSHM);\
^
ERROR: space required after that ',' (ctx:VxO)
+ writeb(val,&(to))
^
ERROR: space required before that '&' (ctx:OxV)
+ writeb(val,&(to))
^
ERROR: space required after that ',' (ctx:VxO)
+ writew(val,&(to))
^
ERROR: space required before that '&' (ctx:OxV)
+ writew(val,&(to))
^
ERROR: space required after that ',' (ctx:VxO)
+ writel(val,&(to))
^
ERROR: space required before that '&' (ctx:OxV)
+ writel(val,&(to))
^
ERROR: space prohibited after that open parenthesis '('
+ ( ( READSHMB(((struct arlan_private *)netdev_priv(dev))->card->registrationMode) > 0) && \
ERROR: space prohibited after that open parenthesis '('
+ ( READSHMB(((struct arlan_private *)netdev_priv(dev))->card->registrationStatus) == 0) )
ERROR: space prohibited before that close parenthesis ')'
+ ( READSHMB(((struct arlan_private *)netdev_priv(dev))->card->registrationStatus) == 0) )
ERROR: code indent should use tabs where possible
+ ^IREADSHMB(((struct arlan_private *)netdev_priv(dev))->card->cntrlRegImage)$
ERROR: space required before the open brace '{'
+#define writeControlRegister(dev, v){\
ERROR: space required after that ',' (ctx:WxV)
+ WRITESHMB(((struct arlan_private *)netdev_priv(dev))->card->cntrlRegImage ,((v) &0xF) );\
^
ERROR: need consistent spacing around '&' (ctx:WxV)
+ WRITESHMB(((struct arlan_private *)netdev_priv(dev))->card->cntrlRegImage ,((v) &0xF) );\
^
ERROR: space prohibited before that close parenthesis ')'
+ WRITESHMB(((struct arlan_private *)netdev_priv(dev))->card->cntrlRegImage ,((v) &0xF) );\
ERROR: space required after that ',' (ctx:WxV)
+ WRITESHMB(((struct arlan_private *)netdev_priv(dev))->card->controlRegister ,(v) );}
^
ERROR: space required after that ';' (ctx:VxV)
+ WRITESHMB(((struct arlan_private *)netdev_priv(dev))->card->controlRegister ,(v) );}
^
ERROR: space prohibited before that close parenthesis ')'
+ WRITESHMB(((struct arlan_private *)netdev_priv(dev))->card->controlRegister ,(v) );}
WARNING: suspect code indent for conditional statements (3, 6)
+ if (cr & ARLAN_CHANNEL_ATTENTION){ \
+ writeControlRegister(dev, (cr & ~ARLAN_CHANNEL_ATTENTION));\
ERROR: space required before the open brace '{'
+ if (cr & ARLAN_CHANNEL_ATTENTION){ \
ERROR: space required after that close brace '}'
+ }else \
ERROR: space required before the open brace '{'
+#define clearChannelAttention(dev){ \
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_CHANNEL_ATTENTION);}
^
ERROR: space required after that ';' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_CHANNEL_ATTENTION);}
^
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_RESET);}
^
ERROR: space required after that ';' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_RESET);}
^
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_RESET);}
^
ERROR: space required after that ';' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_RESET);}
^
ERROR: space required before the open brace '{'
+#define setInterruptEnable(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_INTERRUPT_ENABLE) ;}
^
ERROR: space required after that ';' (ctx:WxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_INTERRUPT_ENABLE) ;}
^
ERROR: space required before the open brace '{'
+#define clearInterruptEnable(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_INTERRUPT_ENABLE) ;}
^
ERROR: space required after that ';' (ctx:WxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_INTERRUPT_ENABLE) ;}
^
ERROR: space required before the open brace '{'
+#define setClearInterrupt(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_CLEAR_INTERRUPT) ;}
^
ERROR: space required after that ';' (ctx:WxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_CLEAR_INTERRUPT) ;}
^
ERROR: space required before the open brace '{'
+#define clearClearInterrupt(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_CLEAR_INTERRUPT);}
^
ERROR: space required after that ';' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_CLEAR_INTERRUPT);}
^
ERROR: space required before the open brace '{'
+#define setPowerOff(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) | (ARLAN_POWER && ARLAN_ACCESS));\
^
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_ACCESS);}
^
ERROR: space required after that ';' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_ACCESS);}
^
ERROR: space required before the open brace '{'
+#define setPowerOn(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~(ARLAN_POWER)); }
^
ERROR: space required before the open brace '{'
+#define arlan_lock_card_access(dev){\
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_ACCESS);}
^
ERROR: space required after that ';' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) & ~ARLAN_ACCESS);}
^
ERROR: space required before the open brace '{'
+#define arlan_unlock_card_access(dev){\
ERROR: trailing whitespace
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_ACCESS ); } $
ERROR: space required after that ',' (ctx:VxV)
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_ACCESS ); }
^
ERROR: space prohibited before that close parenthesis ')'
+ writeControlRegister(dev,readControlRegister(dev) | ARLAN_ACCESS ); }
ERROR: trailing whitespace
+ $
ERROR: trailing whitespace
+ $
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: code indent should use tabs where possible
+ 0 = [-10,10]$
ERROR: code indent should use tabs where possible
+ 0 = [-10,10]$
WARNING: braces {} are not necessary for single statement blocks
+ if (stat & (DT_S_COMPOSITE_ERROR | DT_S_READY)) {
+ return stat;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (stat & DT_S_COMPOSITE_ERROR) {
+ return stat;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (stat & DT_S_READY) {
+ return 0;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (stat & DT_S_COMPOSITE_ERROR) {
+ return stat;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (stat & DT_S_READY) {
+ return 0;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (!(stat & DT_S_READY)) {
+ printk("dt2801: !ready in dt2801_writecmd(), ignoring\n");
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (!timeout) {
+ printk("dt2801: timeout 1 status=0x%02x\n", stat);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (!timeout) {
+ printk("dt2801: timeout 2 status=0x%02x\n", stat);
+ }
WARNING: braces {} are not necessary for any arm of this statement
+ if (stat == -ETIME) {
[...]
+ } else {
[...]
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for any arm of this statement
+ if (data[0] == COMEDI_OUTPUT) {
[...]
+ } else {
[...]
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for single statement blocks
+ if (fifo_empty) {
+ break;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (irq) {
+ printk(", irq %u", irq);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: Macros with complex values should be enclosed in parenthesis
+#define N_WORDS 3000*64
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: do not initialise statics to 0 or NULL
+static unsigned int ui_Temp = 0;
WARNING: braces {} are not necessary for any arm of this statement
+ if (data[1]) {
[...]
+ } else
[...]
WARNING: braces {} are not necessary for single statement blocks
+ for (i = 0; i < devpriv->ui_AiNbrofChannels; i++) {
+ devpriv->ui_AiChannelList[i] = data[4 + i];
+ }
ERROR: that open brace { should be on the previous line
+ } else /* EOC */
+ {
WARNING: braces {} are not necessary for any arm of this statement
+ if (data[1]) {
[...]
+ } else {
[...]
ERROR: that open brace { should be on the previous line
+ if (insn->unused[0] == 222) /* second insn read */
+ {
WARNING: braces {} are not necessary for single statement blocks
+ for (i = 0; i < insn->n; i++) {
+ data[i] = devpriv->ui_AiReadData[i];
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (cmd->start_src != TRIG_NOW && cmd->start_src != TRIG_EXT) {
+ err++;
+ }
ERROR: that open brace { should be on the previous line
+ if (cmd->scan_begin_src == TRIG_TIMER) /* Test Delay timing */
+ {
ERROR: that open brace { should be on the previous line
+ if (cmd->convert_src == TRIG_TIMER) /* Test Acquisition timing */
+ {
WARNING: braces {} are not necessary for any arm of this statement
+ if (cmd->stop_src == TRIG_COUNT) {
[...]
+ } else {
[...]
WARNING: braces {} are not necessary for single statement blocks
+ if (devpriv->b_ExttrigEnable == APCI3120_ENABLE) {
+ i_APCI3120_ExttrigEnable(dev); /* activate EXT trigger */
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (dmalen0 > (devpriv->ui_AiDataLength)) {
+ dmalen0 = devpriv->ui_AiDataLength;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (dmalen1 > (devpriv->ui_AiDataLength)) {
+ dmalen1 = devpriv->ui_AiDataLength;
+ }
WARNING: braces {} are not necessary for any arm of this statement
+ if (CR_RANGE(chanlist[i]) < APCI3120_BIPOLAR_RANGES) {
[...]
+ } else {
[...]
ERROR: that open brace { should be on the previous line
+ if (devpriv->b_EocEosInterrupt == APCI3120_ENABLE) /* enable this in without DMA ??? */
+ {
ERROR: that open brace { should be on the previous line
+ if (data[0] == APCI3120_TIMER) /* initialize timer */
+ {
ERROR: that open brace { should be on the previous line
+ } else /* Initialize Watch dog */
+ {
ERROR: that open brace { should be on the previous line
+ if (data[0] == 2) /* write new value */
+ {
ERROR: that open brace { should be on the previous line
+ if (devpriv->b_Timer2Mode == APCI3120_TIMER) /* start timer */
+ {
ERROR: that open brace { should be on the previous line
+ } else /* start watch dog */
+ {
ERROR: that open brace { should be on the previous line
+ if (devpriv->b_Timer2Mode == APCI3120_TIMER) /* start timer */
+ {
ERROR: that open brace { should be on the previous line
+ } else /* Read watch dog status */
+ {
WARNING: braces {} are not necessary for single statement blocks
+ if (!devpriv->b_OutputMemoryStatus) {
+ ui_Temp = 0;
+
+ } /* if(!devpriv->b_OutputMemoryStatus ) */
ERROR: code indent should use tabs where possible
+ data[0] Value to be written$
ERROR: code indent should use tabs where possible
+ data[1] :1 Set digital o/p ON$
ERROR: code indent should use tabs where possible
+ data[1] 2 Set digital o/p OFF with memory ON$
ERROR: that open brace { should be on the previous line
+ if (ui_Range) /* if 1 then unipolar */
+ {
ERROR: that open brace { should be on the previous line
+ } else /* if 0 then bipolar */
+ {
ERROR: that open brace { should be on the previous line
+ do /* Waiting of DA_READY BIT */
+ {
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: code indent should use tabs where possible
+ NONE$
ERROR: do not initialise statics to 0 or NULL
+static struct pcmcia_device *cur_dev = NULL;
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for single statement blocks
+ if (dev->iobase) {
+ comedi_pci_disable(devpriv->pci_dev);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for any arm of this statement
+ if (it->options[2]) {
[...]
+ } else {
[...]
WARNING: braces {} are not necessary for single statement blocks
+ if (devpriv->registers) {
+ comedi_pci_disable(devpriv->pci_dev);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for any arm of this statement
+ if (thisboard->has_dio) {
[...]
+ } else {
[...]
WARNING: braces {} are not necessary for single statement blocks
+ if (devpriv->BADR0) {
+ comedi_pci_disable(devpriv->pci_dev);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for single statement blocks
+ if (devpriv->dio_reg_base) {
+ comedi_pci_disable(devpriv->pci_dev);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ for (i = 0; i < thisboard->n_8255; i++) {
+ subdev_8255_cleanup(dev, dev->subdevices + i);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for single statement blocks
+ if (devpriv->dac) {
+ comedi_pci_disable(devpriv->pci_dev);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (inw_p(devpriv->dac + DACALIBRATION1) & SERIAL_OUT_BIT) {
+ value |= 1 << (value_width - i);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ for (i = 0; i < max_num_caldacs; i++) {
+ cal2_bits |= DESELECT_CALDAC_BIT(i);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ for (i = 0; i < max_num_caldacs; i++) {
+ cal2_bits |= DESELECT_CALDAC_BIT(i);
+ }
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: "foo * bar" should be "foo *bar"
+ uint8_t * data);
WARNING: braces {} are not necessary for single statement blocks
+ if (devpriv->s5933_config) {
+ comedi_pci_disable(devpriv->pci_dev);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (cmd->stop_src == TRIG_COUNT) {
+ devpriv->count = cmd->chanlist_len * cmd->stop_arg;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (cmd->stop_src == TRIG_COUNT) {
+ devpriv->ao_count = cmd->chanlist_len * cmd->stop_arg;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (cmd->stop_src == TRIG_COUNT) {
+ devpriv->ao_count -= num_points;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (dev->attached == 0) {
+ return IRQ_NONE;
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if ((status & (INT | EOAI | LADFUL | DAHFI | DAEMI)) == 0) {
+ comedi_error(dev, "spurious interrupt");
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (status & (DAHFI | DAEMI)) {
+ handle_ao_interrupt(dev, status);
+ }
WARNING: braces {} are not necessary for single statement blocks
+ if (async->cmd.stop_src == TRIG_COUNT) {
+ devpriv->ao_count -= num_points;
+ }
ERROR: "foo * bar" should be "foo *bar"
+ uint8_t * data)
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: do not initialise externals to 0 or NULL
+unsigned int ui_InterruptStatus = 0;
WARNING: braces {} are not necessary for single statement blocks
+ else {
+ outl(0x6, devpriv->iobase + APCI1032_DIGITAL_IP_IRQ);
+ } /* else if(data[1] == ADDIDATA_OR) */
WARNING: braces {} are not necessary for single statement blocks
+ if (data[1] == 1) {
+ *data = ui_InterruptStatus;
+ } /* if(data[1]==1) */
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ERROR: code indent should use tabs where possible
+ const struct comedi_lrange *pr_AiRangelist;^I/* rangelist for A/D */$
ERROR: code indent should use tabs where possible
+ const struct comedi_lrange *pr_AoRangelist;^I/* rangelist for D/A */$
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
WARNING: braces {} are not necessary for any arm of this statement
+ if (i_Counter == 0) {
[...]
+ } else {
[...]
Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
BKL isn't anymore present into these files thus it is no necessary still include smp_lock.h.
Signed-off-by: Alessio Igor Bogani <abogani@texware.it>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
according to 'linux device drivers',
ENOENT, ECONNRESET, ESHUTDOWN are not real transmission errors.
also, a little cosmetics.
Signed-off-by: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Because the Firmware is loaded from RTL8192SU/rtl8192sfw.bin,
it it save, to remove it from r8192SU_HWImg.c
Signed-off-by: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes some RT-triggered compile errors and typos.
Signed-off-by: Sven-Thorsten Dietrich <sdietrich@novell.com>
Acked-by: Marek Lindner <lindner_marek@yahoo.de>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch does following improvements:
1. Follow kernel style for comments
2. Reorganize code for readability improvement
3. Use PCI helper macros
4. Use __devinit, __devexit, __devexit_p at necessary places
5. Mark functions and data as static when it is not exported
Signed-off-by: Ameya Palande <2ameya@gmail.com>
Cc: Naren Sankar <nsankar@broadcom.com>
Cc: Jarod Wilson <jarod@wilsonet.com>
Cc: Scott Davilla <davilla@4pi.com>
Cc: Manu Abraham <abraham.manu@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the prism2usb.c that fixes one line which was over
the 80 character limit by wrapping it to a new line and removes a
commented out configuration for a ZyDAS ZD1201 device since support for
this device with this driver is now deprecated.
Signed-off-by: Jim Dog <jimdog@northern-indymedia.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
sparse warns that:
drivers/staging/pohmelfs/inode.c:1797: warning: initialization from incompatible pointer type
so fix pohmelfs_write_inode() to have the expected parameters.
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Evgeniy Polyakov <zbr@ioremap.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Fix prism2fw.c printk format warning:
drivers/staging/wlan-ng/prism2fw.c:209: warning: format '%d' expects type 'int', but argument 3 has type 'size_t'
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The rtl8192* drivers in staging use semaphores, so they need
to #include <linux/semaphore.h>.
(similar to staging-rtl8187se-needs-semaphore.h.patch)
drivers/staging/rtl8192e/ieee80211/ieee80211.h:2038: error: field 'ips_sem' has incomplete type
drivers/staging/rtl8192e/ieee80211/ieee80211.h:2249: error: field 'wx_sem' has incomplete type
drivers/staging/rtl8192e/ieee80211/ieee80211.h:2250: error: field 'scan_sem' has incomplete type
drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c:442: error: implicit declaration of function 'down'
drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c:500: error: implicit declaration of function 'up'
drivers/staging/rtl8192e/ieee80 211/ieee80211_softmac.c:3165: error: implicit declaration of function 'sema_init'
drivers/staging/rtl8192su/ieee80211/ieee80211.h:1330: error: field 'wx_sem' has incomplete type
drivers/staging/rtl8192su/ieee80211/ieee80211.h:1331: error: field 'scan_sem' has incomplete type
drivers/staging/rtl8192u/ieee80211/ieee80211.h:2010: error: field 'wx_sem' has incomplete type
drivers/staging/rtl8192u/ieee80211/ieee80211.h:2011: error: field 'scan_sem' has incomplete type
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Increased timeout for digital output dma loading, due to report of it
being too short for some hardware.
Signed-off-by: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the amplc_pc236 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the amplc_dio200 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the aio_iiro_16 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_labpc.c file that fixes the brace warnings and
comments over 80 characters found by the checkpatch.pl tool. Some code
still goes over 80 characters because I didn't know what to do with it.
Signed-off-by: Stewart Robertson <stewart_r@aliencamel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Make sure that batman-adv does not process packets before its
initialization is complete. Some sanity checks added to the receiver
function.
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
skb_share_check() returns NULL if it can't allocate more memory but
it still frees the skbuff.
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The vis code takes a copy of the data inside the skbuf if it is interesting
for us, so we always need to release the skbuf.
Reported-by: Linus Luessing <linus.luessing@web.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes the bug discovered by Marek Lindner which did not allow
turning on the vis-server before an interface has been added. With this
patch we are using a global atomic variable for activating and
deactiating the vis-server-mode, which can be used before
inserting an interface.
Signed-off-by: Linus Luessing <linus.luessing@web.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch removes a variable that became obsolete since the skb handling
replaced the packet handling thread.
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The routing code has 2 sections which warn about ttl exceeded. The
corresponding warnings were identical which makes it hard to debug. In
addition, batman-adv does not need to warn about ttl exceeded in case
we encountered an echo request as this is commonly used to generate
traceroute graphs.
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
As in other parts of batman-adv, we should not lock while sending a packet but
keep the lock held for as short as possible. Additionally, we should check
whether the interface is active, otherwise batman_if->net_dev might not be
available ...
Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Acked-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The code which uses the raw packet sockets was removed. The only related
dependencies are the skb and netdev handling code. This is provided by
NET in Kconfig.
Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staticise a number of functions which aren't exported outside their
source files, and also add a __user annotation for the buffer in
iio_event_chrdev_read().
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the ni_daq_700.c file that fixes up brace and 80 character warnings
found by the checkpatch.pl tool
Signed-off-by: Graham M Howe <gman.1352@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
VMIVME-7805 board has a special control register that has to be used in
order to activate the VME bus access via the Universe II bridge.
This control register also handles endianess convertion.
Signed-off-by: Arthur Benilov <arthur.benilov@iba-group.com>
Signed-off-by: Vincent Bossier <vincent.bossier@iba-group.com>
Acked-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For VME device I/O operations on master windows the user driver tends
to use kern_buf buffer array which is not allocated. This causes an error
when reading from master window device files.
Signed-off-by: Arthur Benilov <arthur.benilov@iba-group.com>
Signed-off-by: Vincent Bossier <vincent.bossier@iba-group.com>
Acked-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
When requesting slave resources A16 addressing mode flag is used to find
available windows. Since the ca91cx42 bridge only supports two A16 slave windows
but four are requested, the driver fails to initialize. The flag has been
changed to A24, which is supported by all slave windows.
Signed-off-by: Arthur Benilov <arthur.benilov@iba-group.com>
Signed-off-by: Vincent Bossier <vincent.bossier@iba-group.com>
Acked-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Remove the code from the drivers that we are not going to implement before
submitting for review.
Signed-off-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add support for Master Read-Modify-Write cycles on the ca91cx42.
Signed-off-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The ca91cx42 driver currently incorrectly handles master windows, setting
and retrieving the sizing parameters incorrectly. Also, in the slave window
handling, it uses an incorrectly set variable.
Signed-off-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add support for the DMA controller in the ca91cx42 bridge.
Signed-off-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The vme_master_resource structure contains an item called "pci_resource".
Rename to make bus agnostic.
Signed-off-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
At the moment the vme bridge drivers are written in a way that only
allows them to support one bridge at a time. Modify the drivers to
enable more than one bridge to be present per board.
Signed-off-by: Martyn Welch <martyn.welch@ge.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Check the directions in which the DMA controller is expected to operate
before giving control of a resource.
Signed-off-by: Martyn Welch <martyn.welch@gefanuc.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch corrects author email addresses and Copyright notices as a
result of the split up of the GE Fanuc joint venture.
Signed-off-by: Martyn Welch <martyn.welch@gefanuc.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
rtl8187se needs to include semaphore.h to prevent build errors:
drivers/staging/rtl8187se/ieee80211/ieee80211.h:1004: error: field 'wx_sem' has incomplete type
drivers/staging/rtl8187se/ieee80211/ieee80211.h:1005: error: field 'scan_sem' has incomplete type
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Specify Kconfig dependencies, and include warnings for building as a module
udlfb is dependent on FB_DEFERRED_IO, FB_SYS_*, and FB_MODE_HELPERS
Because many kernels do not include defio (which cannot be built
as a module), yet users want to be able to build udlfb as a module later,
udlfb has ifdefs and these dependency warnings to help udlfb build with or
without certain dependencies, but also print warnings for any lost function.
Even though this kind of flexibility isn't common, we've gotten feedback
from a significant portion of users that they were frustrated without it.
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add support for fbdev mmap clients who don't send damage ioctls
Because DisplayLink devices are out on the other end of usb, their
"framebuffer" is just normal system memory. So memory mapped writes
don't automatically trigger anything. So up to this point, standard
fbdev clients who rely on mmap() will get an unchanging screen.
This patch makes udlfb a client of Jaya Kumar's defio framework - which sets
up page fault triggers, and those faults are accumulated and sent to udlfb
on a defferred basis, to process as damage notifications for the framebuffer.
Because this involves more overhead than a notification directly from
the application (e.g. just passing on X DAMAGE extension notifications),
a sysfs attribute is provided by udlfb to control defio support.
/sys/class/graphics/fb*/use_defio - writing a "0" to this file before
calling mmap() causes defio to not be initialized - instead udlfb
will rely on getting damage notifications directly through the damage ioctl.
There are unsolved rendering problems with defio (horizontal dead regions
on framebuffer, that accumulate over time) which still needs a fix.
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Rework rendering for improved performance
Approx 10-20% gain avg across several "benchmarks": x11perf, gtkperf, glxgears
Moves from a single pre-alloc'd urb protected by a long-held mutex
To a list of (4) pre-alloc'd urbs which can be dispatched asynchonously
Improved rendering algorithm to hardware with lower CPU consumption,
fewer system memory accesses, and slightly higher compression.
Better scalability to multiple processors, especially with
multiple framebuffers active.
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Rework probe to use refcounts and std functions
Because the different parts of the driver (usb, fbdev) tear down
in different orders, the driver previously could crash accessing
data that had already been freed. Refcounting system used to handle.
Reworked probe to make use of refcounts, set mode using std fbops,
and set up sysfs and pre-allocated urbs.
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add sysfs controls for edid and performance metrics
There are 8 new files exposed in /sys/class/graphics/fb*
edid - returns 128 byte edid blog, suitable for parsing with parse-edid
metrics_bytes_identical
metrics_bytes_rendered
metrics_bytes_sent
metrics_cpu_kcycles_used
metrics_misc
and metrics_reset, which resets all perf metrics to zero
The 6 perf metrics are of type atomic_t.
So these metrics return precise results for short benchmarks, but
any test approx a minute or longer runtime may roll over.
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Move to more consistent naming scheme
All udlfb functions start with udlfb_
All functions for udlfb's fbdev interface start with udlfb_ops_
All functinos for udlfb's usb interface start with udlfb_usb_
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add functions to pre-allocate and free usb bulk urbs for core render path.
Udlfb currently allocates a single urb, guarded by a mutex, that is a key
bottleneck. Because udlfb sends so much data, preallocation is most efficient.
Functions will be used by new rendering functions in later patches.
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Reorganize the location of a few things to be closer to related code
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Eliminate checkpatch.pl warnings and errors so later patches in series are clean
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For the RTL8187SE, the variable priv->rf_chip is always RF_ZEBRA4
and priv->RegThreeWireMode is always HW_THREE_WIRE_SI. Remove these
2 variables.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Cleanups as suggested by checkpatch.pl utiltiy.
.o's from before and after cleanup have matching SHA1s.
Signed-off-by: Svenne Krap <svenne@krap.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to ni_atmio.c file to fix up bracing style problems
found by checkpatch.pl tool
Signed-off-by: Graham M Howe <gman.1352@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
It's clear from the indent levels and the context that there are supposed to
be curly braces here.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The current driver does not follow the state of the RF switch.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The TODO list includes switching this driver to use the common EEPROM
routines in EEPROM_93CX.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Tested by: Bernhard Schiffner <bernhard@schiffner-limbach.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Removed parenthesis from return statements,
split up assignment and if condition
Signed-off-by: Jochen Maes <jochen.maes@sejo.be>
Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the rti800 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add suspend and resume functions (which are currently stubs
returning -ENOSYS)
Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Provides a better explanation of what this
driver is for in the Kconfig file
Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Renames the directory in which the driver files
are located; again for clarity.
Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Patch renames rar_driver to rar_register to clarifiy and differentiate
from rar_handler that will be submitted later
Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
It turns out that Mimio has a userspace solution for this product using
libusb, and the in-kernel driver is just getting in the way now and
causing problems. So they have asked that the in-kernel driver be
removed. As the staging driver wasn't quite working anyway, and Mimio
supports their libusb solution for all distros, I am removing the
in-kernel driver.
The libusb solution can be downloaded from:
http://www.mimio.com/downloads/mimio_studio_software/linux.asp
Cc: <mwilder@cs.nmsu.edu>
Cc: Phil Hannent <phil@hannent.co.uk>
Cc: Marc Rousseau <Marc.Rousseau@mimio.com>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes up the sparse and coding style issues found in the
dt3155_io.c file.
No code is changed, only formatting and removing unused code.
Cc: Scott Smedley <ss@aao.gov.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes almost all checkpatch issues in the ni_pcidio comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the rti802 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the dt2815 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Use DECLARE_WAIT_QUEUE_HEAD_ONSTACK to make lockdep happy
Signed-off-by: Yong Zhang <yong.zhang0@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@suse.de>
Cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: Alexander Beregalov <a.beregalov@gmail.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is a patch to the r8180_93cx6.c file that fixes up spacing issue
warnings found by the checkpatch.pl tool
Signed-off-by: Tim Schofield <tim@weberpafrica.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch to the comedi_fops.c file fixes a brace warning found by the checkpatch.pl tool
Signed-off-by: Paul Elms <paul@uprocera.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch fixes all of the brace style warnings found by the
checkpatch.pl tool
Signed-off-by: Tony Burrows <tony@tonyburrows.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
String constants that are continued on subsequent lines with \
are not good.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Remove iw_handle_def private initialisations as they are no longer need
these.
Signed-off-by: Richard Kennedy <richard@rsk.demon.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
String constants that are continued on subsequent lines with \
are not good.
Fixed a "is tryied" / tried typo
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the adq12b comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the fl512 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the ni_65xx comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the ni_670x comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the ni_660x comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the poc comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the pcm3730 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the pcm3724 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Currently, we crash (issue BUG_ON) if backing swap
disk size is zero. This can happen is user specified
an extended partition or simply a bad disk as backing
swap. A crash is really an unpleasant surprise to user
for such trivial problems.
Now, we check for this condition and simply fail device
initialization if this is the case.
Additional cleanups:
* use static for all functions
* remove extra newline between functions
* memset backing_swap_name to NULL on device reset
Signed-off-by: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ramzswap_free_page() already handles the case for zero filled
pages. So, remove redundant logic for the same in ramzswap_write().
Signed-off-by: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Currently, we return 0 if create_device() fails and 1 otherwise.
Now, proper error code is returned from create_device() and the
same is propagated as module error code from ramzswap_init().
Also added some cleanups for ramzswap_init(), improving function
structure.
Signed-off-by: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
ramzswap block size needs to be set equal to PAGE_SIZE to
avoid receiving any unaligned block I/O requests (happens
due to readahead logic during swapon). These unaligned
accesses produce unnecessary I/O errors, scaring users.
Signed-off-by: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Make sure we flush block device before freeing all metadata
during reset ioctl.
Signed-off-by: Nitin Gupta <ngupta@vflar.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
64-bit stats corruption was observed when ramzswap was
used on SMP systems. To prevent this, use separate spinlock
to protect these stats.
Also, replace stat_*() with rzs_stat*() to avoid possible
conflict with core kernel code.
Eventually, these will be converted to per-cpu counters
if this driver finds use on large scale systems and this
locking is found to affect scalability.
Signed-off-by: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Cleanup the driver and fix a faulty if statement.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Tested by: Bernhard Schiffner <bernhard@schiffner-limbach.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This patch removes the dead uncompiled code in usbip_common.c
Signed-off-by: Himanshu Chauhan <himanshu@symmetricore.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This completes the structures within the txmac block so we can now
propogate a name change and type removal up a layer and clean up TXMAC as
well
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is another one we don't really need to do much to get rid of
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Only used for one trivial thing so turn that into something trivial instead
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
We have lots of tiny files right now that could be one
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This completes the typedef clean up of the rx specific structures, although
there is plenty do on field names and the like
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
All the subtypes are sane so just turn it into something struct and linux
like
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Use the proper pointer types for the higher level pointers to the rx_status
object and kill casts
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
So kill off the top level type and turn it into a struct
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
All the contents of this type are now clean, so kill the top level type
as well
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The contents of MMC_t are clean so kill off the MMC_t typedef
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
All the contents are clean so kill off the top level typedefs
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
We have a clean struct of this now so turn the top level typedefs into a
struct
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is another set of flags as typedef that can be cleaned up.
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Clean up the typdef for fbr_desc itself so we know it is done
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is only used meaningfully as a definition, we never mask and fetch the
bits apart
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This is another 10 bit value with the high bits clear, and where the
type doesn't get used anywhere properly anyway
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Add necessary include to fix build on PowerPC
Signed-off-by: Dave Airlie <airlied@redhat.com>
Signed-off-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
I removed a misplace semicolon. It is clear from the indentation that
TxPwrTracking87SE() was only supposed to be called if CheckTxPwrTracking()
returned true.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
When originally introduced into staging, these drivers had custom
firmware-loading code which checked a version number and CRC at the
end of each blob. This reintroduces those checks, using crc-ccitt
instead of custom code.
The removed firmware will be added to the linux-firmware.git
repository.
Based on work by Darren Salt <linux@youmustbejoking.demon.co.uk>.
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Tested-By: Darren Salt <linux@youmustbejoking.demon.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
These functions do not modify the data they are passed.
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Regardless of the condition, the branches executed the same code
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Regardless of the condition, the branches execute the same code
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For AI command testing, the driver does not need to range-check the
length of the channel list as the comedi core has already checked it.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Check the channel list is valid in step 5 of the AI command test.
Split function check_and_setup_channel_list() in two. Also, remove
unnecessary chanlist_len tests in step 3 of the AI command test as the
comedi core has already checked it.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For multi-channel AI commands, the interrupt handler is missing code to
switch to the next channel. Add some (untested) code to handle this.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For AI command testing, the driver does not need to range-check
the length of the channel list as the comedi core has already
checked it.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For AI commands, the internal channel list used to check incoming
data was only set up partially if the channel list contained
repeats. Set it up fully. Also the current scan position was not
updated properly when moving a block of data from the FIFO unless
a whole number of scans was moved.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For AI commands, the scan counter should be updated after every
scan. It was being updated after every sample except for DMA mode
where it was being updated after every repeated segment of the
channel list.
Also AI commands with multiple channels were being terminated with
an error prematurely except in DMA mode. This was because the
driver was comparing channel numbers received from the hardware
(combined with the sample value) with the expected channel numbers
to check for a "channel dropout". This test was failing
incorrectly because the driver was not keeping the current position
within the (repeated segment of the) channel list up to date.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
For AI commands, the scan counter should be updated after every
scan. In DMA mode, it was being updated after every repeated
segment of the channel list. In non-DMA mode, it was being updated
after every sample.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
The AI scan counter should be updated after every completed scan,
not after every channel. Keep track of current channel.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This fixes all checkpatch issues in the pcl730 comedi driver.
Signed-off-by: Benjamin Adolphi <b.adolphi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>