diff options
author | Jack Rosenthal <jrosenth@chromium.org> | 2019-05-09 09:40:32 -0600 |
---|---|---|
committer | Jett Rink <jettrink@chromium.org> | 2019-05-20 16:55:54 +0000 |
commit | f1a16a6688f52e3de9809402564b26af67f265a6 (patch) | |
tree | 48ca329c12424ff51c8d323942f8bde9344d3b7e | |
parent | af779818bfe909b46e9b26195b7d7f1c2e986880 (diff) | |
download | chrome-ec-f1a16a6688f52e3de9809402564b26af67f265a6.tar.gz |
common: add __maybe_unused attribute
Functions which might go unused under a particular set of CONFIG_
options have historically been wrapped in an ifdef to clear up
compiler warnings about unused functions.
Since we are trying to reduce conditional compilation in favor of
macros like IS_ENABLED, the Linux kernel style guide suggests using
__maybe_unused for these functions. This adds the __maybe_unused
macro.
BUG=none
BRANCH=none
TEST=make buildall -j
Change-Id: Ia8239ad566b5aa518c258eaab0e4dceada790da4
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1602489
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
-rw-r--r-- | include/common.h | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/include/common.h b/include/common.h index b275b27e1f..388d3f413f 100644 --- a/include/common.h +++ b/include/common.h @@ -68,6 +68,17 @@ #endif /* + * __maybe_unused is equivalent to the Linux kernel definition, so we + * can follow the Kernel style guide more closely. + * + * An example use case is a function which is only used under certain + * CONFIG options. + */ +#ifndef __maybe_unused +#define __maybe_unused __attribute__((unused)) +#endif + +/* * externally_visible is required by GCC to avoid kicking out memset. */ #ifndef __visible |