Skip to content

Mspm0 dev

Mspm0 dev #11

Triggered via pull request March 14, 2024 19:07
@msp-timsp-ti
synchronize #4
mspm0_dev
Status Failure
Total duration 2m 31s
Artifacts 1

compliance.yml

on: pull_request
Run compliance checks on patch series (PR)
2m 23s
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/serial/uart_mspm0.c#L206
drivers/serial/uart_mspm0.c:206 do not use C99 // comments
C99_COMMENTS: drivers/serial/uart_mspm0.c#L207
drivers/serial/uart_mspm0.c:207 do not use C99 // comments
C99_COMMENTS: drivers/serial/uart_mspm0.c#L208
drivers/serial/uart_mspm0.c:208 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 6651f7fa1d5708ad9037ec5be4bf38c24c6c65ab: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. 6651f7fa1d5708ad9037ec5be4bf38c24c6c65ab: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. bae017657ebbff0c1a2c671691a515f5034a0174: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. bae017657ebbff0c1a2c671691a515f5034a0174: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. b4e942bb72c96866926eed86d7950462e6ec7b25: author email (msp-ti <[email protected]>) needs to match one of the signed-off-by entries. b4e942bb72c96866926eed86d7950462e6ec7b25: author email (msp-ti <[email protected]>) does not follow the syntax: First Last <email>. 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 LONG_LINE: line length of 103 exceeds 100 columns File:drivers/i2c/i2c_mspm0.c Line:402 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 LONG_LINE: line length of 112 exceeds 100 columns File:drivers/serial/uart_mspm0.c Line:122 LONG_LINE: line length of 113 exceeds 100 columns File:drivers/serial/uart_mspm0.c Line:129 C99_COMMENTS: do not use C99 // comments File:drivers/serial/uart_mspm0.c Line:206 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:207 C99_COMMENTS: do not use C99 // comments File:drivers/serial/uart_mspm0.c Line:207 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:208 C99_COMMENTS: do not use C99 // comments File:drivers/serial/uart_mspm0.c Line:208 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:209 C99_COMMENTS: do not use C99 // comments File:drivers/serial/uart_mspm0.c Line:209 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:210 C99_COMMENTS: do not use C99 // comments File:drivers/serial/uart_mspm0.c Line:210 C99_COMMENTS: do not use C99 // comments File:drivers/serial/uart_mspm0.c Line:211 CODE_INDENT: code indent should use tabs where possible File:drivers/serial/uart_mspm0.c Line:214 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:214 LONG_LINE: line length of 103 exceeds 100 columns File:drivers/serial/uart_mspm0.c Line:215 CODE_INDENT: code indent should use tabs where possible File:drivers/serial/uart_mspm0.c Line:254 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:254 LONG_LINE: line length of 104 exceeds 100 columns File:drivers/serial/uart_mspm0.c Line:275 CODE_INDENT: code indent should use tabs where possible File:drivers/serial/uart_mspm0.c Line:275 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:275 LONG_LINE: line length of 108 exceeds 100 columns File:drivers/serial/uart_mspm0.c Line:276 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/serial/uart_mspm0.c Line:276 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
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 6651f7fa1d: 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:): "disabled target Wakeup Feature on I2C, and allow restarts" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit bae017657e: 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:): "minor I2C changes allow for restart. Disable Target Wakeup post timeout" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing Commit b4e942bb72: 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 the uart implementation to support interrupt driven variants more effectively" 1: UC5 Commit title exceeds max length (88>75): "changes to the uart implementation to support interrupt driven variants more effectively" 1: UC6 Commit message body is empty, should at least have 1 line(s). 3: B6 Body message is missing 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 b4e942bb72c96866926eed86d7950462e6ec7b25: drivers/serial/uart_mspm0.c:214: space before tab in indent. b4e942bb72c96866926eed86d7950462e6ec7b25: drivers/serial/uart_mspm0.c:254: space before tab in indent. b4e942bb72c96866926eed86d7950462e6ec7b25: drivers/serial/uart_mspm0.c:275: space before tab in indent. 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#L402
drivers/i2c/i2c_mspm0.c:402 line length of 103 exceeds 100 columns
LONG_LINE: drivers/i2c/i2c_mspm0.c#L440
drivers/i2c/i2c_mspm0.c:440 line length of 150 exceeds 100 columns

Artifacts

Produced during runtime
Name Size
compliance.xml Expired
5.32 KB