Skip to content

cmake: platforms: xlnx: Update machine from 'zynqmp_' to 'xlnx_' #348

cmake: platforms: xlnx: Update machine from 'zynqmp_' to 'xlnx_'

cmake: platforms: xlnx: Update machine from 'zynqmp_' to 'xlnx_' #348

Triggered via pull request November 19, 2024 15:15
Status Failure
Total duration 57s
Artifacts 1

compliance.yml

on: pull_request
compliance review
46s
compliance review
Fit to window
Zoom out
Zoom in

Annotations

14 errors and 11 warnings
compliance review: Checkpatch.txt#L1
TRAILING_WHITESPACE: trailing whitespace File:examples/system/freertos/xlnx_r5/amp_demo/common.h Line:196 LEADING_SPACE: please, no spaces at the start of a line File:examples/system/freertos/xlnx_r5/amp_demo/common.h Line:196 CAMELCASE: Avoid CamelCase: <XScuGic> File:examples/system/freertos/xlnx_r5/amp_demo/sys_init.c Line:41 CAMELCASE: Avoid CamelCase: <xInterruptController> File:examples/system/freertos/xlnx_r5/amp_demo/sys_init.c Line:41 TRAILING_WHITESPACE: trailing whitespace File:examples/system/generic/xlnx_r5/amp_demo/common.h Line:207 LEADING_SPACE: please, no spaces at the start of a line File:examples/system/generic/xlnx_r5/amp_demo/common.h Line:207 LINE_SPACING: Please don't use multiple blank lines File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:75 BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:89 BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:89 LEADING_SPACE: please, no spaces at the start of a line File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:96 FUNCTION_WITHOUT_ARGS: Bad function definition - int shmem_demo() should probably be int shmem_demo(void) File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:114 FUNCTION_WITHOUT_ARGS: Bad function definition - int ipi_shmem_demo() should probably be int ipi_shmem_demo(void) File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:131 FUNCTION_WITHOUT_ARGS: Bad function definition - int atomic_shmem_demo() should probably be int atomic_shmem_demo(void) File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:151 FUNCTION_WITHOUT_ARGS: Bad function definition - int ipi_latency_demo() should probably be int ipi_latency_demo(void) File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:168 FUNCTION_WITHOUT_ARGS: Bad function definition - int shmem_latency_demo() should probably be int shmem_latency_demo(void) File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:180 FUNCTION_WITHOUT_ARGS: Bad function definition - int shmem_throughput_demo() should probably be int shmem_throughput_demo(void) File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:190 SPACING: space required before the open parenthesis '(' File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:210 LINE_SPACING: Please use a blank line after function/struct/union/enum declarations File:examples/system/linux/zynqmp/zynqmp_amp_demo/common.h Line:286 TYPO_SPELLING: 'follwing' may be misspelled - perhaps 'following'? File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c Line:10 CONCATENATED_STRING: Concatenated strings should use spaces between elements File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c Line:46 CONCATENATED_STRING: Concatenated strings should use spaces between elements File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c Line:47 SPACING: space prohibited between function name and open parenthesis '(' File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c Line:74 SPACING: space required before the open parenthesis '(' File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c Line:84 SPACING: space required after that ',' (ctx:VxV) File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c Line:187 TYPO_SPELLING: 'follwing' may be misspelled - perhaps 'following'? File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-uio.c Line:10 COMPARISON_TO_NULL: Comparison to NULL could be written "chnl->ipi_kick_cb" File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-uio.c Line:74 PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-uio.c Line:77 LINE_SPACING: Please use a blank line after function/struct/union/enum declarations File:examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-uio.c Line:102 LINE_SPACING: Please use a blank line after function/struct/union/enum declarations File:examples/system/linux/zynqmp/zynqmp_
compliance review: Identity.txt#L1
7ae1fe931f349f575fd70e11bcc07002d54def5e: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 7ae1fe931f349f575fd70e11bcc07002d54def5e: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 8eca38ff52318310020472429e381cc636b429d4: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 8eca38ff52318310020472429e381cc636b429d4: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 3c27d8539bd07a9a2713483129b90c59b418ca6d: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 3c27d8539bd07a9a2713483129b90c59b418ca6d: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. d381e628c2f2cf37e13faa65db33a5d4702acf60: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. d381e628c2f2cf37e13faa65db33a5d4702acf60: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 22e8f5fde5bb9a17653e638f69ba93d07ecb2788: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 22e8f5fde5bb9a17653e638f69ba93d07ecb2788: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 5805b7fda9dd3b79e6f370e2c3b8ecb5674cf64f: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 5805b7fda9dd3b79e6f370e2c3b8ecb5674cf64f: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 68f4ba0bfa24438c5349cd08ae43f970910f1d51: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 68f4ba0bfa24438c5349cd08ae43f970910f1d51: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 3ef1b92a8d3c352def2262709dfca2b8ee668cd8: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 3ef1b92a8d3c352def2262709dfca2b8ee668cd8: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. 2ac9c17d4c378e2dc2022c40f3395ee3f1157042: author email (Levinsky, Ben <[email protected]>) needs to match one of the signed-off-by entries. 2ac9c17d4c378e2dc2022c40f3395ee3f1157042: author email (Levinsky, Ben <[email protected]>) does not follow the syntax: First Last <email>. c7edeb09ccd357ba072ec5bda3c4fc7a71c72efc: author email (Sergei Korneichuk <[email protected]>) needs to match one of the signed-off-by entries.
compliance review: Gitlint.txt#L1
Commit 7ae1fe931f: 1: UC5 Title exceeds max length (87>72): "examples: freertos: zynqmp_r5: zynqmp_amp_demo: Add symbols for System Device Flow flow" Commit 3c27d8539b: 1: UC5 Title exceeds max length (83>72): "examples: zynqmp_r5: CMake: Enable demo to build without libxil present in SDT flow" 3: UC4 Line exceeds max length (83>80): "As in SDT Workflow for Rigel there can be template application in BSP for Libmetal," Commit 68f4ba0bfa: 1: UC5 Title exceeds max length (75>72): "lib: generic: xlnx: Use CMAKE_C_FLAGS to determine if SDT symbol is present" Commit 2ac9c17d4c: 1: UC5 Title exceeds max length (74>72): "examples: freertos: define xInterruptController in System Device Tree Flow" Commit c2533b4ec6: 1: UC6 Body has no content, should at least have 1 line.
compliance review
Process completed with exit code 1.
TRAILING_WHITESPACE: examples/system/freertos/xlnx_r5/amp_demo/common.h#L196
examples/system/freertos/xlnx_r5/amp_demo/common.h:196 trailing whitespace
TRAILING_WHITESPACE: examples/system/generic/xlnx_r5/amp_demo/common.h#L207
examples/system/generic/xlnx_r5/amp_demo/common.h:207 trailing whitespace
FUNCTION_WITHOUT_ARGS: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L114
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:114 Bad function definition - int shmem_demo() should probably be int shmem_demo(void)
FUNCTION_WITHOUT_ARGS: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L131
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:131 Bad function definition - int ipi_shmem_demo() should probably be int ipi_shmem_demo(void)
FUNCTION_WITHOUT_ARGS: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L151
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:151 Bad function definition - int atomic_shmem_demo() should probably be int atomic_shmem_demo(void)
FUNCTION_WITHOUT_ARGS: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L168
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:168 Bad function definition - int ipi_latency_demo() should probably be int ipi_latency_demo(void)
FUNCTION_WITHOUT_ARGS: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L180
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:180 Bad function definition - int shmem_latency_demo() should probably be int shmem_latency_demo(void)
FUNCTION_WITHOUT_ARGS: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L190
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:190 Bad function definition - int shmem_throughput_demo() should probably be int shmem_throughput_demo(void)
SPACING: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L210
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:210 space required before the open parenthesis '('
SPACING: examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c#L84
examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c:84 space required before the open parenthesis '('
compliance review
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
LEADING_SPACE: examples/system/freertos/xlnx_r5/amp_demo/common.h#L196
examples/system/freertos/xlnx_r5/amp_demo/common.h:196 please, no spaces at the start of a line
LEADING_SPACE: examples/system/generic/xlnx_r5/amp_demo/common.h#L207
examples/system/generic/xlnx_r5/amp_demo/common.h:207 please, no spaces at the start of a line
BLOCK_COMMENT_STYLE: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L89
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:89 Block comments use * on subsequent lines
BLOCK_COMMENT_STYLE: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L89
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:89 Block comments use a trailing */ on a separate line
LEADING_SPACE: examples/system/linux/zynqmp/zynqmp_amp_demo/common.h#L96
examples/system/linux/zynqmp/zynqmp_amp_demo/common.h:96 please, no spaces at the start of a line
TYPO_SPELLING: examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c#L10
examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c:10 'follwing' may be misspelled - perhaps 'following'?
SPACING: examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c#L74
examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-mb.c:74 space prohibited between function name and open parenthesis '('
TYPO_SPELLING: examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-uio.c#L10
examples/system/linux/zynqmp/zynqmp_amp_demo/ipi-uio.c:10 'follwing' may be misspelled - perhaps 'following'?
BLOCK_COMMENT_STYLE: examples/system/linux/zynqmp/zynqmp_amp_demo/ipi_shmem_demo.c#L72
examples/system/linux/zynqmp/zynqmp_amp_demo/ipi_shmem_demo.c:72 Block comments use * on subsequent lines
BLOCK_COMMENT_STYLE: examples/system/linux/zynqmp/zynqmp_amp_demo/ipi_shmem_demo.c#L72
examples/system/linux/zynqmp/zynqmp_amp_demo/ipi_shmem_demo.c:72 Block comments use a trailing */ on a separate line

Artifacts

Produced during runtime
Name Size
compliance.xml
4.09 KB