Messages in this thread | | | From | Lukas Bulwahn <> | Date | Thu, 18 Mar 2021 18:03:11 +0100 | Subject | Re: Checkpatch still worries about include/asm/ ? |
| |
On Thu, Mar 18, 2021 at 5:47 PM Dwaipayan Ray <dwaipayanray1@gmail.com> wrote: > > Hello, > include/asm/ got removed a long time back (probably v1.1.45). > Checkpatch still worries about that: > > if ($realfile =~ m@^include/asm/@) { > ERROR("MODIFIED_INCLUDE_ASM", > "do not modify files in include/asm, change architecture specific > files in include/asm-<architecture>\n" . "$here$rawline\n"); > } > > Can we remove this check? >
Wow, v1.1.45... that must have been taking quite some digging in the ancient dirt to find that out.
I agree, let us remove that rule.
Thanks for the documentation effort on checkpatch.pl and cleaning up obsolete rules while documenting.
Lukas
| |