Compliance Checks

Adding: IST8310 Driver Support for MIMXRT1062-FMURT6 #13

Triggered via pull request July 24, 2023 08:58

mayankmahajan-nxp

opened #12

ist8310-mimxrt1062-fmurt6

Status Failure

Total duration 4m 31s

Artifacts 1

compliance.yml

on: pull_request

Run compliance checks on patch series (PR)

4m 19s

Annotations

15 errors and 10 warnings

OPEN_BRACE: drivers/mag/ist8310/ist8310.c#L21

drivers/mag/ist8310/ist8310.c:21 open brace '{' following function definitions go on the next line

OPEN_BRACE: drivers/mag/ist8310/ist8310.c#L28

drivers/mag/ist8310/ist8310.c:28 open brace '{' following function definitions go on the next line

OPEN_BRACE: drivers/mag/ist8310/ist8310.c#L35

drivers/mag/ist8310/ist8310.c:35 open brace '{' following function definitions go on the next line

OPEN_BRACE: drivers/mag/ist8310/ist8310.c#L45

drivers/mag/ist8310/ist8310.c:45 open brace '{' following function definitions go on the next line

OPEN_BRACE: drivers/mag/ist8310/ist8310.c#L51

drivers/mag/ist8310/ist8310.c:51 open brace '{' following function definitions go on the next line

OPEN_BRACE: drivers/mag/ist8310/ist8310.c#L64

drivers/mag/ist8310/ist8310.c:64 open brace '{' following function definitions go on the next line

ASSIGN_IN_IF: drivers/mag/ist8310/ist8310.c#L76

drivers/mag/ist8310/ist8310.c:76 do not use assignment in if condition

ASSIGN_IN_IF: drivers/mag/ist8310/ist8310.c#L79

drivers/mag/ist8310/ist8310.c:79 do not use assignment in if condition

C99_COMMENTS: drivers/mag/ist8310/ist8310.c#L82

drivers/mag/ist8310/ist8310.c:82 do not use C99 // comments

ASSIGN_IN_IF: drivers/mag/ist8310/ist8310.c#L84

drivers/mag/ist8310/ist8310.c:84 do not use assignment in if condition

Run compliance checks on patch series (PR): Identity.txt#L1

See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details 71f4d5412232b39d262260a4a246a098f45a3edf: author email (Mayank Mahajan <mayankmahajan.x@nxp.com>) needs to match one of the signed-off-by entries.

Run compliance checks on patch series (PR): Checkpatch.txt#L1

See https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style for more details. OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:21 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:28 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:35 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:45 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:51 LINE_SPACING: Missing a blank line after declarations File:drivers/mag/ist8310/ist8310.c Line:55 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:64 LINE_SPACING: Missing a blank line after declarations File:drivers/mag/ist8310/ist8310.c Line:68 ASSIGN_IN_IF: do not use assignment in if condition File:drivers/mag/ist8310/ist8310.c Line:76 ASSIGN_IN_IF: do not use assignment in if condition File:drivers/mag/ist8310/ist8310.c Line:79 C99_COMMENTS: do not use C99 // comments File:drivers/mag/ist8310/ist8310.c Line:82 ASSIGN_IN_IF: do not use assignment in if condition File:drivers/mag/ist8310/ist8310.c Line:84 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:90 ASSIGN_IN_IF: do not use assignment in if condition File:drivers/mag/ist8310/ist8310.c Line:97 LINE_SPACING: Missing a blank line after declarations File:drivers/mag/ist8310/ist8310.c Line:101 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:119 C99_COMMENTS: do not use C99 // comments File:drivers/mag/ist8310/ist8310.c Line:129 OPEN_BRACE: open brace '{' following function definitions go on the next line File:drivers/mag/ist8310/ist8310.c Line:134 LONG_LINE: line length of 241 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:168 LONG_LINE: line length of 169 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:169 LONG_LINE: line length of 217 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:170 LONG_LINE: line length of 305 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:171 LONG_LINE: line length of 313 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:172 LONG_LINE: line length of 161 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:173 LONG_LINE: line length of 169 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:174 LONG_LINE: line length of 305 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:175 LONG_LINE: line length of 313 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:176 LONG_LINE: line length of 241 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:177 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/mag/ist8310/ist8310.c Line:177 LONG_LINE: line length of 265 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:178 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/mag/ist8310/ist8310.c Line:178 LONG_LINE: line length of 289 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:179 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/mag/ist8310/ist8310.c Line:179 LONG_LINE: line length of 257 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:180 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/mag/ist8310/ist8310.c Line:180 LONG_LINE: line length of 241 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:181 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/mag/ist8310/ist8310.c Line:181 LONG_LINE: line length of 273 exceeds 100 columns File:drivers/mag/ist8310/ist8310.c Line:182 SPACE_BEFORE_TAB: please, no space before tabs File:drivers/mag/ist8310/ist8310.c Line:182 LONG_LINE: line length of 217 exceeds 100 columns File:drivers/mag/

Run compliance checks on patch series (PR): Gitlint.txt#L1

See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details 1: UC2 Commit message does not contain a 'Signed-off-by:' line 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): YAMLLint.txt#L1

Check YAML files with YAMLLint. YAMLLint (None):syntax error: found character '\t' that cannot start any token (syntax) File:samples/drivers/ist8310/sample.yaml Line:7 Column:1

Run compliance checks on patch series (PR)

Process completed with exit code 1.

LINE_SPACING: drivers/mag/ist8310/ist8310.c#L55

drivers/mag/ist8310/ist8310.c:55 Missing a blank line after declarations

LINE_SPACING: drivers/mag/ist8310/ist8310.c#L68

drivers/mag/ist8310/ist8310.c:68 Missing a blank line after declarations

LINE_SPACING: drivers/mag/ist8310/ist8310.c#L101

drivers/mag/ist8310/ist8310.c:101 Missing a blank line after declarations

LONG_LINE: drivers/mag/ist8310/ist8310.c#L168

drivers/mag/ist8310/ist8310.c:168 line length of 241 exceeds 100 columns

LONG_LINE: drivers/mag/ist8310/ist8310.c#L169

drivers/mag/ist8310/ist8310.c:169 line length of 169 exceeds 100 columns