diff mbox series

[3/4] staging: mt7621: Add braces around macro argument

Message ID 1531489266-20410-3-git-send-email-oliver.huebers@web.de
State New
Headers show
Series [1/4] staging: mt7621-pci: Include preferred headers | expand

Commit Message

Oliver Hübers July 13, 2018, 1:41 p.m. UTC 
Add braces around macro argument to avoid precedence problems.
This fixes the corresponding checkpatch warning.

Signed-off-by: Oliver Hübers <oliver.huebers@web.de>
Signed-off-by: Vanessa Borgmann <nessa.x@web.de>
---
 drivers/staging/mt7621-pci/pci-mt7621.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Greg KH July 16, 2018, 8:35 a.m. UTC | #1 
On Fri, Jul 13, 2018 at 03:41:05PM +0200, Oliver Hübers wrote:
> Add braces around macro argument to avoid precedence problems.

That is not what this patch does :(
diff mbox series

Patch

diff --git a/drivers/staging/mt7621-pci/pci-mt7621.c b/drivers/staging/mt7621-pci/pci-mt7621.c
index d0d3fb9..c118d78 100644
--- a/drivers/staging/mt7621-pci/pci-mt7621.c
+++ b/drivers/staging/mt7621-pci/pci-mt7621.c
@@ -132,7 +132,8 @@ 
 		else							\
 			rt_sysc_m32(val, 0, RALINK_RSTCTRL);		\
 	} while (0)
-#define DEASSERT_SYSRST_PCIE(val)	\
+
+#define DEASSERT_SYSRST_PCIE(val)					\
 	do {								\
 		if (rt_sysc_r32(SYSC_REG_CHIP_REV) == 0x00030101)	\
 			rt_sysc_m32(val, 0, RALINK_RSTCTRL);		\