2010-03-29 20:34:46

by Luis de Bethencourt

Subject: [PATCH] Net: wireless: ath: fix macros coding style issue in hw.c

From: Luis de Bethencourt <[email protected]>

This is a patch to the ath/hw.c file that fixes up a warning about
macros found by the checkpatch.pl tool, that said that complex values
should be enclosed in parenthesis.

Signed-off-by: Luis de Bethencourt <[email protected]>
---
drivers/net/wireless/ath/hw.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/hw.c b/drivers/net/wireless/ath/hw.c
index ecc9eb0..a8f81ea 100644
--- a/drivers/net/wireless/ath/hw.c
+++ b/drivers/net/wireless/ath/hw.c
@@ -19,8 +19,8 @@
#include "ath.h"
#include "reg.h"

-#define REG_READ common->ops->read
-#define REG_WRITE common->ops->write
+#define REG_READ (common->ops->read)
+#define REG_WRITE (common->ops->write)

/**
* ath_hw_set_bssid_mask - filter out bssids we listen
--
1.6.3.3

2010-03-29 22:25:01

by Luis R. Rodriguez

Subject: Re: [PATCH] Net: wireless: ath: fix macros coding style issue in hw.c

On Mon, Mar 29, 2010 at 01:34:39PM -0700, Luis de Bethencourt wrote:
> From: Luis de Bethencourt <[email protected]>
>
> This is a patch to the ath/hw.c file that fixes up a warning about
> macros found by the checkpatch.pl tool, that said that complex values
> should be enclosed in parenthesis.
>
> Signed-off-by: Luis de Bethencourt <[email protected]>

Acked-by: Luis R. Rodriguez <[email protected]>

Luis