All of lore.kernel.org
 help / color / mirror / Atom feed
From: sleepingzucchini@gmail.com
To: gilad@benyossef.com
Cc: gregkh@linuxfoundation.orgc, linux-crypto@vger.kernel.org,
	driverdev-devel@linuxdriverproject.org,
	Tyler Olivieri <sleepingzucchini@gmail.com>
Subject: [PATCH 0/5] staging: ccree: fix checkpatch errors
Date: Sun,  9 Jul 2017 17:10:42 -0400	[thread overview]
Message-ID: <cover.1499608047.git.sleepingzucchini@gmail.com> (raw)

From: Tyler Olivieri <sleepingzucchini@gmail.com>

This patchset fixes several checkpatch errors and warnings in /staging/ccree:

ERROR: that open brace { should be on the previous line
ERROR: open brace '{' following function declarations go on the next line
WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
ERROR: do not use assignment in if condition
ERROR: switch and case should be at the same indent
WARNING: Statements terminations use 1 semicolon

This is also a submission for the eudyptula challenge. 

Tyler Olivieri (5):
  staging: ccree: remove redudant semicolons
  staging: ccree: fix placement of curly braces
  staging: ccree: remove assignement in conditional
  staging: ccree: export symbol immediately following function
  staging: ccree: fix switch case indentation

 drivers/staging/ccree/ssi_buffer_mgr.c | 14 ++----
 drivers/staging/ccree/ssi_cipher.c     |  6 ++-
 drivers/staging/ccree/ssi_driver.c     |  5 +-
 drivers/staging/ccree/ssi_fips.c       |  2 -
 drivers/staging/ccree/ssi_fips_ll.c    | 85 +++++++++++-----------------------
 drivers/staging/ccree/ssi_hash.c       | 33 +++++++------
 drivers/staging/ccree/ssi_sysfs.c      |  3 +-
 7 files changed, 57 insertions(+), 91 deletions(-)

-- 
2.9.4

             reply	other threads:[~2017-07-09 22:15 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-09 21:10 sleepingzucchini [this message]
2017-07-09 21:10 ` [PATCH 1/5] staging: ccree: remove redudant semicolons sleepingzucchini
2017-07-09 21:10 ` [PATCH 2/5] staging: ccree: fix placement of curly braces sleepingzucchini
2017-07-09 21:10 ` [PATCH 3/5] staging: ccree: remove assignement in conditional sleepingzucchini
2017-07-09 21:10 ` [PATCH 4/5] staging: ccree: export symbol immediately following function sleepingzucchini
2017-07-09 21:10 ` [PATCH 5/5] staging: ccree: fix switch case indentation sleepingzucchini
2017-07-11 13:38 ` [PATCH 0/5] staging: ccree: fix checkpatch errors Gilad Ben-Yossef
2017-07-11 13:43   ` Gilad Ben-Yossef

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.1499608047.git.sleepingzucchini@gmail.com \
    --to=sleepingzucchini@gmail.com \
    --cc=driverdev-devel@linuxdriverproject.org \
    --cc=gilad@benyossef.com \
    --cc=gregkh@linuxfoundation.orgc \
    --cc=linux-crypto@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.