Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1514073 > unrolled thread

[PATCH 0/5] staging: wlan-ng: Fix several style issues.

Started bySergio Paracuellos <sergio.paracuellos@gmail.com>
First post2016-11-02 19:50 +0100
Last post2016-11-02 19:50 +0100
Articles 3 — 1 participant

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/5] staging: wlan-ng: Fix several style issues. Sergio Paracuellos <sergio.paracuellos@gmail.com> - 2016-11-02 19:50 +0100
    [PATCH 1/5] staging: wlan-ng: add spaces around or operator : HOSTWEP_DEFAULTKEY_MASK Sergio Paracuellos <sergio.paracuellos@gmail.com> - 2016-11-02 19:50 +0100
    [PATCH 2/5] staging: wlan-ng: remove two unnecessary blank lines Sergio Paracuellos <sergio.paracuellos@gmail.com> - 2016-11-02 19:50 +0100

#1514073 — [PATCH 0/5] staging: wlan-ng: Fix several style issues.

FromSergio Paracuellos <sergio.paracuellos@gmail.com>
Date2016-11-02 19:50 +0100
Subject[PATCH 0/5] staging: wlan-ng: Fix several style issues.
Message-ID<sz8LM-1va-19@gated-at.bofh.it>
This patch series fix some style issues in several source and header files.

Sergio Paracuellos (5):
  staging: wlan-ng: add spaces around or operator :
    HOSTWEP_DEFAULTKEY_MASK
  staging: wlan-ng: remove two unnecessary blank lines
  staging: wlan-ng: change comparison to NULL to preferred style.
  staging: wlan-ng: remove unnecessary parentheses
  staging: wlan-ng: remove unnecessary blank line

 drivers/staging/wlan-ng/p80211netdev.c | 2 --
 drivers/staging/wlan-ng/p80211netdev.h | 2 +-
 drivers/staging/wlan-ng/p80211req.c    | 7 +++----
 3 files changed, 4 insertions(+), 7 deletions(-)

-- 
1.9.1

[toc] | [next] | [standalone]


#1514074 — [PATCH 1/5] staging: wlan-ng: add spaces around or operator : HOSTWEP_DEFAULTKEY_MASK

FromSergio Paracuellos <sergio.paracuellos@gmail.com>
Date2016-11-02 19:50 +0100
Subject[PATCH 1/5] staging: wlan-ng: add spaces around or operator : HOSTWEP_DEFAULTKEY_MASK
Message-ID<sz8LM-1va-33@gated-at.bofh.it>
In reply to#1514073
Add spaces around or operator to comply with the standard 
kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 drivers/staging/wlan-ng/p80211netdev.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/wlan-ng/p80211netdev.h b/drivers/staging/wlan-ng/p80211netdev.h
index 972e076..8066ac4 100644
--- a/drivers/staging/wlan-ng/p80211netdev.h
+++ b/drivers/staging/wlan-ng/p80211netdev.h
@@ -143,7 +143,7 @@ struct p80211_frmrx {
 #define NUM_WEPKEYS 4
 #define MAX_KEYLEN 32
 
-#define HOSTWEP_DEFAULTKEY_MASK (BIT(1)|BIT(0))
+#define HOSTWEP_DEFAULTKEY_MASK (BIT(1) | BIT(0))
 #define HOSTWEP_SHAREDKEY BIT(3)
 #define HOSTWEP_DECRYPT  BIT(4)
 #define HOSTWEP_ENCRYPT  BIT(5)
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]


#1514075 — [PATCH 2/5] staging: wlan-ng: remove two unnecessary blank lines

FromSergio Paracuellos <sergio.paracuellos@gmail.com>
Date2016-11-02 19:50 +0100
Subject[PATCH 2/5] staging: wlan-ng: remove two unnecessary blank lines
Message-ID<sz8LM-1va-35@gated-at.bofh.it>
In reply to#1514073
After open brace or before close brace blank lines are not really 
necessary. Remove them.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 drivers/staging/wlan-ng/p80211netdev.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/wlan-ng/p80211netdev.c b/drivers/staging/wlan-ng/p80211netdev.c
index 10dacf8..ae6e635 100644
--- a/drivers/staging/wlan-ng/p80211netdev.c
+++ b/drivers/staging/wlan-ng/p80211netdev.c
@@ -277,7 +277,6 @@ static void p80211netdev_rx_bh(unsigned long arg)
 	/* Let's empty our our queue */
 	while ((skb = skb_dequeue(&wlandev->nsd_rxq))) {
 		if (wlandev->state == WLAN_DEVICE_OPEN) {
-
 			if (dev->type != ARPHRD_ETHER) {
 				/* RAW frame; we shouldn't convert it */
 				/* XXX Append the Prism Header here instead. */
@@ -461,7 +460,6 @@ static void p80211knetdev_set_multicast_list(struct net_device *dev)
 
 	if (wlandev->set_multicast_list)
 		wlandev->set_multicast_list(wlandev, dev);
-
 }
 
 #ifdef SIOCETHTOOL
-- 
1.9.1

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web