On Thu, Jun 22, 2017 at 12:10:04PM +0530, Jaya Durga wrote: > From: Jaya Durga <jayad@xxxxxxx> > > Fix checkpatch.pl warning of the form "CHECK" Macro argument 'x' > may be better as '(x)' to avoid precedence issues. > > Signed-off-by: Jaya Durga <jayad@xxxxxxx> > --- > drivers/staging/rtl8712/osdep_intf.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/rtl8712/osdep_intf.h b/drivers/staging/rtl8712/osdep_intf.h > index 5d37e1f..1985423 100644 > --- a/drivers/staging/rtl8712/osdep_intf.h > +++ b/drivers/staging/rtl8712/osdep_intf.h > @@ -29,7 +29,7 @@ > #include "osdep_service.h" > #include "drv_types.h" > > -#define RND4(x) (((x >> 2) + (((x & 3) == 0) ? 0 : 1)) << 2) > +#define RND4(x) ((((x) >> 2) + ((((x) & 3) == 0) ? 0 : 1)) << 2) Why not move this to use the in-kernel macros instead? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
