Messages in this thread | | | Date | Mon, 2 Nov 2015 17:48:38 +0300 | From | Dan Carpenter <> | Subject | Re: [lustre-devel] [PATCH 1/3] staging: lustre: checkpatch cleanups for nidstring.c |
| |
Yeah. That is often the fastest way to fix all the checkpatch warnings.
Checkpatch warnings are pretty mechanical. Just send like 100 patches at a time until everything is fixed. Don't overthink. Say your patch breaks the alignment then you have to fix that, but otherwise only fix one thing at a time. Sometimes people will ask you to fix something else on the same line, but just say "I didn't introduce that, but yes I am planning to fix that in a later patchset since I am following the one thing per patch rule."
Don't feel shame about sending many small patches. We pretty much merge everything.
regards, dan carpenter
| |