[PATCHv2 3/3] silicom: remaining checkpatch issues in bypass.c

2013-12-20 Thread Michael Hoefler
change the functionality in any way. Signed-off-by: Michael Hoefler Signed-off-by: Christoph Kohl --- drivers/staging/silicom/bypasslib/bypass.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/drivers/staging/silicom/bypasslib/bypass.c b/drivers/staging/silicom

[PATCHv2 2/3] silicom: fix coding style issues in bypass.c

2013-12-20 Thread Michael Hoefler
: Michael Hoefler Signed-off-by: Christoph Kohl --- drivers/staging/silicom/bypasslib/bypass.c | 32 -- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/drivers/staging/silicom/bypasslib/bypass.c b/drivers/staging/silicom/bypasslib/bypass.c index

[PATCHv2 1/3] silicom: fix whitespace issues in bypass.c

2013-12-20 Thread Michael Hoefler
local declarations - empty line after { The empty line issues were not discovered by checkpatch but in my opinion these changes make perfect sense in terms of readability. Signed-off-by: Michael Hoefler Signed-off-by: Christoph Kohl --- drivers/staging/silicom/bypasslib/bypass.c | 111

[PATCHv2 0/3] staging: silicom: fix code styling according to checkpatch

2013-12-20 Thread Michael Hoefler
some empty lines for a better readability and added missing braces. This series does not perform functional changes to the code because i do not have any hardware to test it. Michael Hoefler (3): silicom: fix whitespace issues in bypass.c silicom: fix coding style issues in bypass.c silicom

[PATCH] silicom: fixed checkpatch issues in bypass.c

2013-12-18 Thread Michael Hoefler
ition - whitespace stuff - a C++ one line comment - parenthesis at return statements - missing __init and __exit macros Signed-off-by: Michael Hoefler Signed-off-by: Christoph Kohl --- drivers/staging/silicom/bypasslib/bypass.c | 168 +++-- 1 file changed, 90 insertions(+