Skip to content

Mspm0 dev

Mspm0 dev #9

Triggered via pull request March 13, 2024 18:49
@msp-timsp-ti
synchronize #4
mspm0_dev
Status Failure
Total duration 2m 22s
Artifacts 1

compliance.yml

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

Annotations

17 errors and 10 warnings
TRAILING_WHITESPACE: boards/arm/lp_mspm0g3507/board.cmake#L4
boards/arm/lp_mspm0g3507/board.cmake:4 trailing whitespace
TRAILING_WHITESPACE: boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts#L121
boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts:121 trailing whitespace
TRAILING_WHITESPACE: boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts#L125
boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts:125 trailing whitespace
TRAILING_WHITESPACE: boards/arm/lp_mspm0l2228/lp_mspm0l2228.dts#L100
boards/arm/lp_mspm0l2228/lp_mspm0l2228.dts:100 trailing whitespace
SPACING: drivers/i2c/i2c_mspm0.c#L439
drivers/i2c/i2c_mspm0.c:439 space required before the open brace '{'
SPACING: drivers/i2c/i2c_mspm0.c#L439
drivers/i2c/i2c_mspm0.c:439 space required before the open parenthesis '('
TRAILING_WHITESPACE: drivers/serial/Kconfig.mspm0#L11
drivers/serial/Kconfig.mspm0:11 trailing whitespace
C99_COMMENTS: drivers/watchdog/wdt_mspm0.c#L15
drivers/watchdog/wdt_mspm0.c:15 do not use C99 // comments
C99_COMMENTS: drivers/watchdog/wdt_mspm0.c#L16
drivers/watchdog/wdt_mspm0.c:16 do not use C99 // comments
C99_COMMENTS: drivers/watchdog/wdt_mspm0.c#L17
drivers/watchdog/wdt_mspm0.c:17 do not use C99 // comments
Run compliance checks on patch series (PR): Identity.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details f4b79948e7da152bf8f70ff6d218dea15b1d32fd: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. 8baf4e5c5e4d8cc002073ca6e32453da01cc10a4: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. 8baf4e5c5e4d8cc002073ca6e32453da01cc10a4: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. 002800c11ace08813ec880b7879adee57b1a8e69: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. 002800c11ace08813ec880b7879adee57b1a8e69: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. e847c3d1ad0e01425a7cd1e100e445d68e0fd0ca: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. e847c3d1ad0e01425a7cd1e100e445d68e0fd0ca: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. 9638189a1835fce2b27409dcd7bb01dd3c0cc9cc: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. 9638189a1835fce2b27409dcd7bb01dd3c0cc9cc: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. b2c9077e1b1fd5766748fbabe9b83a5c138a60c2: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. b2c9077e1b1fd5766748fbabe9b83a5c138a60c2: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>.
Run compliance checks on patch series (PR): Checkpatch.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style for more details. TRAILING_WHITESPACE: trailing whitespace File:boards/arm/lp_mspm0g3507/board.cmake Line:4 TRAILING_WHITESPACE: trailing whitespace File:boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts Line:121 TRAILING_WHITESPACE: trailing whitespace File:boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts Line:125 TRAILING_WHITESPACE: trailing whitespace File:boards/arm/lp_mspm0l2228/lp_mspm0l2228.dts Line:100 SPACING: space required before the open brace '{' File:drivers/i2c/i2c_mspm0.c Line:439 SPACING: space required before the open parenthesis '(' File:drivers/i2c/i2c_mspm0.c Line:439 LONG_LINE: line length of 150 exceeds 100 columns File:drivers/i2c/i2c_mspm0.c Line:440 LONG_LINE: line length of 151 exceeds 100 columns File:drivers/i2c/i2c_mspm0.c Line:442 LONG_LINE: line length of 143 exceeds 100 columns File:drivers/i2c/i2c_mspm0.c Line:448 TRAILING_WHITESPACE: trailing whitespace File:drivers/serial/Kconfig.mspm0 Line:11 C99_COMMENTS: do not use C99 // comments File:drivers/watchdog/wdt_mspm0.c Line:15 C99_COMMENTS: do not use C99 // comments File:drivers/watchdog/wdt_mspm0.c Line:16 C99_COMMENTS: do not use C99 // comments File:drivers/watchdog/wdt_mspm0.c Line:17 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:41 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:42 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:46 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:47 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:48 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:52 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:53 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:54 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:55 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:56 LONG_LINE: line length of 109 exceeds 100 columns File:drivers/watchdog/wdt_mspm0.c Line:59 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/watchdog/wdt_mspm0.c Line:59 SPACING: space required before the open brace '{' File:drivers/watchdog/wdt_mspm0.c Line:59 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:60 POINTER_LOCATION: "foo * bar" should be "foo *bar" File:drivers/watchdog/wdt_mspm0.c Line:60 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:78 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:79 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:80 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:81 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:83 SPACING: space required before the open parenthesis '(' File:drivers/watchdog/wdt_mspm0.c Line:83 CODE_INDENT: code indent should use tabs where possible File:drivers/watchdog/wdt_mspm0.c Line:84 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:84 CODE_INDENT: code indent should use tabs where possible File:drivers/watchdog/wdt_mspm0.c Line:85 CODE_INDENT: code indent should use tabs where possible File:drivers/watchdog/wdt_mspm0.c Line:86 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:86 LEADING_SPACE: please, no spaces at the start of a line File:drivers/watchdog/wdt_mspm0.c Line:87 SPACING: space required before the open brace '{' File:drivers/watchdog/wdt_mspm0.c Line:87 CODE_INDENT: code indent should use tabs where p
Run compliance checks on patch series (PR): Gitlint.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details Commit f4b79948e7: 1: UC2 Commit message does not contain a 'Signed-off-by:' line 1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "first draft of watchdog instance" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit 8baf4e5c5e: 1: UC2 Commit message does not contain a 'Signed-off-by:' line 1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "openocd changes" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit 002800c11a: 1: UC2 Commit message does not contain a 'Signed-off-by:' line 1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "added openOCD support, removed unnecessary files" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit e847c3d1ad: 1: UC2 Commit message does not contain a 'Signed-off-by:' line 1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "quality of life improvements and updates to supported pins" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit 9638189a18: 1: UC2 Commit message does not contain a 'Signed-off-by:' line 1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "temporary work on the target api for i2c" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit b2c9077e1b: 1: UC2 Commit message does not contain a 'Signed-off-by:' line 1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "changes to i2c to use ACK override" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing
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 'drivers/watchdog/Kconfig.mspm0' Missing newline at end of 'drivers/watchdog/wdt_mspm0.c'. Check your text editor settings. Missing newline at end of 'dts/bindings/watchdog/ti,mspm0-watchdog.yaml'. Check your text editor settings. Missing newline at end of 'samples/drivers/watchdog/boards/lp_mspm0g3507.overlay'. Check your text editor settings. Missing newline at end of 'tests/drivers/i2c/i2c_target_api2/boards/lp_mspm0g3507.overlay'. Check your text editor settings. Missing newline at end of 'tests/drivers/i2c/i2c_target_api2/testcase.yaml'. Check your text editor settings.
Run compliance checks on patch series (PR): GitDiffCheck.txt#L1
Git conflict markers and whitespace errors are not allowed in added changes f4b79948e7da152bf8f70ff6d218dea15b1d32fd: boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts:121: trailing whitespace. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: boards/arm/lp_mspm0g3507/lp_mspm0g3507.dts:125: trailing whitespace. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: boards/arm/lp_mspm0l2228/lp_mspm0l2228.dts:100: trailing whitespace. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: drivers/watchdog/Kconfig.mspm0:9: new blank line at EOF. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: dts/bindings/watchdog/ti,mspm0-watchdog.yaml:11: trailing whitespace. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: dts/bindings/watchdog/ti,mspm0-watchdog.yaml:18: trailing whitespace. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: dts/bindings/watchdog/ti,mspm0-watchdog.yaml:18: new blank line at EOF. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: samples/drivers/watchdog/boards/lp_mspm0g3507.overlay:1: trailing whitespace. f4b79948e7da152bf8f70ff6d218dea15b1d32fd: samples/drivers/watchdog/boards/lp_mspm0g3507.overlay:15: trailing whitespace. 002800c11ace08813ec880b7879adee57b1a8e69: boards/arm/lp_mspm0g3507/board.cmake:4: trailing whitespace. e847c3d1ad0e01425a7cd1e100e445d68e0fd0ca: drivers/serial/Kconfig.mspm0:11: trailing whitespace. 9638189a1835fce2b27409dcd7bb01dd3c0cc9cc: tests/drivers/i2c/i2c_target_api2/testcase.yaml:1: trailing whitespace. 9638189a1835fce2b27409dcd7bb01dd3c0cc9cc: tests/drivers/i2c/i2c_target_api2/testcase.yaml:10: trailing whitespace.
Run compliance checks on patch series (PR): YAMLLint.txt#L1
Check YAML files with YAMLLint. YAMLLint (trailing-spaces):trailing spaces File:dts/bindings/watchdog/ti,mspm0-watchdog.yaml Line:11 Column:7 YAMLLint (line-length):line too long (104 > 100 characters) File:dts/bindings/watchdog/ti,mspm0-watchdog.yaml Line:17 Column:101 YAMLLint (new-line-at-end-of-file):no new line character at the end of file File:dts/bindings/watchdog/ti,mspm0-watchdog.yaml Line:18 Column:5 YAMLLint (trailing-spaces):trailing spaces File:dts/bindings/watchdog/ti,mspm0-watchdog.yaml Line:18 Column:1 YAMLLint (trailing-spaces):trailing spaces File:tests/drivers/i2c/i2c_target_api2/testcase.yaml Line:1 Column:8 YAMLLint (indentation):wrong indentation: expected 2 but found 4 File:tests/drivers/i2c/i2c_target_api2/testcase.yaml Line:2 Column:5 YAMLLint (trailing-spaces):trailing spaces File:tests/drivers/i2c/i2c_target_api2/testcase.yaml Line:10 Column:20 YAMLLint (new-line-at-end-of-file):no new line character at the end of file File:tests/drivers/i2c/i2c_target_api2/testcase.yaml Line:11 Column:22
Run compliance checks on patch series (PR)
Process completed with exit code 1.
YAMLLint (trailing-spaces): dts/bindings/watchdog/ti#L11
dts/bindings/watchdog/ti,mspm0-watchdog.yaml:11 trailing spaces
YAMLLint (line-length): dts/bindings/watchdog/ti#L17
dts/bindings/watchdog/ti,mspm0-watchdog.yaml:17 line too long (104 > 100 characters)
YAMLLint (new-line-at-end-of-file): dts/bindings/watchdog/ti#L18
dts/bindings/watchdog/ti,mspm0-watchdog.yaml:18 no new line character at the end of file
YAMLLint (trailing-spaces): dts/bindings/watchdog/ti#L18
dts/bindings/watchdog/ti,mspm0-watchdog.yaml:18 trailing spaces
YAMLLint (trailing-spaces): tests/drivers/i2c/i2c_target_api2/testcase.yaml#L1
tests/drivers/i2c/i2c_target_api2/testcase.yaml:1 trailing spaces
YAMLLint (indentation): tests/drivers/i2c/i2c_target_api2/testcase.yaml#L2
tests/drivers/i2c/i2c_target_api2/testcase.yaml:2 wrong indentation: expected 2 but found 4
YAMLLint (trailing-spaces): tests/drivers/i2c/i2c_target_api2/testcase.yaml#L10
tests/drivers/i2c/i2c_target_api2/testcase.yaml:10 trailing spaces
YAMLLint (new-line-at-end-of-file): tests/drivers/i2c/i2c_target_api2/testcase.yaml#L11
tests/drivers/i2c/i2c_target_api2/testcase.yaml:11 no new line character at the end of file
LONG_LINE: drivers/i2c/i2c_mspm0.c#L440
drivers/i2c/i2c_mspm0.c:440 line length of 150 exceeds 100 columns
LONG_LINE: drivers/i2c/i2c_mspm0.c#L442
drivers/i2c/i2c_mspm0.c:442 line length of 151 exceeds 100 columns

Artifacts

Produced during runtime
Name Size
compliance.xml Expired
4.74 KB