linux.git/drivers/staging/cxt1e1, branch v3.15

Staging: cxt1e1: Fix externs should be avoided in .c files in comet.c

2014-03-19T16:17:23+00:00

Fix checkpatch.pl issues with externs should be
avoided in comet.c

Signed-off-by: Ebru Akagunduz 
Signed-off-by: Greg Kroah-Hartman

Staging: cxt1e1: Fix use of volatile is usually wrong in comet.c

2014-03-19T16:17:23+00:00

Fix checkpatch.pl issues with use of volatile is
usually wrong in comet.c

Signed-off-by: Ebru Akagunduz 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: remove unneeded mkret() calls

2014-03-18T20:05:13+00:00

The mkret() change a value of error from positive to
negative. This patch is modified to return negative value
when it failed. It doesn't need to call with function
for changing from positive to negative.

Signed-off-by: Daeseok Youn 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: Fix Sparse Warning for Static Declarations in linux.c

2014-03-18T17:42:34+00:00

This patch fixes the following sparse warnings:
drivers/staging/cxt1e1/linux.c:106:13: warning: symbol 'log_level_default' was not declared. Should it be static?
drivers/staging/cxt1e1/linux.c:110:13: warning: symbol 'max_mru_default' was not declared. Should it be static?
drivers/staging/cxt1e1/linux.c:255:1: warning: symbol 'c4_linux_interrupt' was not declared. Should it be static?
drivers/staging/cxt1e1/linux.c:265:1: warning: symbol 'c4_ebus_interrupt' was not declared. Should it be static?

Signed-off-by: Himangi Saraogi 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: Fix do not use // c99 comments

2014-03-18T17:40:26+00:00

This patch fixes "do not use // C99 comments" errors in linux.c

Signed-off-by: Gulsah Kose 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: Fix line over 80 characters.

2014-03-18T17:38:29+00:00

Fix checkpatch.pl issues with line over 80
characters in comet.c

Signed-off-by: Gulsah Kose 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: remove redundant memset() call

2014-03-17T21:22:47+00:00

The banner array doens't need to set to 0.
sprintf() adds a terminating '\0'.

And the sn array can be declared and initialized to zero.
So remove redundant memset() with zero.

Remove unnecessary cast for memcpy().

Signed-off-by: Daeseok Youn 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: remove redundant memset() call

2014-03-17T21:22:47+00:00

The name array doens't need to set to 0. Because
sprintf/snprintf adds a terminating '\0'.

And also it doesn't need to assign name array
address to np pointer.

Signed-off-by: Daeseok Youn 
Signed-off-by: Greg Kroah-Hartman

Merge 3.14-rc7 into staging-next

2014-03-17T18:27:39+00:00

This resolves a merge issue with drivers/staging/cxt1e1/linux.c that was
fixed in a report from Stephen Rothwell

Reported-by: Stephen Rothwell 
Signed-off-by: Greg Kroah-Hartman

staging: cxt1e1: Fix space prohibited between function name and open parenthesis

2014-03-17T05:04:48+00:00

Fix checkpatch.pl issues with space prohibited between function name and
open parenthesis in libsbew.h

Signed-off-by: Gulsah Kose 
Signed-off-by: Peter P Waskiewicz Jr