summaryrefslogtreecommitdiff
path: root/drivers/base/class.c
diff options
context:
space:
mode:
authorGeert Uytterhoeven <geert+renesas@glider.be>2024-12-05 19:16:34 +0100
committerJonathan Corbet <corbet@lwn.net>2024-12-30 11:40:45 -0700
commit6356f18f09dc0781650c4f128ea48745fa48c415 (patch)
tree19bed6e1a028b2252d7e37238ebdcff71ec367d0 /drivers/base/class.c
parent3f997cbf676b304f927aab9c43adfcb7b32cbc56 (diff)
downloadlwn-docs-mw.tar.gz
lwn-docs-mw.zip
Align git commit ID abbreviation guidelines and checksdocs-mw
The guidelines for git commit ID abbreviation are inconsistent: some places state to use 12 characters exactly, while other places recommend 12 characters or more. The same issue is present in the checkpatch.pl script. E.g. Documentation/dev-tools/checkpatch.rst says: **GIT_COMMIT_ID** The proper way to reference a commit id is: commit <12+ chars of sha1> ("<title line>") However, scripts/checkpatch.pl has two different checks: one warning check accepting 12 characters exactly: # Check Fixes: styles is correct Please use correct Fixes: style 'Fixes: <12 chars of sha1> (\"<title line>\")' and a second error check accepting 12-40 characters: # Check for git id commit length and improperly formed commit descriptions # A correctly formed commit description is: # commit <SHA-1 hash length 12+ chars> ("Complete commit subject") Please use git commit description style 'commit <12+ chars of sha1> Hence patches containing commit IDs with more than 12 characters are flagged by checkpatch, and sometimes rejected by maintainers or reviewers. This is becoming more important with the growth of the repository, as git may decide to use more characters in case of local conflicts. Fix this by settling on at least 12 characters, in both the documentation and in the checkpatch.pl script. Fixes: bd17e036b495bebb ("checkpatch: warn for non-standard fixes tag style") Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Link: https://lore.kernel.org/r/1c244040bf6ce304656e31036e5178b4b9dfb719.1733421037.git.geert+renesas@glider.be Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Diffstat (limited to 'drivers/base/class.c')
0 files changed, 0 insertions, 0 deletions