diff mbox

iwlwifi: mvm: remove an extra tab

Message ID 20151209102608.GF3173@mwanda (mailing list archive)
State Accepted
Headers show

Commit Message

Dan Carpenter Dec. 9, 2015, 10:26 a.m. UTC
Smatch prints a static checker warning here:

    drivers/net/wireless/intel/iwlwifi/mvm/fw-dbg.c:386 iwl_dump_prph()
    warn: curly braces intended?

Curly braces are NOT intended, the extra tab was added by mistake in
commit 1a616dd2f171 ('iwlwifi: dump prph registers in a common place
for all transports').

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Johannes Berg Dec. 9, 2015, 10:41 a.m. UTC | #1
On Wed, 2015-12-09 at 13:26 +0300, Dan Carpenter wrote:
> Smatch prints a static checker warning here:
> 
>     drivers/net/wireless/intel/iwlwifi/mvm/fw-dbg.c:386
> iwl_dump_prph()
>     warn: curly braces intended?
> 
> Curly braces are NOT intended, the extra tab was added by mistake in
> commit 1a616dd2f171 ('iwlwifi: dump prph registers in a common place
> for all transports').
> 

Applied, thanks.

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/fw-dbg.c b/drivers/net/wireless/intel/iwlwifi/mvm/fw-dbg.c
index 9fcabc5..a6985da 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/fw-dbg.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/fw-dbg.c
@@ -383,7 +383,7 @@  static u32 iwl_dump_prph(struct iwl_trans *trans,
 			*val++ = cpu_to_le32(iwl_read_prph_no_grab(trans,
 								   reg));
 
-			*data = iwl_fw_error_next_data(*data);
+		*data = iwl_fw_error_next_data(*data);
 	}
 
 	iwl_trans_release_nic_access(trans, &flags);