From: Wayne Porter <wporter82@gmail.com>
To: gregkh@linuxfoundation.org
Cc: Wayne Porter <wporter82@gmail.com>, linux-kernel@vger.kernel.org
Subject: [PATCH 00/45] staging: rts5208: Checkpatch.pl cleanup
Date: Tue, 11 Oct 2016 21:56:16 +0000 [thread overview]
Message-ID: <cover.1476222674.git.wporter82@gmail.com> (raw)
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(-)
--
2.1.4
next reply other threads:[~2016-10-11 22:14 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-11 21:56 Wayne Porter [this message]
2016-10-11 21:56 ` [PATCH 01/45] staging: rts5208: spi.c: Remove unnecessary parenthesis Wayne Porter
2016-10-11 21:56 ` [PATCH 02/45] staging: rts5208: spi.c: Alignment fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 03/45] staging: rts5208: sd.h: " Wayne Porter
2016-10-11 21:56 ` [PATCH 04/45] staging: rts5208: sd.c: Remove unnecessary parentheses Wayne Porter
2016-10-11 21:56 ` [PATCH 05/45] staging: rts5208: sd.c: Alignment fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 06/45] staging: rts5208: sd.c: Long line fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 07/45] staging: rts5208: sd.c: CamelCase fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 08/45] staging: rts5208: sd.c: Fix logical continuations Wayne Porter
2016-10-11 21:56 ` [PATCH 09/45] staging: rts5208: sd.c: Spacing cleanup Wayne Porter
2016-10-11 21:56 ` [PATCH 10/45] staging: rts5208: Alignment fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 11/45] staging: rts5208: Unnecessary parentheses cleanup Wayne Porter
2016-10-11 21:56 ` [PATCH 12/45] staging: rts5208: Alignment fix Wayne Porter
2016-10-11 21:56 ` [PATCH 13/45] staging: rts5208: Fix typo in function name Wayne Porter
2016-10-11 21:56 ` [PATCH 14/45] staging: rts5208: Alignment fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 15/45] staging: rts5208: Remove parentheses Wayne Porter
2016-10-11 21:56 ` [PATCH 16/45] staging: rts5208: CamelCase fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 17/45] staging: rts5208: Remove multiple assignment Wayne Porter
2016-10-11 21:56 ` [PATCH 18/45] staging: rts5208: Add braces to if() Wayne Porter
2016-10-11 21:56 ` [PATCH 19/45] staging: rts5208: Add space around operator Wayne Porter
2016-10-11 21:56 ` [PATCH 20/45] staging: rts5208: Alignment fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 21/45] staging: rts5208: Long lines fixes Wayne Porter
2016-10-11 21:56 ` [PATCH 22/45] staging: rts5208: Prefer using BIT macro Wayne Porter
2016-10-11 21:56 ` [PATCH 23/45] staging: rts5208: Parenthesis alignment Wayne Porter
2016-10-11 21:56 ` [PATCH 24/45] staging: rts5208: Unecessary parantheses Wayne Porter
2016-10-11 21:56 ` [PATCH 25/45] staging: rts5208: Parenthesis alignment Wayne Porter
2016-10-11 21:56 ` [PATCH 26/45] staging: rts5208: Unnecessary parentheses Wayne Porter
2016-10-11 21:56 ` [PATCH 27/45] staging: rts5208: Spacing Wayne Porter
2016-10-11 21:56 ` [PATCH 28/45] staging: rts5208: Comparison to NULL Wayne Porter
2016-10-11 21:56 ` [PATCH 29/45] staging: rts5208: rtsx.c: Unnecessary parentheses Wayne Porter
2016-10-11 21:56 ` [PATCH 30/45] staging: rts5208: rtsx.c: Alignment Wayne Porter
2016-10-11 21:56 ` [PATCH 31/45] staging: rts5208: rtsx.c: Multiple assignments Wayne Porter
2016-10-11 21:56 ` [PATCH 32/45] staging: rts5208: rtsx.c: Spacing Wayne Porter
2016-10-11 21:56 ` [PATCH 33/45] staging: rts5208: rtsx.c: CamelCase Wayne Porter
2016-10-11 21:56 ` [PATCH 34/45] staging: rts5208: rtsx.c: Spacing Wayne Porter
2016-10-11 21:56 ` [PATCH 35/45] staging: rts5208: rtsx.c: Alloc sizeof struct Wayne Porter
2016-10-11 21:56 ` [PATCH 36/45] staging: rts5208: ms.h: Parenthesis alignment Wayne Porter
2016-10-11 21:56 ` [PATCH 37/45] staging: rts5208: ms.c: " Wayne Porter
2016-10-11 21:56 ` [PATCH 38/45] staging: rts5208: ms.c: Long Lines Wayne Porter
2016-10-11 21:56 ` [PATCH 39/45] staging: rts5208: ms.c: Multiple assignments Wayne Porter
2016-10-11 21:56 ` [PATCH 40/45] staging: rts5208: ms.c: Braces fix Wayne Porter
2016-10-11 21:56 ` [PATCH 41/45] staging: rts5208: CamelCase fix Wayne Porter
2016-10-11 21:56 ` [PATCH 42/45] staging: rts5208: rtsx_scsi.h: Long lines Wayne Porter
2016-10-11 21:56 ` [PATCH 43/45] staging: rts5208: rtxs_chip.h: " Wayne Porter
2016-10-11 21:57 ` [PATCH 44/45] staging: rts5208: rtsx_chip.c: " Wayne Porter
2016-10-11 21:57 ` [PATCH 45/45] staging: rts5208: rtsx.c: Alignment fix Wayne Porter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1476222674.git.wporter82@gmail.com \
--to=wporter82@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome