On Tue, 30 Jan 2024 at 15:11, Alex Bennée <alex.ben...@linaro.org> wrote: > I suspect the problem is that checkpatch.pl is an import from the Linux > source tree which has since had syncs with its upstream as well as a > slew of QEMU specific patches. If we don't care about tracking upstream > anymore we could bite the bullet and fix indentation going forward.
I had a look at trying to update to be based on a newer version of the Linux checkpatch some years ago, but it turned out to not be a trivial thing to do, and we've only diverged further since then. (Which is unfortunate, because I imagine there have been bugfixes to the perl script's handling of parsing fragments of C code which we'd like.) -- PMM