linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis
@ 2017-01-12  3:38 Scott Matheina
  2017-01-12  3:38 ` [PATCH 2/7] staging:wilc1000:host_interface.c Added braces {} on else statemement Scott Matheina
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Greg Kroah-Hartman, Sergio Paracuellos, Claudiu Beznea,
	Johannes Berg, Juliana Rodrigues, Gavin O'Leary,
	sayli karnik, Bogicevic Sasa, devel

Fixed Alignment should match open parenthesis checkpatch CHECK.

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wlan-ng/cfg80211.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.c
index aa0e5a3..11870cb 100644
--- a/drivers/staging/wlan-ng/cfg80211.c
+++ b/drivers/staging/wlan-ng/cfg80211.c
@@ -483,8 +483,8 @@ static int prism2_connect(struct wiphy *wiphy, struct net_device *dev,
 		msg_join.authtype.data = P80211ENUM_authalg_sharedkey;
 	else
 		netdev_warn(dev,
-			"Unhandled authorisation type for connect (%d)\n",
-			sme->auth_type);
+			    "Unhandled authorisation type for connect (%d)\n",
+			    sme->auth_type);
 
 	/* Set the encryption - we only support wep */
 	if (is_wep) {
@@ -667,7 +667,7 @@ void prism2_disconnected(struct wlandevice *wlandev)
 void prism2_roamed(struct wlandevice *wlandev)
 {
 	cfg80211_roamed(wlandev->netdev, NULL, wlandev->bssid,
-		NULL, 0, NULL, 0, GFP_KERNEL);
+			NULL, 0, NULL, 0, GFP_KERNEL);
 }
 
 /* Structures for declaring wiphy interface */
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/7] staging:wilc1000:host_interface.c Added braces {} on else statemement
  2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
@ 2017-01-12  3:38 ` Scott Matheina
  2017-01-12  3:38 ` [PATCH 3/7] staging:wilc1000:linux_wlan.c Align code to match open parenthesis Scott Matheina
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Aditya Shankar, Ganesh Krishna, Greg Kroah-Hartman,
	linux-wireless, devel

Fixes checkpatch warning: braces {} should be used on all arms of
this statement

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wilc1000/host_interface.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c
index b00ea75..c307cce 100644
--- a/drivers/staging/wilc1000/host_interface.c
+++ b/drivers/staging/wilc1000/host_interface.c
@@ -3998,8 +3998,9 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
 				pNewJoinBssParam->rsn_found = true;
 				index += pu8IEs[index + 1] + 2;
 				continue;
-			} else
+			} else {
 				index += pu8IEs[index + 1] + 2;
+			}
 		}
 	}
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/7] staging:wilc1000:linux_wlan.c Align code to match open parenthesis
  2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
  2017-01-12  3:38 ` [PATCH 2/7] staging:wilc1000:host_interface.c Added braces {} on else statemement Scott Matheina
@ 2017-01-12  3:38 ` Scott Matheina
  2017-01-12  3:38 ` [PATCH 4/7] staging:wilc1000:wilc_debugfs.c Aligns " Scott Matheina
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Aditya Shankar, Ganesh Krishna, Greg Kroah-Hartman,
	linux-wireless, devel

Fixes checkpatch CHECK: Alignment should match open parenthesis

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wilc1000/linux_wlan.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c
index 185fcd1..9ab4393 100644
--- a/drivers/staging/wilc1000/linux_wlan.c
+++ b/drivers/staging/wilc1000/linux_wlan.c
@@ -364,7 +364,7 @@ static int linux_wlan_start_firmware(struct net_device *dev)
 		return ret;
 
 	if (!wait_for_completion_timeout(&wilc->sync_event,
-					msecs_to_jiffies(5000)))
+					 msecs_to_jiffies(5000)))
 		return -ETIME;
 
 	return 0;
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 4/7] staging:wilc1000:wilc_debugfs.c Aligns code to match open parenthesis
  2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
  2017-01-12  3:38 ` [PATCH 2/7] staging:wilc1000:host_interface.c Added braces {} on else statemement Scott Matheina
  2017-01-12  3:38 ` [PATCH 3/7] staging:wilc1000:linux_wlan.c Align code to match open parenthesis Scott Matheina
@ 2017-01-12  3:38 ` Scott Matheina
  2017-01-12  3:38 ` [PATCH 5/7] staging:wilc1000:wilc_debugfs.c Removes multiple blank lines Scott Matheina
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Aditya Shankar, Ganesh Krishna, Greg Kroah-Hartman,
	linux-wireless, devel

Fixes checkpatch CHECK: Alignment should match open parenthesis

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wilc1000/wilc_debugfs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/wilc1000/wilc_debugfs.c b/drivers/staging/wilc1000/wilc_debugfs.c
index 07260c4..52aac75 100644
--- a/drivers/staging/wilc1000/wilc_debugfs.c
+++ b/drivers/staging/wilc1000/wilc_debugfs.c
@@ -52,7 +52,7 @@ static ssize_t wilc_debug_level_read(struct file *file, char __user *userbuf, si
 }
 
 static ssize_t wilc_debug_level_write(struct file *filp, const char __user *buf,
-					size_t count, loff_t *ppos)
+				      size_t count, loff_t *ppos)
 {
 	int flag = 0;
 	int ret;
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 5/7] staging:wilc1000:wilc_debugfs.c Removes multiple blank lines
  2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
                   ` (2 preceding siblings ...)
  2017-01-12  3:38 ` [PATCH 4/7] staging:wilc1000:wilc_debugfs.c Aligns " Scott Matheina
