Add braces around macro argument to avoid precedence problems. This fixes the corresponding checkpatch warning. Signed-off-by: Oliver Hübers <oliver.huebers@xxxxxx> Signed-off-by: Vanessa Borgmann <nessa.x@xxxxxx> --- drivers/staging/mt7621-pci/pci-mt7621.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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); \ -- 2.7.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel