Support for baremetal R5 in TI K3 family devices #533
kwillis01compliance.yml
on: pull_request
checkpatch review
1h 14m
Annotations
13 errors and 10 warnings
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L38
apps/machine/ti_k3_r5/helper.c:38 do not use C99 // comments
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L39
apps/machine/ti_k3_r5/helper.c:39 do not use C99 // comments
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L40
apps/machine/ti_k3_r5/helper.c:40 do not use C99 // comments
|
|
OPEN_BRACE:
apps/machine/ti_k3_r5/helper.c#L42
apps/machine/ti_k3_r5/helper.c:42 that open brace { should be on the previous line
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L43
apps/machine/ti_k3_r5/helper.c:43 do not use C99 // comments
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L58
apps/machine/ti_k3_r5/helper.c:58 do not use C99 // comments
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L74
apps/machine/ti_k3_r5/helper.c:74 do not use C99 // comments
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L91
apps/machine/ti_k3_r5/helper.c:91 do not use C99 // comments
|
|
FUNCTION_WITHOUT_ARGS:
apps/machine/ti_k3_r5/helper.c#L103
apps/machine/ti_k3_r5/helper.c:103 Bad function definition - void vectors() should probably be void vectors(void)
|
|
C99_COMMENTS:
apps/machine/ti_k3_r5/helper.c#L116
apps/machine/ti_k3_r5/helper.c:116 do not use C99 // comments
|
|
checkpatch review:
Gitlint.txt#L1
Commit ac3d1c950f:
1: UC5 Title exceeds max length (75>72): "cmake: platforms: Added libmetal library and include path to ti_k3_r5.cmake"
|
|
checkpatch review:
Checkpatch.txt#L1
CAMELCASE: Avoid CamelCase: <HwiP_Config>
File:apps/machine/ti_k3_r5/helper.c
Line:33
CAMELCASE: Avoid CamelCase: <gHwiConfig>
File:apps/machine/ti_k3_r5/helper.c
Line:33
CAMELCASE: Avoid CamelCase: <intcBaseAddr>
File:apps/machine/ti_k3_r5/helper.c
Line:34
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:38
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:39
CAMELCASE: Avoid CamelCase: <CacheP_Config>
File:apps/machine/ti_k3_r5/helper.c
Line:39
CAMELCASE: Avoid CamelCase: <gCacheConfig>
File:apps/machine/ti_k3_r5/helper.c
Line:39
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:40
CAMELCASE: Avoid CamelCase: <MpuP_Config>
File:apps/machine/ti_k3_r5/helper.c
Line:40
CAMELCASE: Avoid CamelCase: <gMpuConfig>
File:apps/machine/ti_k3_r5/helper.c
Line:40
CAMELCASE: Avoid CamelCase: <MpuP_RegionConfig>
File:apps/machine/ti_k3_r5/helper.c
Line:41
CAMELCASE: Avoid CamelCase: <gMpuRegionConfig>
File:apps/machine/ti_k3_r5/helper.c
Line:41
OPEN_BRACE: that open brace { should be on the previous line
File:apps/machine/ti_k3_r5/helper.c
Line:42
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:43
CAMELCASE: Avoid CamelCase: <baseAddr>
File:apps/machine/ti_k3_r5/helper.c
Line:45
CAMELCASE: Avoid CamelCase: <MpuP_RegionSize_2G>
File:apps/machine/ti_k3_r5/helper.c
Line:46
CAMELCASE: Avoid CamelCase: <isEnable>
File:apps/machine/ti_k3_r5/helper.c
Line:48
CAMELCASE: Avoid CamelCase: <isCacheable>
File:apps/machine/ti_k3_r5/helper.c
Line:49
CAMELCASE: Avoid CamelCase: <isBufferable>
File:apps/machine/ti_k3_r5/helper.c
Line:50
CAMELCASE: Avoid CamelCase: <isSharable>
File:apps/machine/ti_k3_r5/helper.c
Line:51
CAMELCASE: Avoid CamelCase: <isExecuteNever>
File:apps/machine/ti_k3_r5/helper.c
Line:52
CAMELCASE: Avoid CamelCase: <accessPerm>
File:apps/machine/ti_k3_r5/helper.c
Line:54
CAMELCASE: Avoid CamelCase: <MpuP_AP_ALL_RW>
File:apps/machine/ti_k3_r5/helper.c
Line:54
CAMELCASE: Avoid CamelCase: <subregionDisableMask>
File:apps/machine/ti_k3_r5/helper.c
Line:55
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:58
CAMELCASE: Avoid CamelCase: <MpuP_RegionSize_1M>
File:apps/machine/ti_k3_r5/helper.c
Line:61
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:74
CAMELCASE: Avoid CamelCase: <MpuP_RegionSize_4K>
File:apps/machine/ti_k3_r5/helper.c
Line:77
C99_COMMENTS: do not use C99 // comments
File:apps/machine/ti_k3_r5/helper.c
Line:91
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:94
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:96
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:97
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:98
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:99
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:100
FUNCTION_ARGUMENTS: function definition argument 'void' should also have an identifier name
File:apps/machine/ti_k3_r5/helper.c
Line:101
FUNCTION_WITHOUT_ARGS: Bad function definition - void vectors() should probably be void vectors(void)
File:apps/machine/ti_k3_r5/helper.c
Line:103
QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
File:apps/machine/ti_k3_r5/helper.c
Line:106
QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
File:apps/machine/ti_k3_r5/helper.c
Line:107
QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
File:apps/machine/ti_k3_r5/helper.c
Line:108
QUOTED_WHITESPACE_BEFORE_NE
|
|
checkpatch review
Process completed with exit code 1.
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L94
apps/machine/ti_k3_r5/helper.c:94 function definition argument 'void' should also have an identifier name
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L96
apps/machine/ti_k3_r5/helper.c:96 function definition argument 'void' should also have an identifier name
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L97
apps/machine/ti_k3_r5/helper.c:97 function definition argument 'void' should also have an identifier name
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L98
apps/machine/ti_k3_r5/helper.c:98 function definition argument 'void' should also have an identifier name
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L99
apps/machine/ti_k3_r5/helper.c:99 function definition argument 'void' should also have an identifier name
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L100
apps/machine/ti_k3_r5/helper.c:100 function definition argument 'void' should also have an identifier name
|
|
FUNCTION_ARGUMENTS:
apps/machine/ti_k3_r5/helper.c#L101
apps/machine/ti_k3_r5/helper.c:101 function definition argument 'void' should also have an identifier name
|
|
QUOTED_WHITESPACE_BEFORE_NEWLINE:
apps/machine/ti_k3_r5/helper.c#L106
apps/machine/ti_k3_r5/helper.c:106 unnecessary whitespace before a quoted newline
|
|
QUOTED_WHITESPACE_BEFORE_NEWLINE:
apps/machine/ti_k3_r5/helper.c#L107
apps/machine/ti_k3_r5/helper.c:107 unnecessary whitespace before a quoted newline
|
|
QUOTED_WHITESPACE_BEFORE_NEWLINE:
apps/machine/ti_k3_r5/helper.c#L108
apps/machine/ti_k3_r5/helper.c:108 unnecessary whitespace before a quoted newline
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
compliance.xml
Expired
|
440 KB |
|