summaryrefslogtreecommitdiff
path: root/Documentation/gerrit_guidelines.md
diff options
context:
space:
mode:
authorLee Leahy <leroy.p.leahy@intel.com>2017-03-07 12:01:02 -0800
committerLee Leahy <leroy.p.leahy@intel.com>2017-03-09 17:18:02 +0100
commitbb4ae07417c9b3be1af500a62514a2d42467cfb0 (patch)
tree36bb50ef045fe12ac50e25de03e4a610a658feab /Documentation/gerrit_guidelines.md
parente0f5dfc6785c53217c0d97331b3ada46a51cc016 (diff)
downloadcoreboot-bb4ae07417c9b3be1af500a62514a2d42467cfb0.tar.xz
src/include: Add do { } while(0) around macros
Fix the following error detected by checkpatch.py: ERROR: Macros with multiple statements should be enclosed in a do - while loop False positives are generated when assembly code is used in a macro. An example is: ERROR: Macros with multiple statements should be enclosed in a do - while loop +#define post_code(value) \ + movb $value, %al; \ + outb %al, $CONFIG_POST_IO_PORT False positives are also generated for linker script include files. An example is: ERROR: Macros with multiple statements should be enclosed in a do - while loop +#define SET_COUNTER(name, addr) \ + _ = ASSERT(. <= addr, STR(name overlaps the previous region!)); \ + . = addr; False positives are also generated for attribute macros. An example is: ERROR: Macros with multiple statements should be enclosed in a do - while loop +#define DISABLE_TRACE_ON_FUNCTION __attribute__ ((no_instrument_function)); TEST=Build and run on Galileo Gen2 Change-Id: I88abf96579e906f6962d558a3d09907f07d00b1c Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com> Reviewed-on: https://review.coreboot.org/18644 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'Documentation/gerrit_guidelines.md')
0 files changed, 0 insertions, 0 deletions