@marcan @ljs You call the requirements arbitrary or silly, I'll give you my perspective:
> "put your declarations in reverse Christmas tree order"
I'm not a fan of the reverse xmas tree, but such local coding style preferences are there for consistency for those who have to read the code base, lots of code is read en bloc and pattern recognition becomes a great help to skim a lot of code; the drawback is that it mandates new code in that specific format, ideally documented somewhere, but practical advice is to mimic the style in the same file however unusual it may feel.
> "always do minimal fixes, ...
As linux cares about backports to old stable kernels a minimal fix has a much higher chance to get applied without merge conflicts and there's a scripting machinery around that to minimize manual fixups be people only when necessary, fixing more bugs in one patch should be avoided unless it would be really awkward so it's a gray area but in general one fix per patch should be the default, what makes sense or not is subjective and opnions differ, in the end it's the maintainer who has to deal with the fallouts of that.
> "do not reformat code to keep alignment after changes, ...
Similar argument as above, for easier backports, but it depends on the change. Patches just fixing coding style should be avoided as it clutters git history and fixing coding style issues in the modified lines should be ok but not fixing surrounding code just because it's in the same file.
What you complain about is nothing new and I've had this discussions with several people over the years. Unfortunatelly documenting the practices also does not work, nobody reads that so it always starts with the first patch and customized 1:1 response.
Frequent contributors should know how to do it (but often they don't) and I personally cut a lot of slack in one-time contributions and fix a lot of things myself when committing patches.