From: Guillaume Clement <gclement@baobob.org>
To: Forest Bond <forest@alittletooquiet.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
Guillaume Clement <gclement@baobob.org>
Subject: [PATCH 08/14] staging: vt6655: break single line if statements
Date: Fri, 25 Jul 2014 01:06:22 +0200 [thread overview]
Message-ID: <1406243188-13413-9-git-send-email-gclement@baobob.org> (raw)
In-Reply-To: <1406243188-13413-1-git-send-email-gclement@baobob.org>
This fixes the "trailing statements should be on next line" checkpatch
warning.
Signed-off-by: Guillaume Clement <gclement@baobob.org>
---
drivers/staging/vt6655/card.c | 6 ++++--
drivers/staging/vt6655/device_main.c | 6 ++++--
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/vt6655/card.c b/drivers/staging/vt6655/card.c
index 58d707f..4ae8d93 100644
--- a/drivers/staging/vt6655/card.c
+++ b/drivers/staging/vt6655/card.c
@@ -934,8 +934,10 @@ bool CARDbRadioPowerOn(void *pDeviceHandler)
pr_debug("chester power on\n");
if (pDevice->bRadioControlOff == true) {
- if (pDevice->bHWRadioOff == true) pr_debug("chester bHWRadioOff\n");
- if (pDevice->bRadioControlOff == true) pr_debug("chester bRadioControlOff\n");
+ if (pDevice->bHWRadioOff == true)
+ pr_debug("chester bHWRadioOff\n");
+ if (pDevice->bRadioControlOff == true)
+ pr_debug("chester bRadioControlOff\n");
return false; }
if (pDevice->bRadioOff == false) {
diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
index c3cae50..7bd728c 100644
--- a/drivers/staging/vt6655/device_main.c
+++ b/drivers/staging/vt6655/device_main.c
@@ -388,7 +388,8 @@ device_set_options(PSDevice pDevice) {
pDevice->b11hEnable = (pDevice->sOpts.flags & DEVICE_FLAGS_80211h_MODE) ? 1 : 0;
pDevice->bDiversityRegCtlON = (pDevice->sOpts.flags & DEVICE_FLAGS_DiversityANT) ? 1 : 0;
pDevice->uConnectionRate = pDevice->sOpts.data_rate;
- if (pDevice->uConnectionRate < RATE_AUTO) pDevice->bFixRate = true;
+ if (pDevice->uConnectionRate < RATE_AUTO)
+ pDevice->bFixRate = true;
pDevice->byBBType = pDevice->sOpts.bbp_type;
pDevice->byPacketType = pDevice->byBBType;
@@ -2889,7 +2890,8 @@ static int device_ioctl(struct net_device *dev, struct ifreq *rq, int cmd)
char abyKey[WLAN_WEP232_KEYLEN];
rc = iwctl_giwencode(dev, NULL, &(wrq->u.encoding), abyKey);
- if (rc != 0) break;
+ if (rc != 0)
+ break;
if (wrq->u.encoding.pointer) {
if (copy_to_user(wrq->u.encoding.pointer,
abyKey,
--
1.8.5.5
next prev parent reply other threads:[~2014-07-24 23:09 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-24 23:06 [PATCH 00/14] staging: vt6655: Sparse and checkpatch fixes Guillaume Clement
2014-07-24 23:06 ` [PATCH 01/14] staging: vt6655: remove useless return statements Guillaume Clement
2014-07-24 23:06 ` [PATCH 02/14] staging: vt6655: Add missing blank lines after declarations Guillaume Clement
2014-07-24 23:06 ` [PATCH 03/14] staging: vt6655: Remove spaces before quoted newlines Guillaume Clement
2014-07-24 23:06 ` [PATCH 04/14] staging: vt6655: fix function braces not on the proper line Guillaume Clement
2014-07-24 23:06 ` [PATCH 05/14] staging: vt6655: fix braces at newline for structs Guillaume Clement
2014-07-24 23:06 ` [PATCH 06/14] staging: vt6655: fix static position in inline function Guillaume Clement
2014-07-24 23:06 ` [PATCH 07/14] staging: vt6655: Use pr_* functions instead of printk Guillaume Clement
2014-07-24 23:06 ` Guillaume Clement [this message]
2014-07-24 23:06 ` [PATCH 09/14] staging: vt6655: Remove unreachable break statements Guillaume Clement
2014-07-24 23:06 ` [PATCH 10/14] staging: vt6655: add missing whitespace Guillaume Clement
2014-07-24 23:06 ` [PATCH 11/14] staging: vt6655: remove braces for single statements if Guillaume Clement
2014-07-24 23:06 ` [PATCH 12/14] staging: vt6655: fix braces at newline in if statements Guillaume Clement
2014-07-24 23:06 ` [PATCH 13/14] staging: vt6655: remove unused macro Guillaume Clement
2014-07-24 23:06 ` [PATCH 14/14] staging: vt6655: Remove NULL pointer sparse warning Guillaume Clement
2014-07-25 11:50 ` [PATCH 00/14] staging: vt6655: Sparse and checkpatch fixes Dan Carpenter
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=1406243188-13413-9-git-send-email-gclement@baobob.org \
--to=gclement@baobob.org \
--cc=devel@driverdev.osuosl.org \
--cc=forest@alittletooquiet.net \
--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
all inboxes | Powered by JetHome®