Skip to content

staging: comedi: Fix checkpatch warning

Anson Jacob lkml 3 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest8 patches 0 repliesNew
  2. v27 patches 3 repliesSuperseded
  3. v19 patches 0 repliesSuperseded

Changes in v3 from the cover letter

Fix trailing space
Fix checkpatch.pl warning for blank line after enum declaration Move macro definition to the top of the file
Align macro definitions

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8staging: comedi: cb_pcidas64.c: Fix checkpatch warningdrivers/staging/comedi/drivers/cb_pcidas64.c+77 -710 replies
2/8staging: comedi: cb_pcidas64: Fix commenting styledrivers/staging/comedi/drivers/cb_pcidas64.c+214 -2170 replies
3/8staging: comedi: jr3_pci.h: Fix checkpatch warningdrivers/staging/comedi/drivers/jr3_pci.h+172 -1180 replies
4/8staging: comedi: ni_atmio.c: Fix checkpatch warningdrivers/staging/comedi/drivers/ni_atmio.c+4 -70 replies
5/8staging: comedi: s626.h: Fix checkpatch warningdrivers/staging/comedi/drivers/s626.h+2 -20 replies
6/8staging: comedi: jr3_pci.c: Fix checkpatch warningdrivers/staging/comedi/drivers/jr3_pci.c+7 -60 replies
7/8staging: comedi: ni_at_a2150: Fix checkpatch warningdrivers/staging/comedi/drivers/ni_at_a2150.c+89 -810 replies
8/8staging: comedi: cb_pcidas64: Fix checkpath warningdrivers/staging/comedi/drivers/cb_pcidas64.c+2 -20 replies

Reviews trailers per patch

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

Read the v3 thread

Cover letter [PATCH v3 0/8] staging: comedi: Fix checkpatch warning

Fix checkpath.pl warning:
line over 80 characters
Block comments use * on subsequent lines
Block comments use a trailing */ on a separate line Statements should start on a tabstop
Comparisons should place the constant on the right side of the test

Changes in V2:
Remove extra space after /*
Align single line comment
Block comments starts with /* followed by comments on the next line

Changes in V3:
Fix trailing space
Fix checkpatch.pl warning for blank line after enum declaration Move macro definition to the top of the file
Align macro definitions

Anson Jacob (8):
staging: comedi: cb_pcidas64.c: Fix checkpatch warning
staging: comedi: cb_pcidas64: Fix commenting style
staging: comedi: jr3_pci.h: Fix checkpatch warning
staging: comedi: ni_atmio.c: Fix checkpatch warning
staging: comedi: s626.h: Fix checkpatch warning
staging: comedi: jr3_pci.c: Fix checkpatch warning
staging: comedi: ni_at_a2150: Fix checkpatch warning
staging: comedi: cb_pcidas64: Fix checkpath warning

 drivers/staging/comedi/drivers/cb_pcidas64.c | 573 ++++++++++++++-------------
 drivers/staging/comedi/drivers/jr3_pci.c     |  13 +-
 drivers/staging/comedi/drivers/jr3_pci.h     | 290 ++++++++------
 drivers/staging/comedi/drivers/ni_at_a2150.c | 170 ++++----
 drivers/staging/comedi/drivers/ni_atmio.c    |  11 +-
 drivers/staging/comedi/drivers/s626.h        |   4 +-
 6 files changed, 562 insertions(+), 499 deletions(-)
Signature · 1 line
2.7.4