Skip to content

Upmerge 2024.12.20 #8463

Upmerge 2024.12.20

Upmerge 2024.12.20 #8463

Triggered via pull request January 24, 2025 12:39
Status Failure
Total duration 31m 16s
Artifacts 1

compliance.yml

on: pull_request
Run compliance checks on patch series (PR)
31m 2s
Run compliance checks on patch series (PR)
Fit to window
Zoom out
Zoom in

Annotations

19 errors and 5 warnings
Run compliance checks on patch series (PR): BinaryFiles.txt#L1
No binary files allowed. Binary file not allowed: samples/net/sockets/http_server/src/certs/ca.der Binary file not allowed: samples/net/sockets/http_server/src/certs/ca_cert.der Binary file not allowed: samples/net/sockets/http_server/src/certs/server_cert.der Binary file not allowed: samples/net/sockets/http_server/src/certs/server_privkey.der
Run compliance checks on patch series (PR): Checkpatch.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style for more details. -:8907: WARNING:BRACES: braces {} are required around if/while/for/else #8907: FILE: arch/xtensa/core/elf.c:48: + if (ext->sect_hdrs[sh_ndx].sh_addr <= *got_entry && + *got_entry < -:118385: WARNING:BRACES: braces {} are required around if/while/for/else #118385: FILE: drivers/clock_control/clock_control_npcm.c:346: + while (sys_test_bit(priv->hfcgctrl, NPCM_HFCGCTRL_CLK_CHNG)) + ; -:122280: WARNING:BRACES: braces {} are required around if/while/for/else #122280: FILE: drivers/counter/counter_renesas_ra_agt.c:76: + while (!(reg->AGTCR & BIT(R_AGTX0_AGT16_CTRL_AGTCR_TCSTF_Pos)) && likely(--timeout)) + ; -:122293: WARNING:BRACES: braces {} are required around if/while/for/else #122293: FILE: drivers/counter/counter_renesas_ra_agt.c:89: + while ((reg->AGTCR & BIT(R_AGTX0_AGT16_CTRL_AGTCR_TCSTF_Pos)) && likely(--timeout)) + ; -:130101: WARNING:BRACES: braces {} are required around if/while/for/else #130101: FILE: drivers/dma/dma_mcux_edma.c:640: + if (pre_idx >= CONFIG_DMA_TCD_QUEUE_SIZE) + pre_idx = CONFIG_DMA_TCD_QUEUE_SIZE - 1; -:131040: WARNING:BRACES: braces {} are required around if/while/for/else #131040: FILE: drivers/dma/dma_nxp_sdma.c:79: + if (bytes > chan_data->stat.pending_length) + return -EINVAL; -:131046: WARNING:BRACES: braces {} must be used on all arms of this statement #131046: FILE: drivers/dma/dma_nxp_sdma.c:85: + if (chan_data->stat.read_position > chan_data->stat.write_position) [...] + else [...] -:131060: WARNING:BRACES: braces {} are required around if/while/for/else #131060: FILE: drivers/dma/dma_nxp_sdma.c:99: + if (bytes > chan_data->stat.free) + return -EINVAL; -:131066: WARNING:BRACES: braces {} must be used on all arms of this statement #131066: FILE: drivers/dma/dma_nxp_sdma.c:105: + if (chan_data->stat.write_position > chan_data->stat.read_position) [...] + else [...] -:131098: WARNING:BRACES: braces {} are required around if/while/for/else #131098: FILE: drivers/dma/dma_nxp_sdma.c:137: + if (chan_data->cb) + chan_data->cb(chan_data->dev, chan_data->arg, i, DMA_STATUS_BLOCK); -:131342: WARNING:BRACES: braces {} are required around if/while/for/else #131342: FILE: drivers/dma/dma_nxp_sdma.c:381: + if (!size) + return 0; -:131345: WARNING:BRACES: braces {} must be used on all arms of this statement #131345: FILE: drivers/dma/dma_nxp_sdma.c:384: + if (chan_data->direction == MEMORY_TO_PERIPHERAL) [...] + else [...] -:131377: WARNING:BRACES: braces {} are required around if/while/for/else #131377: FILE: drivers/dma/dma_nxp_sdma.c:416: + if (chan_id == 0) + return false; -:131380: WARNING:BRACES: braces {} are required around if/while/for/else #131380: FILE: drivers/dma/dma_nxp_sdma.c:419: + if (chan_id >= FSL_FEATURE_SDMA_MODULE_CHANNEL) + return false; -:145122: WARNING:BRACES: braces {} are required around if/while/for/else #145122: FILE: drivers/flash/flash_stm32h7x.c:267: + while ((FLASH_STM32_REGS(dev)->SR1 & FLASH_SR_QW) || + (FLASH_STM32_REGS(dev)->SR2 & FLASH_SR_QW)) #else while (FLASH_STM32_REGS(dev)->SR1 & FLASH_SR_QW) #endif -:152075: WARNING:BRACES: braces {} are required around if/while/for/else #152075: FILE: drivers/gpio/gpio_max149x6.h:16: + if ((bit1) & (bit2)) \ + LOG_ERR("[%s] %d", #bit1, bit1) +#define PRINT_ERR(bit) \ + if (bit) \ + LOG_ERR("[DIAG] [%s] %d\n", #bit, bit) +#define PRINT_INF(bit) LOG_INFO("[%s] %d\n", #bit, bit) -:152078: WARNING:BRACES: braces {} are required around if/while/for/else #152078: FILE: drivers/gpio/gpio_max149x6.h:19: + if (bit) \ + LOG_ERR("[DIAG] [%s] %d\n", #bit, bit) +#define PRINT_INF(bit) LOG_INFO("[%s] %d\n", #bit, bit) +#define LOG_DIAG(...) Z_LOG(LOG_LEVEL_ERR, __VA_ARGS__) -
Run compliance checks on patch series (PR): GitDiffCheck.txt#L1
Git conflict markers and whitespace errors are not allowed in added changes 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:2: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:5: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:10: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:11: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:12: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:13: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:14: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:15: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:16: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:17: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:18: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:19: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:20: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:21: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:22: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:23: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:24: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:25: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:26: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:27: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:28: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:29: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:30: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:33: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:34: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:35: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:36: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:37: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:42: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:43: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_Bluetooth_Host.bqw:44: trailing whitespace. 8448c206a3c691f94e10f4645939f112534cad8f: tests/bluetooth/qualification/ICS_Zephyr_
Run compliance checks on patch series (PR): Identity.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details f1fac54640fcb02cb5d9d293c3b60dc14b5215eb: author email (Andrzej Kuros <[email protected]>) needs to match one of the signed-off-by entries. 17d7594ad2c2cedc517aa802a14139317da09dcc: author email (Andrzej Kuros <[email protected]>) needs to match one of the signed-off-by entries. 04692906f2c3e89a60ef54a574e992e090aca46a: author email (F. Ramu <[email protected]>) needs to match one of the signed-off-by entries. 43d51dcd933a32fdad161fc02597e0753b49a824: author email (Francois RAMU <[email protected]>) needs to match one of the signed-off-by entries. 159809e8608b2cbc3f21ee82da6af96453097ed3: author email (Stine Akredalen <[email protected]>) needs to match one of the signed-off-by entries. 9211fff800de18507c2115b334df08bd4d19581f: author email (Francois RAMU <[email protected]>) needs to match one of the signed-off-by entries. 0dcc84670b474772e280dded52d0e12a437932e5: author email (Stine Akredalen <[email protected]>) needs to match one of the signed-off-by entries. 75542b0cfc3d9df375474ffa9536cabf86c896b5: author email (Andrzej Kuros <[email protected]>) needs to match one of the signed-off-by entries. dcbc48e80d212f273c084b5ac5ef9c0218f0a6c6: author email (Andrzej Kuros <[email protected]>) needs to match one of the signed-off-by entries. 4a688783200072b318b1e9dec81a4a14aacfd1ae: author email (Andrzej Kuros <[email protected]>) needs to match one of the signed-off-by entries. 044be72ed42b191a39a1ac1515823a429ca0edea: author email (Mateusz Holenko <[email protected]>) needs to match one of the signed-off-by entries. 4092088ac3349de304a1f3c7c28bb26fbe9989b2: author email (Mateusz Holenko <[email protected]>) needs to match one of the signed-off-by entries. 635a0ebbdf109b25d73396890218a9ba9275bfb1: author email (Andrzej Kuros <[email protected]>) needs to match one of the signed-off-by entries.
Run compliance checks on patch series (PR): ImageSize.txt#L1
Check the size of image files. Image file too large: boards/intel/socfpga_std/cyclonev_socdk/doc/img/cyclonev_socdk.jpg reduce size to less than 100kB
Run compliance checks on patch series (PR): Pylint.txt#L1
See https://www.pylint.org/ for more details E0611:No name 'TEST_DATA' in module 'conftest' (no-name-in-module) File:doc/develop/test/twister/sample_blackbox_test.py Line:13 Column:0 E0611:No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module) File:doc/develop/test/twister/sample_blackbox_test.py Line:13 Column:0 E0611:No name 'testsuite_filename_mock' in module 'conftest' (no-name-in-module) File:doc/develop/test/twister/sample_blackbox_test.py Line:13 Column:0 E0611:No name 'other_constants' in module 'constants' (no-name-in-module) File:scripts/generate_usb_vif/constants/xml_constants.py Line:8 Column:0 E0611:No name 'vif_element_constants' in module 'constants' (no-name-in-module) File:scripts/generate_usb_vif/constants/xml_constants.py Line:8 Column:0 W0611:Unused natsorted imported from natsort (unused-import) File:scripts/pylib/twister/twisterlib/hardwaremap.py Line:16 Column:0 E0611:No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module) File:scripts/tests/twister/test_harness.py Line:17 Column:0 E1120:No value for argument 'target' in method call (no-value-for-parameter) File:scripts/tests/twister/test_platform.py Line:118 Column:8 E1120:No value for argument 'aliases' in method call (no-value-for-parameter) File:scripts/tests/twister/test_platform.py Line:118 Column:8 E1120:No value for argument 'data' in method call (no-value-for-parameter) File:scripts/tests/twister/test_platform.py Line:118 Column:8 C0117:Consider changing "not testinstance < testinstance_copy" to "testinstance >= testinstance_copy" (unnecessary-negation) File:scripts/tests/twister/test_testinstance.py Line:359 Column:11 C0117:Consider changing "not testinstance_copy < testinstance" to "testinstance_copy >= testinstance" (unnecessary-negation) File:scripts/tests/twister/test_testinstance.py Line:360 Column:11 C0201:Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary) File:scripts/tests/twister/test_testplan.py Line:848 Column:37 E0611:No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_addon.py Line:19 Column:0 E0611:No name 'TEST_DATA' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_addon.py Line:19 Column:0 E0611:No name 'sample_filename_mock' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_addon.py Line:19 Column:0 E0611:No name 'testsuite_filename_mock' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_addon.py Line:19 Column:0 E0611:No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_disable.py Line:16 Column:0 E0611:No name 'TEST_DATA' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_disable.py Line:16 Column:0 E0611:No name 'testsuite_filename_mock' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_disable.py Line:16 Column:0 E0611:No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_quarantine.py Line:18 Column:0 E0611:No name 'TEST_DATA' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_quarantine.py Line:18 Column:0 E0611:No name 'testsuite_filename_mock' in module 'conftest' (no-name-in-module) File:scripts/tests/twister_blackbox/test_quarantine.py Line:18 Column:0 I0021:Useless suppression of 'assignment-from-no-return' (useless-suppression) File:scripts/utils/board_v1_to_v2.py Line:79 Column:0 E0611:No name 'RC_KERNEL_BIN' in module 'conftest' (no-name-in-module) File:scripts/west_commands/tests/test_bossac.py Line:15 Column:0 E0611:No name 'RC_KERNEL_HEX' in module 'conftest' (no-name-in-module) File:scripts/west_commands/tests/test_stm32cubeprogrammer.py Line:12 Column:0 E0611:No name 'RC_KERNEL_ELF' in module 'conftest' (no-name-in-module) File:scripts/west_commands/tests/test_stm32cubeprogrammer.py Line:12 Column:0 E0611:No name 'RC_KERNEL_ELF' in module 'conftest' (no-name-in-module) Fil
Run compliance checks on patch series (PR): Ruff.txt#L1
Check python files with ruff. Python lint error (F401) see https://docs.astral.sh/ruff/rules/unused-import:`natsort.natsorted` imported but unused File:scripts/pylib/twister/twisterlib/hardwaremap.py Line:16 Column:21 EndLine:16 EndColumn:30 Python lint error (I001) see https://docs.astral.sh/ruff/rules/unsorted-imports:Import block is un-sorted or un-formatted File:tests/lib/devicetree/memory_region_flags/pytest/test_memory_region_flags.py Line:4 Column:1 EndLine:10 EndColumn:1 Python lint error (E402) see https://docs.astral.sh/ruff/rules/module-import-not-at-top-of-file:Module level import not at top of file File:tests/lib/devicetree/memory_region_flags/pytest/test_memory_region_flags.py Line:14 Column:1 EndLine:14 EndColumn:30 Python lint error (E501) see https://docs.astral.sh/ruff/rules/line-too-long:Line too long (104 > 100) File:tests/lib/devicetree/memory_region_flags/pytest/test_memory_region_flags.py Line:67 Column:101 EndLine:67 EndColumn:105 Python format error:Run 'ruff format tests/lib/devicetree/memory_region_flags/pytest/test_memory_region_flags.py' File:tests/lib/devicetree/memory_region_flags/pytest/test_memory_region_flags.py
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 1.
E0611: doc/develop/test/twister/sample_blackbox_test.py#L13
doc/develop/test/twister/sample_blackbox_test.py:13 No name 'TEST_DATA' in module 'conftest' (no-name-in-module)
E0611: doc/develop/test/twister/sample_blackbox_test.py#L13
doc/develop/test/twister/sample_blackbox_test.py:13 No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module)
E0611: doc/develop/test/twister/sample_blackbox_test.py#L13
doc/develop/test/twister/sample_blackbox_test.py:13 No name 'testsuite_filename_mock' in module 'conftest' (no-name-in-module)
E0611: scripts/generate_usb_vif/constants/xml_constants.py#L8
scripts/generate_usb_vif/constants/xml_constants.py:8 No name 'other_constants' in module 'constants' (no-name-in-module)
E0611: scripts/generate_usb_vif/constants/xml_constants.py#L8
scripts/generate_usb_vif/constants/xml_constants.py:8 No name 'vif_element_constants' in module 'constants' (no-name-in-module)
E0611: scripts/tests/twister/test_harness.py#L17
scripts/tests/twister/test_harness.py:17 No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module)
E1120: scripts/tests/twister/test_platform.py#L118
scripts/tests/twister/test_platform.py:118 No value for argument 'target' in method call (no-value-for-parameter)
E1120: scripts/tests/twister/test_platform.py#L118
scripts/tests/twister/test_platform.py:118 No value for argument 'aliases' in method call (no-value-for-parameter)
E1120: scripts/tests/twister/test_platform.py#L118
scripts/tests/twister/test_platform.py:118 No value for argument 'data' in method call (no-value-for-parameter)
E0611: scripts/tests/twister_blackbox/test_addon.py#L19
scripts/tests/twister_blackbox/test_addon.py:19 No name 'ZEPHYR_BASE' in module 'conftest' (no-name-in-module)
W0611: scripts/pylib/twister/twisterlib/hardwaremap.py#L16
scripts/pylib/twister/twisterlib/hardwaremap.py:16 Unused natsorted imported from natsort (unused-import)
C0117: scripts/tests/twister/test_testinstance.py#L359
scripts/tests/twister/test_testinstance.py:359 Consider changing "not testinstance < testinstance_copy" to "testinstance >= testinstance_copy" (unnecessary-negation)
C0117: scripts/tests/twister/test_testinstance.py#L360
scripts/tests/twister/test_testinstance.py:360 Consider changing "not testinstance_copy < testinstance" to "testinstance_copy >= testinstance" (unnecessary-negation)
C0201: scripts/tests/twister/test_testplan.py#L848
scripts/tests/twister/test_testplan.py:848 Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
I0021: scripts/utils/board_v1_to_v2.py#L79
scripts/utils/board_v1_to_v2.py:79 Useless suppression of 'assignment-from-no-return' (useless-suppression)

Artifacts

Produced during runtime
Name Size
compliance.xml
72.2 KB