Skip to content

staging: octeon: Fix several checkpatch issues

Laura Garcia Liebana outreachy 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest10 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10staging: octeon: Move logical operators on the correct linedrivers/staging/octeon/ethernet-tx.c drivers/staging/octeon/ethernet.c+16 -160 replies
2/10staging: octeon: Fix alignment with open parenthesisdrivers/staging/octeon/ethernet-rx.c drivers/staging/octeon/ethernet-rx.h drivers/staging/octeon/ethernet-tx.c +1 more+13 -101 reply
3/10staging: octeon: Insert blank line after struct declarationdrivers/staging/octeon/ethernet.c+5 -00 replies
4/10staging: octeon: Remove comparison to NULLdrivers/staging/octeon/ethernet-mdio.c drivers/staging/octeon/ethernet-mem.c drivers/staging/octeon/ethernet-rx.c +1 more+7 -70 replies
5/10staging: octeon: Fix block commentsdrivers/staging/octeon/ethernet-rx.c drivers/staging/octeon/ethernet-tx.c+13 -400 replies
6/10staging: octeon: Remove blank lines after open bracedrivers/staging/octeon/ethernet-rx.c drivers/staging/octeon/ethernet-spi.c drivers/staging/octeon/ethernet-tx.c+0 -40 replies
7/10staging: octeon: Fix lines over 80 charactersdrivers/staging/octeon/ethernet-rx.c+7 -30 replies
8/10staging: octeon: Use type s32 instead of int32_tdrivers/staging/octeon/ethernet-tx.c+5 -51 reply
9/10staging: octeon: Remove multiple blank linesdrivers/staging/octeon/ethernet-tx.c+0 -20 replies
10/10staging: octeon: Fix braces in condition statementdrivers/staging/octeon/ethernet-tx.c+2 -10 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 00/10] staging: octeon: Fix several checkpatch issues

This patchset updates the octeon driver in order to fix several checkpatch issues detected in the source code.

Laura Garcia Liebana (10):
staging: octeon: Move logical operators on the correct line
staging: octeon: Fix alignment with open parenthesis
staging: octeon: Insert blank line after struct declaration
staging: octeon: Remove comparison to NULL
staging: octeon: Fix block comments
staging: octeon: Remove blank lines after open brace
staging: octeon: Fix lines over 80 characters
staging: octeon: Use type s32 instead of int32_t
staging: octeon: Remove multiple blank lines
staging: octeon: Fix braces in condition statement

 drivers/staging/octeon/ethernet-mdio.c |  2 +-
 drivers/staging/octeon/ethernet-mem.c  |  4 +--
 drivers/staging/octeon/ethernet-rx.c   | 44 ++++++++++-------------
 drivers/staging/octeon/ethernet-rx.h   |  2 +-
 drivers/staging/octeon/ethernet-spi.c  |  2 --
 drivers/staging/octeon/ethernet-tx.c   | 64 ++++++++++++----------------------
 drivers/staging/octeon/ethernet.c      | 38 +++++++++++---------
 7 files changed, 68 insertions(+), 88 deletions(-)
Signature · 1 line
2.7.0