Re: [PATCH V6 2/2] checkpatch: check format of Vec<String> in modules

From: Miguel Ojeda
Date: Wed Mar 05 2025 - 06:05:14 EST


On Wed, Mar 5, 2025 at 8:53 AM Andreas Hindborg <a.hindborg@xxxxxxxxxx> wrote:
>
> Right, it needs a bit more tool support than running checkpatch.pl
> needs. Perhaps we could move it from checkpatch.pl to the rustfmt make
> target?

That could perhaps be an option for this case, though not sure if it
applies to all cases, i.e. `checkpatch.pl` also checks things that
only make sense to check in a patch and also things that are not
related to formatting.

Perhaps we want an entirely separate thing in `tools/` eventually, or
even out of the kernel tree, so that it can be easily run as a bot
etc. like in the past.

In any case, landing checks here is fine (as long as Joe et al.
agree), they can be moved or removed later if needed.

Cheers,
Miguel