staging: rts5208: Checkpatch.pl cleanup
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest45 patches 0 repliesNew
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/45 | staging: rts5208: spi.c: Remove unnecessary parenthesisdrivers/ | +7 | 0 replies | |
| 2/45 | staging: rts5208: spi.c: Alignment fixesdrivers/ | +65 | 0 replies | |
| 3/45 | staging: rts5208: sd.h: Alignment fixesdrivers/ | +3 | 0 replies | |
| 4/45 | staging: rts5208: sd.c: Remove unnecessary parenthesesdrivers/ | +47 | 0 replies | |
| 5/45 | staging: rts5208: sd.c: Alignment fixesdrivers/ | +255 | 0 replies | |
| 6/45 | staging: rts5208: sd.c: Long line fixesdrivers/ | +15 | 0 replies | |
| 7/45 | staging: rts5208: sd.c: CamelCase fixesdrivers/ | +84 | 0 replies | |
| 8/45 | staging: rts5208: sd.c: Fix logical continuationsdrivers/ | +15 | 0 replies | |
| 9/45 | staging: rts5208: sd.c: Spacing cleanupdrivers/ | +3 | 0 replies | |
| 10/45 | staging: rts5208: Alignment fixesdrivers/ | +17 | 0 replies | |
| 11/45 | staging: rts5208: Unnecessary parentheses cleanupdrivers/ | +2 | 0 replies | |
| 12/45 | staging: rts5208: Alignment fixdrivers/ | +2 | 0 replies | |
| 13/45 | staging: rts5208: Fix typo in function namedrivers/ | +2 | 0 replies | |
| 14/45 | staging: rts5208: Alignment fixesdrivers/ | +101 | 0 replies | |
| 15/45 | staging: rts5208: Remove parenthesesdrivers/ | +23 | 0 replies | |
| 16/45 | staging: rts5208: CamelCase fixesdrivers/ | +21 | 0 replies | |
| 17/45 | staging: rts5208: Remove multiple assignmentdrivers/ | +7 | 0 replies | |
| 18/45 | staging: rts5208: Add braces to if()drivers/ | +4 | 0 replies | |
| 19/45 | staging: rts5208: Add space around operatordrivers/ | +1 | 0 replies | |
| 20/45 | staging: rts5208: Alignment fixesdrivers/ | +3 | 0 replies | |
| 21/45 | staging: rts5208: Long lines fixesdrivers/ | +57 | 0 replies | |
| 22/45 | staging: rts5208: Prefer using BIT macrodrivers/ | +15 | 0 replies | |
| 23/45 | staging: rts5208: Parenthesis alignmentdrivers/ | +7 | 0 replies | |
| 24/45 | staging: rts5208: Unecessary paranthesesdrivers/ | +1 | 0 replies | |
| 25/45 | staging: rts5208: Parenthesis alignmentdrivers/ | +22 | 0 replies | |
| 26/45 | staging: rts5208: Unnecessary parenthesesdrivers/ | +24 | 0 replies | |
| 27/45 | staging: rts5208: Spacingdrivers/ | +1 | 0 replies | |
| 28/45 | staging: rts5208: Comparison to NULLdrivers/ | +1 | 0 replies | |
| 29/45 | staging: rts5208: rtsx.c: Unnecessary parenthesesdrivers/ | +5 | 0 replies | |
| 30/45 | staging: rts5208: rtsx.c: Alignmentdrivers/ | +2 | 0 replies | |
| 31/45 | staging: rts5208: rtsx.c: Multiple assignmentsdrivers/ | +4 | 0 replies | |
| 32/45 | staging: rts5208: rtsx.c: Spacingdrivers/ | +4 | 0 replies | |
| 33/45 | staging: rts5208: rtsx.c: CamelCasedrivers/ | +4 | 0 replies | |
| 34/45 | staging: rts5208: rtsx.c: Spacingdrivers/ | +0 | 0 replies | |
| 35/45 | staging: rts5208: rtsx.c: Alloc sizeof structdrivers/ | +1 | 0 replies | |
| 36/45 | staging: rts5208: ms.h: Parenthesis alignmentdrivers/ | +2 | 0 replies | |
| 37/45 | staging: rts5208: ms.c: Parenthesis alignmentdrivers/ | +150 | 0 replies | |
| 38/45 | staging: rts5208: ms.c: Long Linesdrivers/ | +63 | 0 replies | |
| 39/45 | staging: rts5208: ms.c: Multiple assignmentsdrivers/ | +2 | 0 replies | |
| 40/45 | staging: rts5208: ms.c: Braces fixdrivers/ | +2 | 0 replies | |
| 41/45 | staging: rts5208: CamelCase fixdrivers/ | +2 | 0 replies | |
| 42/45 | staging: rts5208: rtsx_scsi.h: Long linesdrivers/ | +6 | 0 replies | |
| 43/45 | staging: rts5208: rtxs_chip.h: Long linesdrivers/ | +10 | 0 replies | |
| 44/45 | staging: rts5208: rtsx_chip.c: Long linesdrivers/ | +10 | 0 replies | |
| 45/45 | staging: rts5208: rtsx.c: Alignment fixdrivers/ | +4 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 00/45] staging: rts5208: Checkpatch.pl cleanup
This set is a general clenup of issues found by checkpatch.pl. There are a few checks still detected and these either require more knowledge of the driver or should possibly just be left alone.
Wayne Porter (45):
staging: rts5208: spi.c: Remove unnecessary parenthesis
staging: rts5208: spi.c: Alignment fixes
staging: rts5208: sd.h: Alignment fixes
staging: rts5208: sd.c: Remove unnecessary parentheses
staging: rts5208: sd.c: Alignment fixes
staging: rts5208: sd.c: Long line fixes
staging: rts5208: sd.c: CamelCase fixes
staging: rts5208: sd.c: Fix logical continuations
staging: rts5208: sd.c: Spacing cleanup
staging: rts5208: Alignment fixes
staging: rts5208: Unnecessary parentheses cleanup
staging: rts5208: Alignment fix
staging: rts5208: Fix typo in function name
staging: rts5208: Alignment fixes
staging: rts5208: Remove parentheses
staging: rts5208: CamelCase fixes
staging: rts5208: Remove multiple assignment
staging: rts5208: Add braces to if()
staging: rts5208: Add space around operator
staging: rts5208: Alignment fixes
staging: rts5208: Long lines fixes
staging: rts5208: Prefer using BIT macro
staging: rts5208: Parenthesis alignment
staging: rts5208: Unecessary parantheses
staging: rts5208: Parenthesis alignment
staging: rts5208: Unnecessary parentheses
staging: rts5208: Spacing
staging: rts5208: Comparison to NULL
staging: rts5208: rtsx.c: Unnecessary parentheses
staging: rts5208: rtsx.c: Alignment
staging: rts5208: rtsx.c: Multiple assignments
staging: rts5208: rtsx.c: Spacing
staging: rts5208: rtsx.c: CamelCase
staging: rts5208: rtsx.c: Spacing
staging: rts5208: rtsx.c: Alloc sizeof struct
staging: rts5208: ms.h: Parenthesis alignment
staging: rts5208: ms.c: Parenthesis alignment
staging: rts5208: ms.c: Long Lines
staging: rts5208: ms.c: Multiple assignments
staging: rts5208: ms.c: Braces fix
staging: rts5208: CamelCase fix
staging: rts5208: rtsx_scsi.h: Long lines
staging: rts5208: rtxs_chip.h: Long lines
staging: rts5208: rtsx_chip.c: Long lines
staging: rts5208: rtsx.c: Alignment fix
drivers/staging/rts5208/ms.c | 393 ++++++++------- drivers/staging/rts5208/ms.h | 4 +- drivers/staging/rts5208/rtsx.c | 43 +- drivers/staging/rts5208/rtsx.h | 2 +- drivers/staging/rts5208/rtsx_card.c | 94 ++-- drivers/staging/rts5208/rtsx_card.h | 16 +- drivers/staging/rts5208/rtsx_chip.c | 17 +- drivers/staging/rts5208/rtsx_chip.h | 137 ++++-- drivers/staging/rts5208/rtsx_scsi.c | 319 ++++++------ drivers/staging/rts5208/rtsx_scsi.h | 4 +- drivers/staging/rts5208/rtsx_sys.h | 4 +- drivers/staging/rts5208/rtsx_transport.h | 30 +- drivers/staging/rts5208/sd.c | 813 ++++++++++++++++--------------- drivers/staging/rts5208/sd.h | 5 +- drivers/staging/rts5208/spi.c | 144 +++--- 15 files changed, 1075 insertions(+), 950 deletions(-)
Signature · 1 line
2.1.4