sdk-nrf-upmerge #73050
compliance.yml
on: pull_request
Run compliance checks on patch series (PR)
1m 50s
Annotations
7 errors and 8 warnings
Run compliance checks on patch series (PR):
KconfigBasic.txt#L1
See https://docs.zephyrproject.org/latest/build/kconfig/tips.html for more details.
Undefined Kconfig symbols:
warning: undefined symbol SOC_NRF54L_VREG_MAIN_DCDC:
- Referenced at boards/nordic/nrf54l15pdk/Kconfig:8:
config BOARD_ENABLE_DCDC
bool "DCDC mode"
default y
select SOC_NRF54L_VREG_MAIN_DCDC
depends on BOARD_NRF54L15PDK_NRF54L15_CPUAPP
warning: undefined symbol WPA_SUPP:
- Referenced at subsys/net/conn_mgr/Kconfig:25:
config NET_CONNECTION_MANAGER_MONITOR_STACK_SIZE
int "Size of the stack allocated for the conn_mgr_monitor thread"
default 8192 if WPA_SUPP
default 512
depends on NET_CONNECTION_MANAGER && NETWORKING
help
Sets the stack size which will be used by the connection manager for connectivity monitoring.
|
Run compliance checks on patch series (PR):
GitDiffCheck.txt#L1
Git conflict markers and whitespace errors are not allowed in added changes
f6014f86e6b5b4824759c2a9e73dafee373f9e91: subsys/nrf_security/src/utils/nrf_security_mutexes.h:29: trailing whitespace.
f6014f86e6b5b4824759c2a9e73dafee373f9e91: subsys/nrf_security/src/utils/nrf_security_mutexes.h:31: trailing whitespace.
a99535689f0e87d3fbf504f11b9fb5bb9c5affea: subsys/nrf_security/src/drivers/CMakeLists.txt:13: trailing whitespace.
b8714470bf50c1404454a9c84ebf8e77b89c8286: subsys/nrf_security/src/legacy/CMakeLists.txt:41: trailing whitespace.
b0782554e161f3b9b2e8838aa960ee2e23be7bbe: tests/subsys/nrf_compress/decompression/lzma/prj.conf:16: new blank line at EOF.
b3531ec10f2e697401991eea79219f4e5d78eb94: subsys/nrf_security/src/drivers/cracen/CMakeLists.txt:50: trailing whitespace.
b3531ec10f2e697401991eea79219f4e5d78eb94: subsys/nrf_security/src/utils/nrf_security_utils.cmake:18: trailing whitespace.
b3531ec10f2e697401991eea79219f4e5d78eb94: subsys/nrf_security/src/utils/nrf_security_utils.cmake:28: trailing whitespace.
ee5f5ad40725574c59b8a8f24a4f41a3dbbf4ccd: subsys/nrf_security/src/utils/nrf_security_utils.cmake:18: trailing whitespace.
858075b19fc5547f39352c69b122e2f3ed713ed2: subsys/nrf_security/src/threading/threading.cmake:23: trailing whitespace.
90f16c201bc2d7ac10320522fce9aeba91cf17b7: subsys/nrf_security/src/CMakeLists.txt:188: trailing whitespace.
6eb0d1d78a35ce3058121026b121741dc668ef39: subsys/nrf_security/cmake/extensions.cmake:147: trailing whitespace.
6eb0d1d78a35ce3058121026b121741dc668ef39: subsys/nrf_security/cmake/extensions.cmake:153: trailing whitespace.
6eb0d1d78a35ce3058121026b121741dc668ef39: subsys/nrf_security/cmake/extensions.cmake:157: trailing whitespace.
393f25b70abd9225421b4278995060ebe02f6eba: subsys/nrf_security/src/threading/CMakeLists.txt:21: trailing whitespace.
393f25b70abd9225421b4278995060ebe02f6eba: subsys/nrf_security/src/threading/CMakeLists.txt:33: trailing whitespace.
5772eaa332cb404f9aa35dab98f3cb85e7a2b8f4: subsys/nrf_security/src/CMakeLists.txt:137: trailing whitespace.
5772eaa332cb404f9aa35dab98f3cb85e7a2b8f4: subsys/nrf_security/src/CMakeLists.txt:139: trailing whitespace.
5772eaa332cb404f9aa35dab98f3cb85e7a2b8f4: subsys/nrf_security/src/core/nrf_oberon/CMakeLists.txt:54: new blank line at EOF.
5772eaa332cb404f9aa35dab98f3cb85e7a2b8f4: subsys/nrf_security/src/drivers/nrf_cc3xx/CMakeLists.txt:8: trailing whitespace.
5772eaa332cb404f9aa35dab98f3cb85e7a2b8f4: subsys/nrf_security/src/drivers/nrf_oberon/CMakeLists.txt:7: trailing whitespace.
f51f4ca404b7ed809ae3571062f3cb011c4ce24e: subsys/nrf_security/cmake/generate_configs.cmake:47: trailing whitespace.
f51f4ca404b7ed809ae3571062f3cb011c4ce24e: subsys/nrf_security/cmake/generate_configs.cmake:59: trailing whitespace.
f51f4ca404b7ed809ae3571062f3cb011c4ce24e: subsys/nrf_security/cmake/generate_configs.cmake:134: trailing whitespace.
f51f4ca404b7ed809ae3571062f3cb011c4ce24e: subsys/nrf_security/src/legacy/CMakeLists.txt:136: trailing whitespace.
f51f4ca404b7ed809ae3571062f3cb011c4ce24e: subsys/nrf_security/src/zephyr/CMakeLists.txt:9: trailing whitespace.
d8cd1dee1feffb6e563312a0d8f6ba54519fff74: subsys/nrf_security/src/CMakeLists.txt:144: trailing whitespace.
d8cd1dee1feffb6e563312a0d8f6ba54519fff74: subsys/nrf_security/src/CMakeLists.txt:158: trailing whitespace.
c85f9247137149561c82f04b3d16655074507886: modules/trusted-firmware-m/tfm_boards/external_core.cmake:16: trailing whitespace.
c85f9247137149561c82f04b3d16655074507886: modules/trusted-firmware-m/tfm_boards/external_core.cmake:50: trailing whitespace.
c85f9247137149561c82f04b3d16655074507886: modules/trusted-firmware-m/tfm_boards/external_core_install.cmake:33: trailing whitespace.
60270537016f37ca5ddb6a25399b4312422e5cba: subsys/nrf_security/configs/config_extra.cmake.in:12: trailing whitespace.
9ff93b8584b4236ccfdf79a207e3d87e71bd7076: subsys/nrf_security/cmake/nrf_config.cmake:6: trailing whitespace.
9ff93b8584b4236ccfdf79a207e3d87e71bd7076: subsys/nrf_security/configs/nrf-config.h.template:8: trailing whitespace.
112e8163589d0f06837f52068c1d787b12d51d0d: subsys/nrf_
|
Run compliance checks on patch series (PR):
Checkpatch.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style for more details.
-:1700: ERROR:CODE_INDENT: code indent should use tabs where possible
#1700: FILE: applications/serial_lte_modem/src/slm_ppp.c:103:
+^I^I^I^I htons(IPPROTO_RAW));$
-:21923: ERROR:SPACING: need consistent spacing around '-' (ctx:WxV)
#21923: FILE: include/tfm/platform_otp_ids.h:17:
+ PLAT_OTP_ID_LCS = UINT32_MAX -1,
^
-:26548: ERROR:TRAILING_WHITESPACE: trailing whitespace
#26548: FILE: modules/trusted-firmware-m/tfm_boards/external_core.cmake:16:
+# Note that $
-:26582: ERROR:TRAILING_WHITESPACE: trailing whitespace
#26582: FILE: modules/trusted-firmware-m/tfm_boards/external_core.cmake:50:
+ ${ARM_MBEDTLS_PATH}/include/library $
-:26706: ERROR:TRAILING_WHITESPACE: trailing whitespace
#26706: FILE: modules/trusted-firmware-m/tfm_boards/external_core_install.cmake:33:
+ DESTINATION $
-:33833: WARNING:TYPO_SPELLING: 'an an' may be misspelled - perhaps 'an'?
#33833: FILE: subsys/nrf_security/cmake/extensions.cmake:170:
+ # Add an an filtered version of zephyr_interface (PSA crypto interface filtered out)
-:33837: ERROR:TRAILING_WHITESPACE: trailing whitespace
#33837: FILE: subsys/nrf_security/cmake/extensions.cmake:174:
+ target_include_directories(${lib_name} $
-:33842: ERROR:TRAILING_WHITESPACE: trailing whitespace
#33842: FILE: subsys/nrf_security/cmake/extensions.cmake:179:
+ target_include_directories(${lib_name} $
-:33914: ERROR:TRAILING_WHITESPACE: trailing whitespace
#33914: FILE: subsys/nrf_security/cmake/generate_configs.cmake:47:
+ # Generate MBEDCRYPTO_CONFIG_FILE $
-:33926: ERROR:TRAILING_WHITESPACE: trailing whitespace
#33926: FILE: subsys/nrf_security/cmake/generate_configs.cmake:59:
+ # Restore the backup configurations $
-:33999: ERROR:TRAILING_WHITESPACE: trailing whitespace
#33999: FILE: subsys/nrf_security/cmake/generate_configs.cmake:132:
+ # Restore the backup configurations $
-:34113: ERROR:TRAILING_WHITESPACE: trailing whitespace
#34113: FILE: subsys/nrf_security/cmake/nrf_config.cmake:6:
+# Convert all platform and TLS/DTLS and X.509 Kconfig variables for Mbed TLS $
-:34584: ERROR:TRAILING_WHITESPACE: trailing whitespace
#34584: FILE: subsys/nrf_security/configs/config_extra.cmake.in:12:
+# It is added to circumvent issues with install-targets inside TF-M and to unify the $
-:34713: ERROR:TRAILING_WHITESPACE: trailing whitespace
#34713: FILE: subsys/nrf_security/configs/nrf-config.h.template:8:
+ * added to the build and used by accident. Hence, this guard is not $
-:35379: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35379: FILE: subsys/nrf_security/include/crypto_compat.h:43:
+ return mbedtls_svc_key_id_is_null(handle);$
-:35443: ERROR:CODE_INDENT: code indent should use tabs where possible
#35443: FILE: subsys/nrf_security/include/crypto_compat.h:107:
+ psa_key_handle_t *handle);$
-:35443: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35443: FILE: subsys/nrf_security/include/crypto_compat.h:107:
+ psa_key_handle_t *handle);$
-:35499: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35499: FILE: subsys/nrf_security/include/crypto_compat.h:163:
+ ((psa_dh_family_t) MBEDTLS_DEPRECATED_NUMERIC_CONSTANT(0x7e))$
-:35518: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35518: FILE: subsys/nrf_security/include/crypto_compat.h:182:
+ psa_key_attributes_t *attributes,$
-:35519: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35519: FILE: subsys/nrf_security/include/crypto_compat.h:183:
+ psa_key_type_t type, const uint8_t *data, size_t data_length)$
-:35521: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35521: FILE: subsys/nrf_security/include/crypto_compat.h:185:
+ (void) data;$
-:35522: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35522: FILE: subsys/nrf_security/include/crypto_compat.h:186:
+ if (data_length != 0) {$
-:355
|
Run compliance checks on patch series (PR):
YAMLLint.txt#L1
Check YAML files with YAMLLint.
YAMLLint (comments):missing starting space in comment
File:samples/bootloader/pm.yml
Line:1
Column:2
YAMLLint (comments):missing starting space in comment
File:samples/bootloader/pm.yml
Line:39
Column:2
YAMLLint (comments-indentation):comment not indented like content
File:samples/bootloader/pm.yml
Line:39
Column:1
YAMLLint (comments):missing starting space in comment
File:samples/bootloader/pm.yml
Line:41
Column:2
YAMLLint (comments-indentation):comment not indented like content
File:samples/bootloader/pm.yml
Line:41
Column:1
YAMLLint (comments):missing starting space in comment
File:samples/nrf5340/netboot/pm.yml
Line:1
Column:2
|
Run compliance checks on patch series (PR):
Nits.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style for more details.
Please remove blank lines at end of 'subsys/nrf_security/src/core/nrf_oberon/CMakeLists.txt'
Missing newline at end of 'subsys/nrf_security/src/drivers/nrf_cc3xx/CMakeLists.txt'. Check your text editor settings.
Please remove blank lines at start of 'subsys/nrf_security/src/threading/threading_alt.c'
Missing newline at end of 'tests/subsys/net/lib/wifi_credentials_backend_psa/include/mbedtls_config.h'. Check your text editor settings.
Please remove blank lines at end of 'tests/subsys/nrf_compress/decompression/lzma/prj.conf'
|
Run compliance checks on patch series (PR)
Process completed with exit code 1.
|
Run compliance checks on patch series (PR)
Process completed with exit code 5.
|
Run compliance checks on patch series (PR)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
YAMLLint (comments):
samples/bootloader/pm.yml#L1
samples/bootloader/pm.yml:1 missing starting space in comment
|
YAMLLint (comments):
samples/bootloader/pm.yml#L39
samples/bootloader/pm.yml:39 missing starting space in comment
|
YAMLLint (comments-indentation):
samples/bootloader/pm.yml#L39
samples/bootloader/pm.yml:39 comment not indented like content
|
YAMLLint (comments):
samples/bootloader/pm.yml#L41
samples/bootloader/pm.yml:41 missing starting space in comment
|
YAMLLint (comments-indentation):
samples/bootloader/pm.yml#L41
samples/bootloader/pm.yml:41 comment not indented like content
|
YAMLLint (comments):
samples/nrf5340/netboot/pm.yml#L1
samples/nrf5340/netboot/pm.yml:1 missing starting space in comment
|
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "compliance.xml".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
compliance.xml
|
110 KB |
|