Re: [PATCH] docs: dev-tools: checkpatch: Add targets for checkpatch tags

Jonathan Corbet
2024-05-13T13:22:44Z

Diederik de Haas <didi.debian@cknow.org> writes:

> Make the tags directly linkable by defining targets for them.
>
> Closes: https://lore.kernel.org/r/8090211.0vHzs8tI1a@bagend/
> Signed-off-by: Diederik de Haas <didi.debian@cknow.org>
> ---
> Documentation/dev-tools/checkpatch.rst | 216 +++++++++++++++++++++++++
> 1 file changed, 216 insertions(+)

In addition to the other comments, I have to add: you have said what you
are doing but not *why*. All of these labels clutter up the text; what
use will be made of them to justify that?

Thanks,

jon