This is an automated email from Gerrit. "Antonio Borneo <[email protected]>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/9783
-- gerrit commit b81761b1e2ad48e2cccacc34326b2e1e1030dcfe Author: Antonio Borneo <[email protected]> Date: Sun Jul 12 23:49:22 2026 +0200 checkpatch: add list of OpenOCD attributes The script checkpatch gets confused by function attributes that it does not understand, e.g.: tools/scripts/checkpatch.pl --types SPACING \ -f src/jtag/drivers/OpenULINK/src/main.c produces ERROR:SPACING: No space is necessary after a cast #14: FILE: src/jtag/drivers/OpenULINK/src/main.c:14: +extern void sudav_isr(void) __interrupt(SUDAV_ISR); because it interprets '(void)' as a cast for the rest of the line. Add the attribute '__interrupt' in the list of known attributes. Change-Id: I70f1ebe21e14922d13323a22b970034d406b6fbf Signed-off-by: Antonio Borneo <[email protected]> diff --git a/tools/scripts/checkpatch.pl b/tools/scripts/checkpatch.pl index f77f364d1c..010b7f43d9 100755 --- a/tools/scripts/checkpatch.pl +++ b/tools/scripts/checkpatch.pl @@ -512,9 +512,14 @@ our $InitAttributeConst = qr{$InitAttributePrefix(?:initconst\b)}; our $InitAttributeInit = qr{$InitAttributePrefix(?:init\b)}; our $InitAttribute = qr{$InitAttributeData|$InitAttributeConst|$InitAttributeInit}; +# OpenOCD specific: Begin: list of attributes +our $OpenOCD_Attribute = qr{__interrupt}; +# OpenOCD specific: End + # Notes to $Attribute: # We need \b after 'init' otherwise 'initconst' will cause a false positive in a check our $Attribute = qr{ + $OpenOCD_Attribute| const| volatile| __percpu| --
