LLEXT full ARM relocation support (adding executable) #123963
compliance.yml
on: pull_request
Run compliance checks on patch series (PR)
2m 43s
Annotations
15 errors and 10 warnings
POINTER_LOCATION:
include/zephyr/llext/symbol.h#L82
include/zephyr/llext/symbol.h:82 "(foo*)" should be "(foo *)"
|
TRAILING_WHITESPACE:
samples/subsys/llext/fs_loader/makeimg.sh#L30
samples/subsys/llext/fs_loader/makeimg.sh:30 trailing whitespace
|
SPACING:
samples/subsys/llext/fs_loader/src/main.c#L189
samples/subsys/llext/fs_loader/src/main.c:189 space required after that ',' (ctx:VxV)
|
SPACING:
samples/subsys/llext/fs_loader/src/main.c#L190
samples/subsys/llext/fs_loader/src/main.c:190 space required after that ',' (ctx:VxV)
|
SPACING:
samples/subsys/llext/fs_loader/src/main.c#L192
samples/subsys/llext/fs_loader/src/main.c:192 spaces required around that '=' (ctx:VxV)
|
SPACING:
samples/subsys/llext/fs_loader/src/main.c#L192
samples/subsys/llext/fs_loader/src/main.c:192 spaces required around that '<' (ctx:VxV)
|
POINTER_LOCATION:
samples/subsys/llext/fs_loader/src/main.c#L193
samples/subsys/llext/fs_loader/src/main.c:193 "foo * bar" should be "foo *bar"
|
POINTER_LOCATION:
samples/subsys/llext/fs_loader/src/main.c#L194
samples/subsys/llext/fs_loader/src/main.c:194 "foo * bar" should be "foo *bar"
|
SPACING:
samples/subsys/llext/fs_loader/src/main.c#L199
samples/subsys/llext/fs_loader/src/main.c:199 space required before the open parenthesis '('
|
C99_COMMENTS:
samples/subsys/llext/fs_loader/src/main.c#L214
samples/subsys/llext/fs_loader/src/main.c:214 do not use C99 // comments
|
Run compliance checks on patch series (PR):
Kconfig.txt#L1
See https://docs.zephyrproject.org/latest/build/kconfig/tips.html for more details.
Found references to undefined Kconfig symbols. If any of these are false
positives, then add them to UNDEF_KCONFIG_WHITELIST in /home/runner/work/zephyr/zephyr/./scripts/ci/check_compliance.py.
If the reference is for a comment like /* CONFIG_FOO_* */ (or
/* CONFIG_FOO_*_... */), then please use exactly that form (with the '*'). The
CI check knows not to flag it.
More generally, a reference followed by $, @, {, *, or ## will never be
flagged.
CONFIG_LLEXT_ARM_V4BX arch/arm/core/elf.c:259
CONFIG_LLEXT_PIC samples/subsys/llext/fs_loader/Kconfig:12
|
Run compliance checks on patch series (PR):
GitDiffCheck.txt#L1
Git conflict markers and whitespace errors are not allowed in added changes
9d201ab51c26c35678e13c37d51555021626c2c1: samples/subsys/llext/fs_loader/makeimg.sh:30: trailing whitespace.
be558941de14a8aac3e74783076c771bbdb1e8d8: arch/arm/core/llext.ld:46: trailing whitespace.
be558941de14a8aac3e74783076c771bbdb1e8d8: arch/arm/core/llext.ld:100: trailing whitespace.
be558941de14a8aac3e74783076c771bbdb1e8d8: arch/arm/core/llext.ld:105: trailing whitespace.
be558941de14a8aac3e74783076c771bbdb1e8d8: arch/arm/core/llext.ld:110: new blank line at EOF.
|
Run compliance checks on patch series (PR):
Checkpatch.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style for more details.
TYPO_SPELLING: 'funtions' may be misspelled - perhaps 'functions'?
File:include/zephyr/linker/llext.ld
Line:63
TYPO_SPELLING: 'usefull' may be misspelled - perhaps 'useful'?
File:include/zephyr/linker/llext.ld
Line:113
POINTER_LOCATION: "(foo*)" should be "(foo *)"
File:include/zephyr/llext/symbol.h
Line:82
TYPO_SPELLING: 'Independant' may be misspelled - perhaps 'Independent'?
File:samples/subsys/llext/fs_loader/Kconfig
Line:15
LINE_SPACING: Missing a blank line after declarations
File:samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c
Line:26
LINE_SPACING: Missing a blank line after declarations
File:samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c
Line:29
LONG_LINE: line length of 111 exceeds 100 columns
File:samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c
Line:52
TRAILING_WHITESPACE: trailing whitespace
File:samples/subsys/llext/fs_loader/makeimg.sh
Line:30
SPACING: space required after that ',' (ctx:VxV)
File:samples/subsys/llext/fs_loader/src/main.c
Line:189
SPACING: space required after that ',' (ctx:VxV)
File:samples/subsys/llext/fs_loader/src/main.c
Line:190
SPACING: spaces required around that '=' (ctx:VxV)
File:samples/subsys/llext/fs_loader/src/main.c
Line:192
SPACING: spaces required around that '<' (ctx:VxV)
File:samples/subsys/llext/fs_loader/src/main.c
Line:192
POINTER_LOCATION: "foo * bar" should be "foo *bar"
File:samples/subsys/llext/fs_loader/src/main.c
Line:193
POINTER_LOCATION: "foo * bar" should be "foo *bar"
File:samples/subsys/llext/fs_loader/src/main.c
Line:194
LINE_SPACING: Missing a blank line after declarations
File:samples/subsys/llext/fs_loader/src/main.c
Line:195
LINE_SPACING: Missing a blank line after declarations
File:samples/subsys/llext/fs_loader/src/main.c
Line:199
SPACING: space required before the open parenthesis '('
File:samples/subsys/llext/fs_loader/src/main.c
Line:199
LONG_LINE: line length of 114 exceeds 100 columns
File:samples/subsys/llext/fs_loader/src/main.c
Line:213
C99_COMMENTS: do not use C99 // comments
File:samples/subsys/llext/fs_loader/src/main.c
Line:214
LINE_SPACING: Missing a blank line after declarations
File:samples/subsys/llext/fs_loader/src/main.c
Line:217
OPEN_BRACE: that open brace { should be on the previous line
File:samples/subsys/llext/fs_loader/src/main.c
Line:217
SPACING: space required before the open parenthesis '('
File:samples/subsys/llext/fs_loader/src/main.c
Line:217
LONG_LINE: line length of 108 exceeds 100 columns
File:samples/subsys/llext/fs_loader/src/main.c
Line:222
LEADING_SPACE: please, no spaces at the start of a line
File:subsys/llext/file_loader.c
Line:18
LEADING_SPACE: please, no spaces at the start of a line
File:subsys/llext/file_loader.c
Line:20
SPACING: space required before the open parenthesis '('
File:subsys/llext/file_loader.c
Line:20
CODE_INDENT: code indent should use tabs where possible
File:subsys/llext/file_loader.c
Line:21
LEADING_SPACE: please, no spaces at the start of a line
File:subsys/llext/file_loader.c
Line:21
LEADING_SPACE: please, no spaces at the start of a line
File:subsys/llext/file_loader.c
Line:22
POINTER_LOCATION: "(foo*)" should be "(foo *)"
File:subsys/llext/llext.c
Line:437
SPACING: space required before the open parenthesis '('
File:subsys/llext/llext.c
Line:446
TYPO_SPELLING: 'aligment' may be misspelled - perhaps 'alignment'?
File:subsys/llext/llext.c
Line:447
LONG_LINE: line length of 108 exceeds 100 columns
File:subsys/llext/llext.c
Line:447
SPACING: space prohibited before that close parenthesis ')'
File:subsys/llext/llext.c
Line:466
LONG_LINE: line length of 107 exceeds 100 columns
File:subsys/llext/llext.c
Line:529
SPACING: space required before the open parenthesis '('
File:subsys/llext/llext.c
Line:540
TYPO_SPELLING: 'aligment' may be misspelled - perhaps 'alignment'?
File:subsys/llext/llext.c
Line:541
LONG_LINE: line length of 108 exceeds 100 columns
File:subsys/llext/llext.c
Line:545
SPACING: space prohibited aft
|
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 'include/zephyr/linker/llext.ld'
Missing newline at end of 'samples/subsys/llext/fs_loader/boards/nucleo_l496zg.overlay'. Check your text editor settings.
Missing newline at end of 'samples/subsys/llext/fs_loader/boards/stm32f4_disco.overlay'. Check your text editor settings.
|
Run compliance checks on patch series (PR)
Process completed with exit code 1.
|
TYPO_SPELLING:
include/zephyr/linker/llext.ld#L63
include/zephyr/linker/llext.ld:63 'funtions' may be misspelled - perhaps 'functions'?
|
TYPO_SPELLING:
include/zephyr/linker/llext.ld#L113
include/zephyr/linker/llext.ld:113 'usefull' may be misspelled - perhaps 'useful'?
|
TYPO_SPELLING:
samples/subsys/llext/fs_loader/Kconfig#L15
samples/subsys/llext/fs_loader/Kconfig:15 'Independant' may be misspelled - perhaps 'Independent'?
|
LINE_SPACING:
samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c#L26
samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c:26 Missing a blank line after declarations
|
LINE_SPACING:
samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c#L29
samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c:29 Missing a blank line after declarations
|
LONG_LINE:
samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c#L52
samples/subsys/llext/fs_loader/hello_world.llext/hello_world.c:52 line length of 111 exceeds 100 columns
|
LINE_SPACING:
samples/subsys/llext/fs_loader/src/main.c#L195
samples/subsys/llext/fs_loader/src/main.c:195 Missing a blank line after declarations
|
LINE_SPACING:
samples/subsys/llext/fs_loader/src/main.c#L199
samples/subsys/llext/fs_loader/src/main.c:199 Missing a blank line after declarations
|
LONG_LINE:
samples/subsys/llext/fs_loader/src/main.c#L213
samples/subsys/llext/fs_loader/src/main.c:213 line length of 114 exceeds 100 columns
|
LINE_SPACING:
samples/subsys/llext/fs_loader/src/main.c#L217
samples/subsys/llext/fs_loader/src/main.c:217 Missing a blank line after declarations
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
compliance.xml
Expired
|
2.41 KB |
|