diff options
| author | Alper Nebi Yasak <alpernebiyasak@gmail.com> | 2020-10-05 09:57:30 +0300 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2020-10-14 11:16:34 -0400 |
| commit | b9cca2c57a7f3f51dc044030a2f1665e517edb51 (patch) | |
| tree | 42172f930f85c5529e8ea9670b8c6e84cd0dc247 /tools | |
| parent | 957a3e549314c3d9c587c91d0b582d73d3d00da7 (diff) | |
| download | u-boot-b9cca2c57a7f3f51dc044030a2f1665e517edb51.tar.gz u-boot-b9cca2c57a7f3f51dc044030a2f1665e517edb51.tar.xz u-boot-b9cca2c57a7f3f51dc044030a2f1665e517edb51.zip | |
checkpatch.pl: Make CONFIG_IS_ENABLED(CONFIG_*) an error
CONFIG_IS_ENABLED() takes the kconfig name without the CONFIG_ prefix,
e.g. CONFIG_IS_ENABLED(CLK) for CONFIG_CLK. Make including the prefix
an error in checkpatch.pl so calls in the wrong format aren't
accidentally reintroduced.
Signed-off-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/patman/test_checkpatch.py | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tools/patman/test_checkpatch.py b/tools/patman/test_checkpatch.py index 792196e689..f71c70fb13 100644 --- a/tools/patman/test_checkpatch.py +++ b/tools/patman/test_checkpatch.py @@ -405,6 +405,12 @@ index 0000000..2234c87 pm.add_line('include/myfile.h', '#include <dm.h>') self.checkSingleMessage(pm, 'BARRED_INCLUDE_IN_HDR', 'error') + def testConfigIsEnabledConfig(self): + """Test for accidental CONFIG_IS_ENABLED(CONFIG_*) calls""" + pm = PatchMaker() + pm.add_line('common/main.c', 'if (CONFIG_IS_ENABLED(CONFIG_CLK))') + self.checkSingleMessage(pm, 'CONFIG_IS_ENABLED_CONFIG', 'error') + if __name__ == "__main__": unittest.main() |