Re: [PATCH] drivers/input: Macros with complex values should be enclosed in parentheses

2016-04-26 Thread Dmitry Torokhov
On Tue, Apr 26, 2016 at 10:52:50AM +0800, Rui Teng wrote: > The bitwise shift operator has lower priority than plus operator. So the > values on macros > should be enclosed in parentheses. > > For example, "(1 << 4 + 1)" means "(1 << (4 + 1))", but it is not expected by > the macros. > > And al

[PATCH] drivers/input: Macros with complex values should be enclosed in parentheses

2016-04-25 Thread Rui Teng
The bitwise shift operator has lower priority than plus operator. So the values on macros should be enclosed in parentheses. For example, "(1 << 4 + 1)" means "(1 << (4 + 1))", but it is not expected by the macros. And also fix other two coding style problems reported by scripts/checkpatch.pl.