- 28 Mar, 2016 40 commits
-
-
Leslie Klein authored
Fix checkpatch warning: Prefer 'unsigned int' to bare use of 'unsigned' in file comedi_fops.c Signed-off-by:
Leslie Klein <lesliebklein@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Leslie Klein authored
Fix checkpatch warning: Prefer 'unsigned int' to bare use of 'unsigned' in file comedi_buf.c Signed-off-by:
-
H Hartley Sweeten authored
Arnd Bergmann pointed out that gcc-6 warns about passing negative signed integer into swab16() due to the macro expansion of 'outw'. It appears that the register map constants are causing the warnings. Actually, it might just be the (1 << 15) ones... Convert all the constants as suggested by checkpatch.pl: CHECK: Prefer using the BIT macro The BIT() macro will make all the constants explicitly 'unsigned', which helps to avoid the warning. Fix the, unsused, DT2821_CHANCSR_PRESLA() macro. The "Present List Address" (PRESLA) bits in the CHANCSR register are read only. This define was meant to extract the bits from the read value. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reported-by:
Arnd Bergmann <arnd@arndb.de> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Tested-by:
-
Sandhya Bankar authored
Use x instead of x != NULL.This issue is detected by checkpatch.pl script. Signed-off-by:
Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by:
-
Claudiu Beznea authored
This patch defines oui_rfc1042[] and oui_8021h[] arrays from p80211conv.c as const arrays since these are not changed anywhere in code. Signed-off-by:
Claudiu Beznea <claudiu.beznea@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Use sizeof type *pointer instead of sizeof type. Signed-off-by:
-
Parth Sane authored
Fixed checkpatch warning after removing FSF address block as per guidelines. Signed-off-by:
Parth Sane <laerdevstudios@gmail.com> Signed-off-by:
-
Parth Sane authored
Fixed checkpatch warning after removing FSF address block as per guidelines. Signed-off-by:
-
Parth Sane authored
Fixed checkpatch warning after removing FSF address block as per guidelines. Signed-off-by:
-
Parth Sane authored
Fixed checkpatch warning after removing FSF address block as per guidelines. Signed-off-by:
-
Parth Sane authored
Fixed checkpatch warning after removing FSF address paragraph as per guidelines. Signed-off-by:
-
Bhaktipriya Shridhar authored
Some functions like kmalloc/kzalloc return NULL on failure. When NULL represents failure, !x is commonly used. This was done using Coccinelle: @@ expression *e; identifier l1; @@ e = \(kmalloc\|kzalloc\|kcalloc\|devm_kzalloc\)(...); ... - e == NULL + !e Signed-off-by:
Bhaktipriya Shridhar <bhaktipriya96@gmail.com> Signed-off-by:
-
Bhaktipriya Shridhar authored
Some functions like kmalloc/kzalloc return NULL on failure. When NULL represents failure, !x is commonly used. This was done using Coccinelle: @@ expression *e; identifier l1; @@ e = \(kmalloc\|kzalloc\|kcalloc\|devm_kzalloc\)(...); ... - e == NULL + !e Signed-off-by:
-
Bhaktipriya Shridhar authored
Some functions like kmalloc/kzalloc return NULL on failure. When NULL represents failure, !x is commonly used. This was done using Coccinelle: @@ expression *e; identifier l1; @@ e = \(kmalloc\|kzalloc\|kcalloc\|devm_kzalloc\)(...); ... - e == NULL + !e Signed-off-by:
-
Bhaktipriya Shridhar authored
Some functions like kmalloc/kzalloc return NULL on failure. When NULL represents failure, !x is commonly used. This was done using Coccinelle: @@ expression *e; identifier l1; @@ e = \(kmalloc\|kzalloc\|kcalloc\|devm_kzalloc\)(...); ... - e == NULL + !e Signed-off-by:
-
Bhaktipriya Shridhar authored
Change null comparisons of the form x == NULL to !x. This was done using Coccinelle. @@ expression e; @@ - e == NULL + !e Signed-off-by:
-
Bhaktipriya Shridhar authored
Removed Unused macro IS_MAC_ADDRESS_BROADCAST. Grep'd to find occurences. Signed-off-by:
-
Sandhya Bankar authored
Remove unnecessary return statement. Signed-off-by:
-
Sandhya Bankar authored
Remove unnecessary semicolon. Coccinelle sementic patch as follows: @r_case@ position p; @@ switch (...) { case ...: ...;@p } @r_default@ position p; @@ switch (...) { default: ...;@p } @r1@ statement S; position p1; position p != {r_case.p, r_default.p}; identifier label; @@ ( label:; | S@p1;@p ) Signed-off-by: -
Edward Lipinsky authored
This patch fixes the checkpatch.pl warning: WARNING: line over 80 characters Signed-off-by:
Edward Lipinsky <ellipinsky@gmail.com> Signed-off-by:
-
Nik Nyby authored
This fixes a mis-spelled word in a few debug statements. Signed-off-by:
Nik Nyby <nikolas@gnu.org> Signed-off-by:
-
Sandhya Bankar authored
Remove unnecessary semicolon.This issue is found by coccinelle script. Signed-off-by:
-
Ben Marsh authored
This patch removes blank lines in r8192U_wx.c that were flagged by checkpatch.pl Signed-off-by:
Ben Marsh <bmarsh94@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Remove & from function name,when function name passed as an argument to another function. Function name is used as pointer without &. Signed-off-by:
-
Alison Schofield authored
Use * on subsequent lines and trailing */ on a separate line in block comments. Signed-off-by:
Alison Schofield <amsfield22@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Use sizeof type *pointer instead of sizeof type. Signed-off-by:
-
Sandhya Bankar authored
Use sizeof type *pointer instead of sizeof type. Signed-off-by:
-
Sandhya Bankar authored
Remove return statement from void function. Signed-off-by:
-
Sandhya Bankar authored
Remove return statement from void function. Signed-off-by:
-
Sandhya Bankar authored
Remove unnecessary space after cast. Signed-off-by:
-
Sandhya Bankar authored
Remove unnecessary semicolon.This issue is detected by coccinelle script. Signed-off-by:
-
Sandhya Bankar authored
Add parentheses to complex macro. Signed-off-by:
-
Sandhya Bankar authored
Do not initialise statics to 0. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. Signed-off-by:
-
Sandhya Bankar authored
Add parentheses to complex macro. Signed-off-by:
-
Sandhya Bankar authored
No space is necessary after cast.This problem is found by checkpatch.pl script. Signed-off-by:
-
Sandhya Bankar authored
No space is necessary after cast.This patch is found by checkpatch.pl script. Signed-off-by:
-
Sandhya Bankar authored
No space is necessary after cast. Signed-off-by:
-
Heena Sirwani authored
The following patch fixes the following checkpatch.pl warning: CHECK: Alignment should match open parenthesis Signed-off-by:
Heena Sirwani <heenasirwani@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Add space around operator.This patch is found by checkpatch.pl script. Signed-off-by:
-
