Attention is currently required from: Hoernchen.
313 comments:
File CMSIS/Include/cmsis_version.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1: /**************************************************************************//**
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2: * @file cmsis_version.h
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3: * @brief CMSIS Core(M) Version definitions
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 4: * @version V5.0.2
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 5: * @date 19. April 2017
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 6: ******************************************************************************/
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 8: * Copyright (c) 2009-2017 ARM Limited. All rights reserved.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 10: * SPDX-License-Identifier: Apache-2.0
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 12: * Licensed under the Apache License, Version 2.0 (the License); you may
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 13: * not use this file except in compliance with the License.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 14: * You may obtain a copy of the License at
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 16: * www.apache.org/licenses/LICENSE-2.0
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 18: * Unless required by applicable law or agreed to in writing, software
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 19: * distributed under the License is distributed on an AS IS BASIS, WITHOUT
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 20: * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 21: * See the License for the specific language governing permissions and
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 22: * limitations under the License.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 25: #if defined ( __ICCARM__ )
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 25: #if defined ( __ICCARM__ )
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 25: #if defined ( __ICCARM__ )
space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 25: #if defined ( __ICCARM__ )
space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 26: #pragma system_include /* treat file as system include file for MISRA check */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 27: #elif defined (__clang__)
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 27: #elif defined (__clang__)
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 28: #pragma clang system_header /* treat file as system include file */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 29: #endif
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 31: #ifndef __CMSIS_VERSION_H
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 32: #define __CMSIS_VERSION_H
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 34: /* CMSIS Version definitions */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 35: #define __CM_CMSIS_VERSION_MAIN ( 5U) /*!< [31:16] CMSIS Core(M) main version */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 35: #define __CM_CMSIS_VERSION_MAIN ( 5U) /*!< [31:16] CMSIS Core(M) main version */
space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 36: #define __CM_CMSIS_VERSION_SUB ( 1U) /*!< [15:0] CMSIS Core(M) sub version */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 36: #define __CM_CMSIS_VERSION_SUB ( 1U) /*!< [15:0] CMSIS Core(M) sub version */
space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 37: #define __CM_CMSIS_VERSION ((__CM_CMSIS_VERSION_MAIN << 16U) | \
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 37: #define __CM_CMSIS_VERSION ((__CM_CMSIS_VERSION_MAIN << 16U) | \
Whitespace after \ makes next lines useless
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 38: __CM_CMSIS_VERSION_SUB ) /*!< CMSIS Core(M) version number */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 38: __CM_CMSIS_VERSION_SUB ) /*!< CMSIS Core(M) version number */
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 38: __CM_CMSIS_VERSION_SUB ) /*!< CMSIS Core(M) version number */
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 38: __CM_CMSIS_VERSION_SUB ) /*!< CMSIS Core(M) version number */
space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 39: #endif
DOS line endings
File hpl/core/hpl_core_m4.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 220: __asm(".align 3 \n"
unnecessary whitespace before a quoted newline
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 226: ".align 3 \n"
unnecessary whitespace before a quoted newline
File hpl/sercom/hpl_sercom.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 89: | (CONF_SERCOM_##n##_USART_RUNSTDBY << SERCOM_USART_CTRLA_RUNSTDBY_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 90: | (CONF_SERCOM_##n##_USART_IBON << SERCOM_USART_CTRLA_IBON_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 91: | (CONF_SERCOM_##n##_USART_TXINV << SERCOM_USART_CTRLA_TXINV_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 92: | (CONF_SERCOM_##n##_USART_RXINV << SERCOM_USART_CTRLA_RXINV_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 93: | SERCOM_USART_CTRLA_SAMPR(CONF_SERCOM_##n##_USART_SAMPR) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 94: | SERCOM_USART_CTRLA_TXPO(CONF_SERCOM_##n##_USART_TXPO) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 95: | SERCOM_USART_CTRLA_RXPO(CONF_SERCOM_##n##_USART_RXPO) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 96: | SERCOM_USART_CTRLA_SAMPA(CONF_SERCOM_##n##_USART_SAMPA) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 97: | SERCOM_USART_CTRLA_FORM(CONF_SERCOM_##n##_USART_FORM) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 98: | (CONF_SERCOM_##n##_USART_CMODE << SERCOM_USART_CTRLA_CMODE_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 99: | (CONF_SERCOM_##n##_USART_CPOL << SERCOM_USART_CTRLA_CPOL_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 100: | (CONF_SERCOM_##n##_USART_DORD << SERCOM_USART_CTRLA_DORD_Pos), \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 102: | (CONF_SERCOM_##n##_USART_SBMODE << SERCOM_USART_CTRLB_SBMODE_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 103: | (CONF_SERCOM_##n##_USART_CLODEN << SERCOM_USART_CTRLB_COLDEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 104: | (CONF_SERCOM_##n##_USART_SFDE << SERCOM_USART_CTRLB_SFDE_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 105: | (CONF_SERCOM_##n##_USART_ENC << SERCOM_USART_CTRLB_ENC_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 106: | (CONF_SERCOM_##n##_USART_PMODE << SERCOM_USART_CTRLB_PMODE_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 107: | (CONF_SERCOM_##n##_USART_TXEN << SERCOM_USART_CTRLB_TXEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 108: | (CONF_SERCOM_##n##_USART_RXEN << SERCOM_USART_CTRLB_RXEN_Pos), \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 110: | (CONF_SERCOM_##n##_USART_DSNACK << SERCOM_USART_CTRLC_DSNACK_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 111: | (CONF_SERCOM_##n##_USART_INACK << SERCOM_USART_CTRLC_INACK_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 112: | SERCOM_USART_CTRLC_MAXITER(CONF_SERCOM_##n##_USART_MAXITER), \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 140: SERCOM_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 143: SERCOM_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 146: SERCOM_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 149: SERCOM_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 152: SERCOM_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 155: SERCOM_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 158: SERCOM_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 161: SERCOM_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 188: const enum usart_baud_rate_mode mode, const uint8_t fraction);
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 188: const enum usart_baud_rate_mode mode, const uint8_t fraction);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 218: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 254: const enum usart_baud_rate_mode mode, const uint8_t fraction)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 254: const enum usart_baud_rate_mode mode, const uint8_t fraction)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 263: const enum usart_baud_rate_mode mode, const uint8_t fraction)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 263: const enum usart_baud_rate_mode mode, const uint8_t fraction)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 471: void _usart_sync_set_flow_control_state(struct _usart_sync_device *const device,
"foo *const bar" should be "foo *const bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 472: const union usart_flow_control_state state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 472: const union usart_flow_control_state state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 481: void _usart_async_set_flow_control_state(struct _usart_async_device *const device,
"foo *const bar" should be "foo *const bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 482: const union usart_flow_control_state state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 482: const union usart_flow_control_state state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 535: Sercom *const sercom_modules[] = SERCOM_INSTS;
need consistent spacing around '*' (ctx:WxV) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 538: if ((uint32_t)hw == (uint32_t)sercom_modules[i]) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 565: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 565: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 626: if (_usarts[i].number == sercom_offset) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 641: if (hw == SERCOM0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 645: if (hw == SERCOM1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 649: if (hw == SERCOM2) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 653: if (hw == SERCOM3) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 657: if (hw == SERCOM4) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 661: if (hw == SERCOM5) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 665: if (hw == SERCOM6) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 730: const enum usart_baud_rate_mode mode, const uint8_t fraction)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 730: const enum usart_baud_rate_mode mode, const uint8_t fraction)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 732: if (USART_BAUDRATE_ASYNCH_ARITHMETIC == mode) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 736: if (USART_BAUDRATE_ASYNCH_FRACTIONAL == mode) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 740: if (USART_BAUDRATE_SYNCH == mode) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 822: if (USART_PARITY_NONE != parity) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 871: if (enabled) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 918: | (CONF_SERCOM_##n##_I2CM_SPEED << SERCOM_I2CM_CTRLA_SPEED_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 919: | (CONF_SERCOM_##n##_I2CM_MEXTTOEN << SERCOM_I2CM_CTRLA_MEXTTOEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 920: | (CONF_SERCOM_##n##_I2CM_SEXTTOEN << SERCOM_I2CM_CTRLA_SEXTTOEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 921: | (CONF_SERCOM_##n##_I2CM_INACTOUT << SERCOM_I2CM_CTRLA_INACTOUT_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 922: | (CONF_SERCOM_##n##_I2CM_LOWTOUT << SERCOM_I2CM_CTRLA_LOWTOUTEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 923: | (CONF_SERCOM_##n##_I2CM_SDAHOLD << SERCOM_I2CM_CTRLA_SDAHOLD_Pos), \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 966: I2CM_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 969: I2CM_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 972: I2CM_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 975: I2CM_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 978: I2CM_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 981: I2CM_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 984: I2CM_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 987: I2CM_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1005: if (_i2cms[i].number == sercom_offset) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1034: if (status & SERCOM_I2CM_STATUS_BUSERR) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1039: } else {
else is not generally useful after a break or return
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1043: if (msg->len > 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1047: if (msg->flags & I2C_M_STOP) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1058: ((((msg->addr & TEN_ADDR_MASK) >> 8) | TEN_ADDR_FRAME) << 1) | I2C_M_RD
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1059: | (hri_sercomi2cm_read_ADDR_reg(hw) & SERCOM_I2CM_ADDR_HS));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1066: if (msg->flags & I2C_M_STOP) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1084: if ((msg->len == 0 && !sclsm) || (msg->len == 1 && sclsm)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1153: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1155: if (hri_sercomi2cm_get_CTRLA_ENABLE_bit(hw)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1164: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1168: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1203: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1240: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1246: if (msg->len == 1 && sclsm) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1254: if (msg->flags & I2C_M_RD) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1259: ((msg->addr & TEN_ADDR_MASK) << 1) | SERCOM_I2CM_ADDR_TENBITEN
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1260: | (hri_sercomi2cm_read_ADDR_reg(hw) & SERCOM_I2CM_ADDR_HS));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1263: ((msg->addr & SEVEN_ADDR_MASK) << 1) | (msg->flags & I2C_M_RD ? I2C_M_RD : 0x0)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1264: | (hri_sercomi2cm_read_ADDR_reg(hw) & SERCOM_I2CM_ADDR_HS));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1288: if (msg->len == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1292: if (i2c_dev->service.msg.flags & I2C_M_BUSY) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1319: FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1319: FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1345: * \retval I2C_OK Operation was successfull
'successfull' may be misspelled - perhaps 'successful'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1366: if (i2c_dev->service.msg.flags & I2C_M_BUSY) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1441: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1443: if (hri_sercomi2cm_get_CTRLA_ENABLE_bit(hw)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1452: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1456: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1473: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1473: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1493: inline static int32_t _sercom_i2c_sync_wait_bus(struct _i2c_m_sync_device *const i2c_dev, uint32_t *flags)
inline keyword should sit between storage class and type
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1496: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1501: if (timeout-- == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1516: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1523: if (msg->len == 1 && sclsm) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1531: if (msg->flags & I2C_M_RD) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1536: ((msg->addr & TEN_ADDR_MASK) << 1) | SERCOM_I2CM_ADDR_TENBITEN
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1537: | (hri_sercomi2cm_read_ADDR_reg(hw) & SERCOM_I2CM_ADDR_HS));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1540: ((msg->addr & SEVEN_ADDR_MASK) << 1) | (msg->flags & I2C_M_RD ? I2C_M_RD : 0x0)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1541: | (hri_sercomi2cm_read_ADDR_reg(hw) & SERCOM_I2CM_ADDR_HS));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1562: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1568: if (i2c_dev->service.msg.flags & I2C_M_BUSY) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1588: if (msg->flags & I2C_M_STOP) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1704: | SERCOM_I2CS_CTRLA_SDAHOLD(CONF_SERCOM_##n##_I2CS_SDAHOLD) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1705: | (CONF_SERCOM_##n##_I2CS_SEXTTOEN << SERCOM_I2CS_CTRLA_SEXTTOEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1706: | (CONF_SERCOM_##n##_I2CS_SPEED << SERCOM_I2CS_CTRLA_SPEED_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1707: | (CONF_SERCOM_##n##_I2CS_SCLSM << SERCOM_I2CS_CTRLA_SCLSM_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1708: | (CONF_SERCOM_##n##_I2CS_LOWTOUT << SERCOM_I2CS_CTRLA_LOWTOUTEN_Pos), \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1711: | SERCOM_I2CS_ADDR_ADDR(CONF_SERCOM_##n##_I2CS_ADDRESS) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1712: | (CONF_SERCOM_##n##_I2CS_TENBITEN << SERCOM_I2CS_ADDR_TENBITEN_Pos) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1713: | SERCOM_I2CS_ADDR_ADDRMASK(CONF_SERCOM_##n##_I2CS_ADDRESS_MASK) \
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1745: I2CS_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1748: I2CS_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1751: I2CS_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1754: I2CS_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1757: I2CS_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1760: I2CS_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1763: I2CS_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1766: I2CS_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1781: if (status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1799: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1992: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1992: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 1996: if (I2C_S_DEVICE_TX == type || I2C_S_DEVICE_RX_COMPLETE == type) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2006: * \internal Initalize i2c slave hardware
'Initalize' may be misspelled - perhaps 'Initialize'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2015: if (i == -1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2049: if (_i2css[i].number == sercom_offset) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2086: if (enabled) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2179: SERCOMSPI_REGS(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2182: SERCOMSPI_REGS(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2185: SERCOMSPI_REGS(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2188: SERCOMSPI_REGS(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2191: SERCOMSPI_REGS(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2194: SERCOMSPI_REGS(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2197: SERCOMSPI_REGS(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2200: SERCOMSPI_REGS(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2227: if (hri_sercomspi_is_syncing(hw, SERCOM_SPI_SYNCBUSY_SWRST)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2246: for (uint32_t i = 0; i < 4; i++) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2261: if (hri_sercomspi_is_syncing(hw, SERCOM_SPI_SYNCBUSY_SWRST)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2281: for (uint32_t i = 0; i < 4; i++) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2299: if (hri_sercomspi_is_syncing(hw, SERCOM_SPI_SYNCBUSY_SWRST | SERCOM_SPI_SYNCBUSY_ENABLE)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2320: if (hri_sercomspi_is_syncing(hw, SERCOM_SPI_SYNCBUSY_SWRST)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2340: if (!(char_size == SPI_CHAR_SIZE_8 || char_size == SPI_CHAR_SIZE_9)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2344: if (hri_sercomspi_is_syncing(hw, SERCOM_SPI_SYNCBUSY_SWRST | SERCOM_SPI_SYNCBUSY_CTRLB)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2365: if (hri_sercomspi_is_syncing(hw, SERCOM_SPI_SYNCBUSY_SWRST)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2371: if (dord == SPI_DATA_ORDER_LSB_1ST) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2396: | (SERCOM_SPI_CTRLB_RXEN));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2413: (regs->ctrlb & ~(SERCOM_SPI_CTRLB_MSSEN))
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2414: | (SERCOM_SPI_CTRLB_RXEN | SERCOM_SPI_CTRLB_SSDE | SERCOM_SPI_CTRLB_PLOADEN));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2431: if (sercomspi_regs[i].n == n) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2648: if (regs == NULL) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2664: if ((regs->ctrla & SERCOM_SPI_CTRLA_MODE_Msk) == SERCOM_USART_CTRLA_MODE_SPI_SLAVE) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2689: if (rc < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2829: if (clk == 0 || baud == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2834: if (!(baud <= (clk >> 1) && baud >= (clk >> 8))) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2915: while (!(hri_sercomspi_get_INTFLAG_reg(hw, SERCOM_SPI_INTFLAG_TXC | SERCOM_SPI_INTFLAG_DRE))) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2940: if (!(iflag & SERCOM_SPI_INTFLAG_RXC)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2949: if (ctrl->char_size > 1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2961: uint16_t dummy)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2961: uint16_t dummy)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2965: if (!(SERCOM_SPI_INTFLAG_DRE & iflag)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 2998: void * hw = dev->prvt;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3006: hw, (SERCOM_SPI_SYNCBUSY_SWRST | SERCOM_SPI_SYNCBUSY_ENABLE | SERCOM_SPI_SYNCBUSY_CTRLB))) {
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3011: if (!hri_sercomspi_get_CTRLA_ENABLE_bit(hw)) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3021: if (ctrl.rxcnt >= ctrl.txcnt) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3028: if (rc < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3048: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3069: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3087: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3150: const FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3150: const FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3164: const FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3164: const FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3218: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3218: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3222: if (SPI_DEV_CB_ERROR == type) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3235: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 3235: const bool state)
please, no spaces at the start of a line
File usb_dfu_main.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 18: * License along with this library; if not, write to the Free Software
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 19: * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 36: static volatile const uint32_t bl_update_data[BLSZ_BYTE / sizeof(uint32_t)] __attribute__((section(".fwupdate"),used,retain)) = { 0xFF };
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 36: static volatile const uint32_t bl_update_data[BLSZ_BYTE / sizeof(uint32_t)] __attribute__((section(".fwupdate"),used,retain)) = { 0xFF };
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 38: _Static_assert (BLSZ_BYTE % NVMCTRL_PAGE_SIZE == 0, "invalid bl size");
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 42: void same54_flash_write_page(uint32_t address, uint8_t *data) {
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 44: while (!NVMCTRL->STATUS.bit.READY) {}
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 44: while (!NVMCTRL->STATUS.bit.READY) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 48: while (!NVMCTRL->STATUS.bit.READY) {}
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 48: while (!NVMCTRL->STATUS.bit.READY) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 51: volatile uint32_t *src = (uint32_t *)data;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 53: for (uint32_t i = 0; i < (NVMCTRL_PAGE_SIZE / sizeof(uint32_t)); i++) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 53: for (uint32_t i = 0; i < (NVMCTRL_PAGE_SIZE / sizeof(uint32_t)); i++) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 54: dst[i] = src[i];
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 54: dst[i] = src[i];
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 57: NVMCTRL->ADDR.reg = address;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 58: NVMCTRL->CTRLB.reg = NVMCTRL_CTRLB_CMD_WP | NVMCTRL_CTRLB_CMDEX_KEY;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 59: while (!NVMCTRL->INTFLAG.bit.DONE) {}
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 59: while (!NVMCTRL->INTFLAG.bit.DONE) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 60: while (!NVMCTRL->STATUS.bit.READY) {}
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 60: while (!NVMCTRL->STATUS.bit.READY) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 63: __attribute__((section(".ramfunc"),OPTNONE))
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 64: void do_update() {
Bad function definition - void do_update() should probably be void do_update(void)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 64: void do_update() {
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 65: uint16_t mask = NVMCTRL_CTRLA_AHBNS0 |NVMCTRL_CTRLA_AHBNS1|NVMCTRL_CTRLA_RWS_Msk;
need consistent spacing around '|' (ctx:WxV) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 66: uint16_t _nvm = (NVMCTRL_CTRLA_CACHEDIS0 | NVMCTRL_CTRLA_CACHEDIS1 | NVMCTRL_CTRLA_PRM(3) | NVMCTRL_CTRLA_RWS(0x7) | NVMCTRL_CTRLA_WMODE(0) |NVMCTRL_CTRLA_AUTOWS );
need consistent spacing around '|' (ctx:WxV) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 66: uint16_t _nvm = (NVMCTRL_CTRLA_CACHEDIS0 | NVMCTRL_CTRLA_CACHEDIS1 | NVMCTRL_CTRLA_PRM(3) | NVMCTRL_CTRLA_RWS(0x7) | NVMCTRL_CTRLA_WMODE(0) |NVMCTRL_CTRLA_AUTOWS );
space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 83: _Static_assert (BL_IN_REGIONS > 0, "bootprot regions must be at least 1");
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 86: for(int i = 0; i< NVMCTRL_REGIONS_NUM; i++){
spaces required around that '<' (ctx:VxW)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 86: for(int i = 0; i< NVMCTRL_REGIONS_NUM; i++){
space required before the open brace '{'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 86: for(int i = 0; i< NVMCTRL_REGIONS_NUM; i++){
space required before the open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 87: while (!hri_nvmctrl_get_STATUS_READY_bit(NVMCTRL)) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 88: hri_nvmctrl_write_ADDR_reg(NVMCTRL, REGION_SZ* i);
need consistent spacing around '*' (ctx:VxW) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 94: while (!hri_nvmctrl_get_STATUS_READY_bit(NVMCTRL)) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 96: for(int i = 0; i< bootloader_regions; i++){
spaces required around that '<' (ctx:VxW)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 96: for(int i = 0; i< bootloader_regions; i++){
space required before the open brace '{'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 96: for(int i = 0; i< bootloader_regions; i++){
space required before the open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 97: while (!hri_nvmctrl_get_STATUS_READY_bit(NVMCTRL)) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 102: while (!hri_nvmctrl_get_STATUS_READY_bit(NVMCTRL)) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 105: uint32_t address = 0;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 106: uint8_t *data_ptr = (uint8_t *)bl_update_data;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 107: uint32_t remaining = BLSZ_BYTE;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 109: while (remaining >= NVMCTRL_PAGE_SIZE) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 110: same54_flash_write_page(address, data_ptr);
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 110: same54_flash_write_page(address, data_ptr);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 111: address += NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 111: address += NVMCTRL_PAGE_SIZE;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 112: data_ptr += NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 112: data_ptr += NVMCTRL_PAGE_SIZE;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 113: remaining -= NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 113: remaining -= NVMCTRL_PAGE_SIZE;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 117: for(int i = bootloader_regions; i < bootloader_regions+1; i++){
space required before the open brace '{'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 117: for(int i = bootloader_regions; i < bootloader_regions+1; i++){
space required before the open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 118: while (!(((Nvmctrl *)NVMCTRL)->STATUS.reg & NVMCTRL_STATUS_READY) >> NVMCTRL_STATUS_READY_Pos) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 122: while (!(((Nvmctrl *)NVMCTRL)->STATUS.reg & NVMCTRL_STATUS_READY) >> NVMCTRL_STATUS_READY_Pos) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 125: SCB->AIRCR = (uint32_t)((0x5FAUL << SCB_AIRCR_VECTKEY_Pos) |(SCB->AIRCR & SCB_AIRCR_PRIGROUP_Msk) |SCB_AIRCR_SYSRESETREQ_Msk );
need consistent spacing around '|' (ctx:WxV) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 125: SCB->AIRCR = (uint32_t)((0x5FAUL << SCB_AIRCR_VECTKEY_Pos) |(SCB->AIRCR & SCB_AIRCR_PRIGROUP_Msk) |SCB_AIRCR_SYSRESETREQ_Msk );
need consistent spacing around '|' (ctx:WxV) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 125: SCB->AIRCR = (uint32_t)((0x5FAUL << SCB_AIRCR_VECTKEY_Pos) |(SCB->AIRCR & SCB_AIRCR_PRIGROUP_Msk) |SCB_AIRCR_SYSRESETREQ_Msk );
space prohibited before that close parenthesis ')'
File usb_flash_main.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 30: static uint32_t* application_start_address = (uint32_t*)(BL_SIZE_BYTE);
"(foo*)" should be "(foo *)"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 30: static uint32_t* application_start_address = (uint32_t*)(BL_SIZE_BYTE);
"foo* bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 137: while (!hri_nvmctrl_get_STATUS_READY_bit(NVMCTRL)) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
Patch Set #2, Line 139: _user_area_write_bits((void*)NVMCTRL_FUSES_BOOTPROT_ADDR, NVMCTRL_FUSES_BOOTPROT_Pos, 13, 4);
"(foo*)" should be "(foo *)"
To view, visit change 39434. To unsubscribe, or for help writing mail filters, visit settings.