On Sat, Mar 11, 2023 at 06:43:18AM -0800, Sumitra Sharma wrote: > Hi Dan, > > Your suggestion for correcting the indentation to > "[tab][tab][space][space][space][space](i ==." conflicts with the > statement "Outside of comments, documentation and except in Kconfig, > spaces are never used for indentation" written in > https://elixir.bootlin.com/linux/latest/source/Documentation/process/coding-style.rst Huh... That documentation is very wrong. Maybe you are not the only person who has read that and been confused. Not only do we use spaces, but checkpatch will complain if you don't use spaces to align function parameters. Aligning conditions is not universal but it's generally prefered. There isn't a checkpatch warning for misaligned conditions, but I think that's because it's much trickier to parse conditions correctly. Aligning conditions is much more subtle. What this means is that maybe you should consult with your mentor and try to fix the documentation. Maybe say something to the effect of "You can use spaces for micro alignments to function parameters and conditions in an if statement". > > However, If you still recommend to correct the indentation in the manner > "[tab][tab][space][space][space][space](i ==." Should I create a > patch for the same? I'm not going to tell you what to do with your life. :P But if you send that patch then we will merge it. regards, dan carpenter