From 794dd05459236e25c9be184d36f9c0bb70b86f4a Mon Sep 17 00:00:00 2001 From: Pieter De Gendt Date: Fri, 25 Oct 2024 15:02:54 +0200 Subject: [PATCH] scripts: ci: check_compliance: Keep UNDEF list sorted Keep the UNDEF_KCONFIG_ALLOWLIST list sorted, as mentioned in the comment. Signed-off-by: Pieter De Gendt --- scripts/ci/check_compliance.py | 52 ++++++++++++++++++---------------- 1 file changed, 27 insertions(+), 25 deletions(-) diff --git a/scripts/ci/check_compliance.py b/scripts/ci/check_compliance.py index a1db8bc7c54..5de6a81e44b 100755 --- a/scripts/ci/check_compliance.py +++ b/scripts/ci/check_compliance.py @@ -909,6 +909,7 @@ flagged. # Many of these are symbols used as examples. Note that the list is sorted # alphabetically, and skips the CONFIG_ prefix. UNDEF_KCONFIG_ALLOWLIST = { + # zephyr-keep-sorted-start re(^\s+") "ALSO_MISSING", "APP_LINK_WITH_", "APP_LOG_LEVEL", # Application log level is not detected correctly as @@ -919,51 +920,52 @@ flagged. # toolchain Kconfig which is sourced based on # Zephyr toolchain variant and therefore not # visible to compliance. + "BINDESC_", # Used in documentation as a prefix "BOARD_", # Used as regex in scripts/utils/board_v1_to_v2.py "BOARD_MPS2_AN521_CPUTEST", # Used for board and SoC extension feature tests "BOARD_NATIVE_SIM_NATIVE_64_TWO", # Used for board and SoC extension feature tests "BOARD_NATIVE_SIM_NATIVE_ONE", # Used for board and SoC extension feature tests "BOOT_DIRECT_XIP", # Used in sysbuild for MCUboot configuration "BOOT_DIRECT_XIP_REVERT", # Used in sysbuild for MCUboot configuration - "BOOT_FIRMWARE_LOADER", # Used in sysbuild for MCUboot configuration - "BOOT_RAM_LOAD", # Used in sysbuild for MCUboot configuration - "BOOT_SWAP_USING_MOVE", # Used in sysbuild for MCUboot configuration - "BOOT_SWAP_USING_SCRATCH", # Used in sysbuild for MCUboot configuration "BOOT_ENCRYPTION_KEY_FILE", # Used in sysbuild "BOOT_ENCRYPT_IMAGE", # Used in sysbuild + "BOOT_FIRMWARE_LOADER", # Used in sysbuild for MCUboot configuration "BOOT_MAX_IMG_SECTORS_AUTO", # Used in sysbuild - "BINDESC_", # Used in documentation as a prefix - "BOOT_UPGRADE_ONLY", # Used in example adjusting MCUboot config, but - # symbol is defined in MCUboot itself. + "BOOT_RAM_LOAD", # Used in sysbuild for MCUboot configuration "BOOT_SERIAL_BOOT_MODE", # Used in (sysbuild-based) test/ # documentation "BOOT_SERIAL_CDC_ACM", # Used in (sysbuild-based) test "BOOT_SERIAL_ENTRANCE_GPIO", # Used in (sysbuild-based) test "BOOT_SERIAL_IMG_GRP_HASH", # Used in documentation + "BOOT_SHARE_BACKEND_RETENTION", # Used in Kconfig text "BOOT_SHARE_DATA", # Used in Kconfig text "BOOT_SHARE_DATA_BOOTINFO", # Used in (sysbuild-based) test - "BOOT_SHARE_BACKEND_RETENTION", # Used in Kconfig text "BOOT_SIGNATURE_KEY_FILE", # MCUboot setting used by sysbuild "BOOT_SIGNATURE_TYPE_ECDSA_P256", # MCUboot setting used by sysbuild "BOOT_SIGNATURE_TYPE_ED25519", # MCUboot setting used by sysbuild "BOOT_SIGNATURE_TYPE_NONE", # MCUboot setting used by sysbuild "BOOT_SIGNATURE_TYPE_RSA", # MCUboot setting used by sysbuild + "BOOT_SWAP_USING_MOVE", # Used in sysbuild for MCUboot configuration + "BOOT_SWAP_USING_SCRATCH", # Used in sysbuild for MCUboot configuration + "BOOT_UPGRADE_ONLY", # Used in example adjusting MCUboot config, but + # symbol is defined in MCUboot itself. "BOOT_VALIDATE_SLOT0", # Used in (sysbuild-based) test "BOOT_WATCHDOG_FEED", # Used in (sysbuild-based) test + "BT_6LOWPAN", # Defined in Linux, mentioned in docs "CDC_ACM_PORT_NAME_", "CHRE", # Optional module "CHRE_LOG_LEVEL_DBG", # Optional module "CLOCK_STM32_SYSCLK_SRC_", + "CMD_CACHE", # Defined in U-Boot, mentioned in docs "CMU", "COMPILER_RT_RTLIB", - "BT_6LOWPAN", # Defined in Linux, mentioned in docs - "CMD_CACHE", # Defined in U-Boot, mentioned in docs "CRC", # Used in TI CC13x2 / CC26x2 SDK comment "DEEP_SLEEP", # #defined by RV32M1 in ext/ "DESCRIPTION", "ERR", "ESP_DIF_LIBRARY", # Referenced in CMake comment "EXPERIMENTAL", + "EXTRA_FIRMWARE_DIR", # Linux, in boards/xtensa/intel_adsp_cavs25/doc "FFT", # Used as an example in cmake/extensions.cmake "FLAG", # Used as an example "FOO", @@ -971,24 +973,28 @@ flagged. "FOO_SETTING_1", "FOO_SETTING_2", "HEAP_MEM_POOL_ADD_SIZE_", # Used as an option matching prefix - "LSM6DSO_INT_PIN", + "HUGETLBFS", # Linux, in boards/xtensa/intel_adsp_cavs25/doc "LIBGCC_RTLIB", "LLVM_USE_LD", # Both LLVM_USE_* are in cmake/toolchain/llvm/Kconfig "LLVM_USE_LLD", # which are only included if LLVM is selected but # not other toolchains. Compliance check would complain, # for example, if you are using GCC. - "MCUBOOT_LOG_LEVEL_WRN", # Used in example adjusting MCUboot - # config, - "MCUBOOT_LOG_LEVEL_INF", - "MCUBOOT_DOWNGRADE_PREVENTION", # but symbols are defined in MCUboot - # itself. + "LOG_BACKEND_MOCK_OUTPUT_DEFAULT", #Referenced in tests/subsys/logging/log_syst + "LOG_BACKEND_MOCK_OUTPUT_SYST", #Referenced in testcase.yaml of log_syst test + "LSM6DSO_INT_PIN", "MCUBOOT_ACTION_HOOKS", # Used in (sysbuild-based) test "MCUBOOT_CLEANUP_ARM_CORE", # Used in (sysbuild-based) test + "MCUBOOT_DOWNGRADE_PREVENTION", # but symbols are defined in MCUboot + # itself. + "MCUBOOT_LOG_LEVEL_INF", + "MCUBOOT_LOG_LEVEL_WRN", # Used in example adjusting MCUboot + # config, "MCUBOOT_SERIAL", # Used in (sysbuild-based) test/ # documentation "MCUMGR_GRP_EXAMPLE_OTHER_HOOK", # Used in documentation "MISSING", "MODULES", + "MODVERSIONS", # Linux, in boards/xtensa/intel_adsp_cavs25/doc "MYFEATURE", "MY_DRIVER_0", "NORMAL_SLEEP", # #defined by RV32M1 in ext/ @@ -1000,8 +1006,7 @@ flagged. "REG1", "REG2", "RIMAGE_SIGNING_SCHEMA", # Optional module - "LOG_BACKEND_MOCK_OUTPUT_DEFAULT", #Referenced in tests/subsys/logging/log_syst - "LOG_BACKEND_MOCK_OUTPUT_SYST", #Referenced in testcase.yaml of log_syst test + "SECURITY_LOADPIN", # Linux, in boards/xtensa/intel_adsp_cavs25/doc "SEL", "SHIFT", "SINGLE_APPLICATION_SLOT", # Used in sysbuild for MCUboot configuration @@ -1014,6 +1019,9 @@ flagged. "SRAM2", # Referenced in a comment in samples/application_development "STACK_SIZE", # Used as an example in the Kconfig docs "STD_CPP", # Referenced in CMake comment + "SUIT_MPI_APP_AREA_PATH", # Used by nRF runners to program provisioning data, based on build configuration + "SUIT_MPI_GENERATE", # Used by nRF runners to program provisioning data, based on build configuration + "SUIT_MPI_RAD_AREA_PATH", # Used by nRF runners to program provisioning data, based on build configuration "TEST1", "TOOLCHAIN_ARCMWDT_SUPPORTS_THREAD_LOCAL_STORAGE", # The symbol is defined in the toolchain # Kconfig which is sourced based on Zephyr @@ -1023,16 +1031,10 @@ flagged. "USB_CONSOLE", "USE_STDC_", "WHATEVER", - "EXTRA_FIRMWARE_DIR", # Linux, in boards/xtensa/intel_adsp_cavs25/doc - "HUGETLBFS", # Linux, in boards/xtensa/intel_adsp_cavs25/doc - "MODVERSIONS", # Linux, in boards/xtensa/intel_adsp_cavs25/doc - "SECURITY_LOADPIN", # Linux, in boards/xtensa/intel_adsp_cavs25/doc "ZEPHYR_TRY_MASS_ERASE", # MCUBoot setting described in sysbuild # documentation "ZTEST_FAIL_TEST_", # regex in tests/ztest/fail/CMakeLists.txt - "SUIT_MPI_GENERATE", # Used by nRF runners to program provisioning data, based on build configuration - "SUIT_MPI_APP_AREA_PATH", # Used by nRF runners to program provisioning data, based on build configuration - "SUIT_MPI_RAD_AREA_PATH", # Used by nRF runners to program provisioning data, based on build configuration + # zephyr-keep-sorted-stop }