All of lore.kernel.org
 help / color / mirror / Atom feed
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

             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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.