linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Net: wireless: ath: fix macros coding style issue in hw.c
       [not found] <ath/hw.c>
@ 2010-03-29 20:34 ` Luis de Bethencourt
  2010-03-29 22:24   ` Luis R. Rodriguez
  0 siblings, 1 reply; 2+ messages in thread
From: Luis de Bethencourt @ 2010-03-29 20:34 UTC (permalink / raw)
  To: linville, lrodriguez
  Cc: linux-wireless, netdev, linux-kernel, Luis de Bethencourt,
	Luis de Bethencourt

From: Luis de Bethencourt <luisbg@openshine.com>

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 <luisbg@ubuntu.com>
---
 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


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

* Re: [PATCH] Net: wireless: ath: fix macros coding style issue in hw.c
  2010-03-29 20:34 ` [PATCH] Net: wireless: ath: fix macros coding style issue in hw.c Luis de Bethencourt
@ 2010-03-29 22:24   ` Luis R. Rodriguez
  0 siblings, 0 replies; 2+ messages in thread
From: Luis R. Rodriguez @ 2010-03-29 22:24 UTC (permalink / raw)
  To: Luis de Bethencourt
  Cc: linville, Luis Rodriguez, linux-wireless, netdev, linux-kernel,
	Luis de Bethencourt

On Mon, Mar 29, 2010 at 01:34:39PM -0700, Luis de Bethencourt wrote:
> From: Luis de Bethencourt <luisbg@openshine.com>
> 
> 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 <luisbg@ubuntu.com>
 
Acked-by: Luis R. Rodriguez <lrodriguez@atheros.com>

  Luis

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

end of thread, other threads:[~2010-03-29 22:25 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <ath/hw.c>
2010-03-29 20:34 ` [PATCH] Net: wireless: ath: fix macros coding style issue in hw.c Luis de Bethencourt
2010-03-29 22:24   ` Luis R. Rodriguez

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).