checkpatch: add new check PLACEHOLDER_USE

[PATCH v2 0/2] checkpatch: add new check PLACEHOLDER_USE

Posted by Onur Özkan 4 months, 3 weeks ago

Changes in v2:
  - The check is documented in Documentation/dev-tools/checkpatch.rst file.
  - Used ERROR instead of WARN on detection.

Onur Özkan (2):
  checkpatch: detect unhandled placeholders in cover letters
  checkpatch: document new check PLACEHOLDER_USE

 Documentation/dev-tools/checkpatch.rst | 10 ++++++++++
 scripts/checkpatch.pl                  |  7 +++++++
 2 files changed, 17 insertions(+)

-- 
2.51.0

Re: [PATCH v2 0/2] checkpatch: add new check PLACEHOLDER_USE

Posted by Joe Perches 4 months ago

On Wed, 2025-09-17 at 20:37 +0300, Onur Özkan wrote:
> Changes in v2:
>   - The check is documented in Documentation/dev-tools/checkpatch.rst file.
>   - Used ERROR instead of WARN on detection.
> 
> Onur Özkan (2):
>   checkpatch: detect unhandled placeholders in cover letters
>   checkpatch: document new check PLACEHOLDER_USE
> 
>  Documentation/dev-tools/checkpatch.rst | 10 ++++++++++
>  scripts/checkpatch.pl                  |  7 +++++++
>  2 files changed, 17 insertions(+)

Acked-by: Joe Perches <joe@perches.com>

Re: [PATCH v2 0/2] checkpatch: add new check PLACEHOLDER_USE

Posted by Onur Özkan 4 months ago

On Wed, 17 Sep 2025 20:37:23 +0300
Onur Özkan <work@onurozkan.dev> wrote:

> Changes in v2:
>   - The check is documented in Documentation/dev-tools/checkpatch.rst
> file.
>   - Used ERROR instead of WARN on detection.
> 
> Onur Özkan (2):
>   checkpatch: detect unhandled placeholders in cover letters
>   checkpatch: document new check PLACEHOLDER_USE
> 
>  Documentation/dev-tools/checkpatch.rst | 10 ++++++++++
>  scripts/checkpatch.pl                  |  7 +++++++
>  2 files changed, 17 insertions(+)
> 

A friendly ping on this patch. It's been over two weeks with no
feedback, would appreciate any thoughts or suggestions.

Regards,
Onur