On Fri Sep 27, 2024 at 11:45 PM CST, Dan Carpenter wrote:
Greg doesn't agree with checkpatch about unnecessary parentheses. If the code
looks like this:
if ((a == b) || (b == c)) {
then just ignore checkpatch. However if the code looks like:
p = &(foo);
Then change it to:
p = &foo;
This sort of change would need to be split up by driver instead of all of
driver/staging. So it would be a series of patches. You don't need to split
it up by file. Split it by driver.
Thanks, this information is important. -- Xingquan Liu
Attachment:
signature.asc
Description: PGP signature