@ 2017-01-12  3:38 ` Scott Matheina
  2017-01-12  3:38 ` [PATCH 6/7] staging:wilc1000:wilc_sdio.c Aligns code match open parenthesis Scott Matheina
  2017-01-12  3:38 ` [PATCH 7/7] staging:wilc1000:wilc_sdio.c Deleted un-needed blank lines Scott Matheina
  5 siblings, 0 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Aditya Shankar, Ganesh Krishna, Greg Kroah-Hartman,
	linux-wireless, devel

Fixes checkpatch CHECK: Please don't use multiple blank lines

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wilc1000/wilc_debugfs.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/wilc1000/wilc_debugfs.c b/drivers/staging/wilc1000/wilc_debugfs.c
index 52aac75..7d32de9 100644
--- a/drivers/staging/wilc1000/wilc_debugfs.c
+++ b/drivers/staging/wilc1000/wilc_debugfs.c
@@ -17,7 +17,6 @@
 
 #include "wilc_wlan_if.h"
 
-
 static struct dentry *wilc_dir;
 
 /*
@@ -36,7 +35,6 @@ EXPORT_SYMBOL_GPL(WILC_DEBUG_LEVEL);
  * --------------------------------------------------------------------------------
  */
 
-
 static ssize_t wilc_debug_level_read(struct file *file, char __user *userbuf, size_t count, loff_t *ppos)
 {
 	char buf[128];
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 6/7] staging:wilc1000:wilc_sdio.c Aligns code match open parenthesis
  2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
                   ` (3 preceding siblings ...)
  2017-01-12  3:38 ` [PATCH 5/7] staging:wilc1000:wilc_debugfs.c Removes multiple blank lines Scott Matheina
@ 2017-01-12  3:38 ` Scott Matheina
  2017-01-12  3:38 ` [PATCH 7/7] staging:wilc1000:wilc_sdio.c Deleted un-needed blank lines Scott Matheina
  5 siblings, 0 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Aditya Shankar, Ganesh Krishna, Greg Kroah-Hartman,
	linux-wireless, devel

Fixes checkpatch CHECK: Alignment should match open parenthesis

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wilc1000/wilc_sdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000/wilc_sdio.c
index 3ad7cec..e48cebe 100644
--- a/drivers/staging/wilc1000/wilc_sdio.c
+++ b/drivers/staging/wilc1000/wilc_sdio.c
@@ -127,7 +127,7 @@ static int linux_sdio_probe(struct sdio_func *func,
 
 	dev_dbg(&func->dev, "Initializing netdev\n");
 	ret = wilc_netdev_init(&wilc, &func->dev, HIF_SDIO, gpio,
-			     &wilc_hif_sdio);
+			       &wilc_hif_sdio);
 	if (ret) {
 		dev_err(&func->dev, "Couldn't initialize netdev\n");
 		return ret;
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 7/7] staging:wilc1000:wilc_sdio.c Deleted un-needed blank lines
  2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
                   ` (4 preceding siblings ...)
  2017-01-12  3:38 ` [PATCH 6/7] staging:wilc1000:wilc_sdio.c Aligns code match open parenthesis Scott Matheina
@ 2017-01-12  3:38 ` Scott Matheina
  5 siblings, 0 replies; 7+ messages in thread
From: Scott Matheina @ 2017-01-12  3:38 UTC (permalink / raw)
  To: linux-kernel
  Cc: Aditya Shankar, Ganesh Krishna, Greg Kroah-Hartman,
	linux-wireless, devel

Fixes checkpatch CHECK:
Blank lines aren't necessary before a close brace '}'
Please don't use multiple blank lines

Signed-off-by: Scott Matheina <scott@matheina.com>
---
 drivers/staging/wilc1000/wilc_sdio.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000/wilc_sdio.c
index e48cebe..cd6b8ba 100644
--- a/drivers/staging/wilc1000/wilc_sdio.c
+++ b/drivers/staging/wilc1000/wilc_sdio.c
@@ -81,7 +81,6 @@ static int wilc_sdio_cmd52(struct wilc *wilc, struct sdio_cmd52 *cmd)
 	return ret;
 }
 
-
 static int wilc_sdio_cmd53(struct wilc *wilc, struct sdio_cmd53 *cmd)
 {
 	struct sdio_func *func = container_of(wilc->dev, struct sdio_func, dev);
@@ -915,7 +914,6 @@ static int sdio_clear_int_ext(struct wilc *wilc, u32 val)
 					__LINE__);
 				goto _fail_;
 			}
-
 		}
 	} else {
 		if (g_sdio.irq_gpio) {
@@ -945,7 +943,6 @@ static int sdio_clear_int_ext(struct wilc *wilc, u32 val)
 								__LINE__);
 							goto _fail_;
 						}
-
 					}
 					if (!ret)
 						break;
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-01-12  3:40 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-12  3:38 [PATCH 1/7] staging:wlan-ng:cfg80211.c Aligned code with open parenthesis Scott Matheina
2017-01-12  3:38 ` [PATCH 2/7] staging:wilc1000:host_interface.c Added braces {} on else statemement Scott Matheina
2017-01-12  3:38 ` [PATCH 3/7] staging:wilc1000:linux_wlan.c Align code to match open parenthesis Scott Matheina
2017-01-12  3:38 ` [PATCH 4/7] staging:wilc1000:wilc_debugfs.c Aligns " Scott Matheina
2017-01-12  3:38 ` [PATCH 5/7] staging:wilc1000:wilc_debugfs.c Removes multiple blank lines Scott Matheina
2017-01-12  3:38 ` [PATCH 6/7] staging:wilc1000:wilc_sdio.c Aligns code match open parenthesis Scott Matheina
2017-01-12  3:38 ` [PATCH 7/7] staging:wilc1000:wilc_sdio.c Deleted un-needed blank lines Scott Matheina

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).