[v5] checkpatch: userspace improvements

[PATCH v5 0/5] checkpatch: userspace improvements

Posted by Petr Vorel 3 weeks, 4 days ago

Changes v4->v5 (all by sashiko):
* Explicitly match the directory separator in new entries for
  is_userspace()
* Allow --no-userspace to override setup in the config (using undef
  instead of 0, separate checks in is_userspace())
* 2 new commits:
  - checkpatch: Factor out the check for UAPI files
  - checkpatch: Skip kernel specific checks for userspace

Link to v4:
https://lore.kernel.org/lkml/20260810164552.1049483-1-pvorel@suse.cz/

Petr Vorel (5):
  checkpatch: Add more userspace directories to is_userspace()
  checkpatch: Ignore <inttypes.h> format macros for userspace tools
  checkpatch: Add --userspace to force userspace rules
  checkpatch: Factor out the check for UAPI files
  checkpatch: Skip kernel specific checks for userspace

 scripts/checkpatch.pl | 34 ++++++++++++++++++++++++----------
 1 file changed, 24 insertions(+), 10 deletions(-)

-- 
2.55.0

Re: [PATCH v5 0/5] checkpatch: userspace improvements

Posted by Andrew Morton 3 weeks, 3 days ago

On Tue,  1 Sep 2026 12:56:37 +0200 Petr Vorel <pvorel@suse.cz> wrote:

> Changes v4->v5 (all by sashiko):
> * Explicitly match the directory separator in new entries for
>   is_userspace()
> * Allow --no-userspace to override setup in the config (using undef
>   instead of 0, separate checks in is_userspace())
> * 2 new commits:
>   - checkpatch: Factor out the check for UAPI files
>   - checkpatch: Skip kernel specific checks for userspace

Please prepare a proper [0/N] overview for the series.

Sashiko still has much to say:
	https://lore.kernel.org/lkml/20260810164552.1049483-1-pvorel@suse.cz/

In [4/5] changelog.  What does "DRY" mean?

Re: [PATCH v5 0/5] checkpatch: userspace improvements

Posted by Petr Vorel 3 weeks, 3 days ago

Hi Andrew,

> On Tue,  1 Sep 2026 12:56:37 +0200 Petr Vorel <pvorel@suse.cz> wrote:

> > Changes v4->v5 (all by sashiko):
> > * Explicitly match the directory separator in new entries for
> >   is_userspace()
> > * Allow --no-userspace to override setup in the config (using undef
> >   instead of 0, separate checks in is_userspace())
> > * 2 new commits:
> >   - checkpatch: Factor out the check for UAPI files
> >   - checkpatch: Skip kernel specific checks for userspace

> Please prepare a proper [0/N] overview for the series.

I'm sorry, what exactly is missing or wrong? (to do it correctly in v6)

> Sashiko still has much to say:
> 	https://lore.kernel.org/lkml/20260810164552.1049483-1-pvorel@suse.cz/

This is a link to v4, but I suppose you don't refer to v4 but to v5, right?

> In [4/5] changelog.  What does "DRY" mean?

https://en.wikipedia.org/wiki/Don%27t_repeat_yourself
Using single abbreviation was probably not a good idea.

Kind regards,
Petr