Thread (2 messages) 2 messages, 2 authors, 2015-10-18

Re: [PATCH] net: wireless: rt2x00: statics code style

From: Sergei Shtylyov <hidden>
Date: 2015-10-18 13:16:25
Also in: linux-wireless, lkml

Hello.

On 10/17/2015 11:25 PM, Paul McQuade wrote:
quoted hunk ↗ jump to hunk
Do not initialise statics to 0 or NULL
Also use tabs where possible

Signed-off-by: Paul McQuade <redacted>
---
  drivers/net/wireless/rt2x00/rt61pci.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/rt2x00/rt61pci.c b/drivers/net/wireless/rt2x00/rt61pci.c
index c0e730e..53bd23d 100644
--- a/drivers/net/wireless/rt2x00/rt61pci.c
+++ b/drivers/net/wireless/rt2x00/rt61pci.c
@@ -39,7 +39,7 @@
  /*
   * Allow hardware encryption to be disabled.
   */
-static bool modparam_nohwcrypt = false;
+static bool modparam_nohwcrypt;
  module_param_named(nohwcrypt, modparam_nohwcrypt, bool, S_IRUGO);
  MODULE_PARM_DESC(nohwcrypt, "Disable hardware encryption.");
@@ -1388,7 +1388,7 @@ static bool rt61pci_get_entry_state(struct queue_entry *entry)
  		rt2x00_desc_read(entry_priv->desc, 0, &word);

  		return (rt2x00_get_field32(word, TXD_W0_OWNER_NIC) ||
-		        rt2x00_get_field32(word, TXD_W0_VALID));
+		rt2x00_get_field32(word, TXD_W0_VALID));
    This line should indented more, like it was before. And the outer parens 
aren't needed.

MBR, Sergei
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help