[S] Change in osmo-asf4-dfu[master]: errata 2.6.10
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435?usp=email )
Change subject: errata 2.6.10
......................................................................
Patch Set 4:
(7 comments)
File gcc/gcc/startup_same54.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/21f6dc3c_6b32b17… :
PS4, Line 675: __asm volatile (
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/6bd42d6b_6d7539e… :
PS4, Line 676: "movs r0, #0\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/339b2fd7_c017abb… :
PS4, Line 677: "ldr r1, =0x40002000\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/3f9259be_e86c29c… :
PS4, Line 678: "strb r0, [r1]\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/cca64a37_2fe2970… :
PS4, Line 679: "bl _Reset_Handler\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/4cdf6a09_f339987… :
PS4, Line 680: ::: "r0", "r1", "memory"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21272):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/58b241d6_fbf4ca7… :
PS4, Line 681: );
please, no spaces at the start of a line
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: Ifbc6a3dfe91462029fce50ed42f20440debaa552
Gerrit-Change-Number: 39435
Gerrit-PatchSet: 4
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 19:10:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0
[L] Change in osmo-asf4-dfu[master]: add fw update crcstub
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437?usp=email )
Change subject: add fw update crcstub
......................................................................
Patch Set 4:
(68 comments)
File crc_code.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/f3e516c4_be02384… :
PS4, Line 17: uint32_t crc = 0xFFFFFFFF;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/918e9e03_a4115f5… :
PS4, Line 18: for (uint32_t i = 0; i < length; i++) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/5856f687_7511e6a… :
PS4, Line 18: for (uint32_t i = 0; i < length; i++) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/6636f236_58a76e4… :
PS4, Line 19: crc ^= data[i];
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/a2882f3e_d8fe627… :
PS4, Line 20: for (uint8_t j = 0; j < 8; j++) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/3bcba590_4de9bd8… :
PS4, Line 20: for (uint8_t j = 0; j < 8; j++) {
suspect code indent for conditional statements (4, 6)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/7ce8cae0_0a9590d… :
PS4, Line 22: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/2f4d0fb4_cb21eba… :
PS4, Line 23: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/fcf74b2a_6c5a766… :
PS4, Line 24: return ~crc;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/9a141e28_c39b7fb… :
PS4, Line 29: extern uint32_t __CRC_Checksum;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/6947c4d6_359eebc… :
PS4, Line 30: extern uint32_t __CRC_Start;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/437181b3_c7cee8d… :
PS4, Line 31: extern uint32_t __CRC_End;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/6d2bd7f5_b8de467… :
PS4, Line 32: extern int main(void);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4d2cc97b_8a27a75… :
PS4, Line 39: USB->DEVICE.CTRLA.reg = 0x1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d52e3268_591c910… :
PS4, Line 41: uint32_t length = (uint32_t)&__CRC_End - (uint32_t)&__CRC_Start;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/b16850ae_8d79dea… :
PS4, Line 42: uint32_t calculated_crc = calculate_crc((uint8_t *)&__CRC_Start, length);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/a934d927_6f12f17… :
PS4, Line 43: uint32_t expected_crc = *(uint32_t *)&__CRC_Checksum;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/588c4670_bc525cc… :
PS4, Line 45: if (calculated_crc != expected_crc) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/58cb0e81_c188fad… :
PS4, Line 45: if (calculated_crc != expected_crc) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/301b5ca3_74b8b3c… :
PS4, Line 46: static volatile uint32_t *dfu_magic = (uint32_t *)HSRAM_ADDR;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/52d53bf1_035536d… :
PS4, Line 47: *dfu_magic = 0x44465521;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d25c871b_b98bc35… :
PS4, Line 50: __DSB();
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/6a7abe27_d181552… :
PS4, Line 51: SCB->AIRCR = (uint32_t)((0x5FAUL << SCB_AIRCR_VECTKEY_Pos) | (SCB->AIRCR & SCB_AIRCR_PRIGROUP_Msk) |
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/21f06f7a_45322b7… :
PS4, Line 53: __DSB();
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/803b43aa_83238d2… :
PS4, Line 54: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1d2dbeb5_a4e099c… :
PS4, Line 58: _Reset_Handler();
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/17a357c3_3250994… :
PS4, Line 61: while (1)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/466211c2_12ac7cb… :
PS4, Line 61: while (1)
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/e43d5179_d8b551d… :
PS4, Line 62: ;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/93e7bfd7_a7e9e26… :
PS4, Line 74: if (argc != 4) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/3c49766e_6c84865… :
PS4, Line 74: if (argc != 4) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d31e2b7a_2485341… :
PS4, Line 75: fprintf(stderr, "Usage: %s <binary_file> <start_offset_hex> <length_hex>\n", argv[0]);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/47d6b34f_f897d70… :
PS4, Line 76: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/2a920fe4_b3d94df… :
PS4, Line 77: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/ae4ae439_d9d40ed… :
PS4, Line 79: const char *filename = argv[1];
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/35aa2ac0_4f633e2… :
PS4, Line 82: uint32_t start_offset = strtoul(argv[2], NULL, 10);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/e91f9d5c_15a0cbc… :
PS4, Line 83: uint32_t length = strtoul(argv[3], NULL, 10);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/03598e40_102b770… :
PS4, Line 85: FILE *f = fopen(filename, "rb");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/7d681359_d7381f5… :
PS4, Line 86: if (!f) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/8cb0d3d7_3cca514… :
PS4, Line 86: if (!f) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1c9bce00_5a0f900… :
PS4, Line 87: perror("fopen");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/c368621f_98e255d… :
PS4, Line 88: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/943a7fb3_ba37966… :
PS4, Line 89: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/fa526b3e_1a7b20d… :
PS4, Line 92: if (fseek(f, (long)start_offset, SEEK_SET) != 0) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4eae480c_b2835e9… :
PS4, Line 92: if (fseek(f, (long)start_offset, SEEK_SET) != 0) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4c8d80ec_383913d… :
PS4, Line 93: perror("fseek");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/84e7a899_9d88e1f… :
PS4, Line 94: fclose(f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/3fb6750b_6d70d07… :
PS4, Line 95: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4f4e655c_372f1fb… :
PS4, Line 96: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1c6866a4_b6262e5… :
PS4, Line 98: uint8_t *buf = (uint8_t *)malloc(length);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/82d583cc_96ffb75… :
PS4, Line 99: if (!buf) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/bef9d47d_0462441… :
PS4, Line 99: if (!buf) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/89a359d3_b00dadb… :
PS4, Line 100: fprintf(stderr, "Memory allocation error\n");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/90842160_097c460… :
PS4, Line 101: fclose(f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/f6e925f4_415ee23… :
PS4, Line 102: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/65e162de_35c7c3b… :
PS4, Line 103: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4ae04cb1_a694d4b… :
PS4, Line 105: size_t read_bytes = fread(buf, 1, length, f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/76beecc6_a0d490a… :
PS4, Line 106: fclose(f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/c1a0f7e9_35d94b3… :
PS4, Line 108: if (read_bytes != length) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/b8afb52e_2df990a… :
PS4, Line 108: if (read_bytes != length) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/a51b9388_c2c013e… :
PS4, Line 109: fprintf(stderr, "Error reading file or not enough data\n");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/54780b29_1d7cca2… :
PS4, Line 110: free(buf);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/891f26e3_5cad7b8… :
PS4, Line 111: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/859a8d02_36d61f3… :
PS4, Line 112: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/ffe48f37_5354a2e… :
PS4, Line 114: uint32_t crc = calculate_crc(buf, length);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/243b511d_25b52e6… :
PS4, Line 115: free(buf);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/33f2d29b_6735875… :
PS4, Line 117: printf("%08X\n", crc);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21271):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/69750672_e03c41b… :
PS4, Line 118: return 0;
please, no spaces at the start of a line
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I39eae7aaafd5531db6ce48837c9499432caadbed
Gerrit-Change-Number: 39437
Gerrit-PatchSet: 4
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 19:10:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0
[S] Change in osmo-asf4-dfu[master]: tame the linter
by Hoernchen
Attention is currently required from: Hoernchen.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39439?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: tame the linter
......................................................................
tame the linter
Change-Id: I745b741dedf2c2ed5679c289e70adcb01c750e4d
---
A .checkpatch.conf
1 file changed, 15 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-asf4-dfu refs/changes/39/39439/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39439?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I745b741dedf2c2ed5679c289e70adcb01c750e4d
Gerrit-Change-Number: 39439
Gerrit-PatchSet: 3
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
4 months, 3 weeks
- 1
- 0
[NoOp] Change in osmo-asf4-dfu[master]: rename bl app
by Hoernchen
Hoernchen has abandoned this change. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39432?usp=email )
Change subject: rename bl app
......................................................................
Abandoned
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39432?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: abandon
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I313b2c45017b7c83989d4c61594ce47edb2f2965
Gerrit-Change-Number: 39432
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
4 months, 3 weeks
- 1
- 0
[XL] Change in osmo-asf4-dfu[master]: bl updater app
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434?usp=email )
Change subject: bl updater app
......................................................................
Patch Set 3:
(322 comments)
File CMSIS/Include/cmsis_version.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/675b6f4f_f60752c… :
PS3, Line 1: /**************************************************************************//**
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/050c3176_8c2b846… :
PS3, Line 2: * @file cmsis_version.h
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8475c5ef_379855d… :
PS3, Line 3: * @brief CMSIS Core(M) Version definitions
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0d2d5bc9_ea63dcd… :
PS3, Line 4: * @version V5.0.2
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/624713db_2e4a109… :
PS3, Line 5: * @date 19. April 2017
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/90e86adc_2c0ed3f… :
PS3, Line 6: ******************************************************************************/
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/918077d6_ac19029… :
PS3, Line 7: /*
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f50a4938_2902de6… :
PS3, Line 8: * Copyright (c) 2009-2017 ARM Limited. All rights reserved.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/937f8d00_7c87a9a… :
PS3, Line 9: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b8b53957_09b43f8… :
PS3, Line 10: * SPDX-License-Identifier: Apache-2.0
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7a34c2e5_4b7ccd3… :
PS3, Line 11: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ddaaa62a_f7a04fc… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6bf2198d_a6b6663… :
PS3, Line 13: * not use this file except in compliance with the License.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/93e7c7da_3f3d531… :
PS3, Line 14: * You may obtain a copy of the License at
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/331010d5_354e052… :
PS3, Line 15: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0074e2c3_557d8ea… :
PS3, Line 16: * www.apache.org/licenses/LICENSE-2.0
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c5b37212_a31056d… :
PS3, Line 17: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8f0023e3_0e8acaa… :
PS3, Line 18: * Unless required by applicable law or agreed to in writing, software
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/45009aa4_c0866a2… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/46d29b93_e526849… :
PS3, Line 20: * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f2b5e8bf_34798a6… :
PS3, Line 21: * See the License for the specific language governing permissions and
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/508f32c4_f21d94a… :
PS3, Line 22: * limitations under the License.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d23d5f2d_df77138… :
PS3, Line 23: */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/57d27d4a_bbd7180… :
PS3, Line 24:
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e6ce6169_24b5511… :
PS3, Line 25: #if defined ( __ICCARM__ )
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/59affe71_309b678… :
PS3, Line 25: #if defined ( __ICCARM__ )
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e34f6d31_fb2641d… :
PS3, Line 25: #if defined ( __ICCARM__ )
space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c4dc937f_deac6b4… :
PS3, Line 25: #if defined ( __ICCARM__ )
space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/454bf3ec_c7475d7… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e14de423_eab335b… :
PS3, Line 27: #elif defined (__clang__)
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/532341d2_6f0c6b3… :
PS3, Line 27: #elif defined (__clang__)
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4360bb87_bb89338… :
PS3, Line 28: #pragma clang system_header /* treat file as system include file */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/75456734_0167499… :
PS3, Line 29: #endif
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/901ccbb3_bedb430… :
PS3, Line 30:
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/75c52e0a_1409c8b… :
PS3, Line 31: #ifndef __CMSIS_VERSION_H
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0f05935e_d5fef77… :
PS3, Line 32: #define __CMSIS_VERSION_H
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6e2c3d1c_d38c598… :
PS3, Line 33:
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/38981c46_0748318… :
PS3, Line 34: /* CMSIS Version definitions */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b4e8c307_0407fbe… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3e98d90a_0bcf994… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/87878a2a_3e844b3… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e962dad2_5db2b7a… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/51e33dd8_944ab02… :
PS3, Line 37: #define __CM_CMSIS_VERSION ((__CM_CMSIS_VERSION_MAIN << 16U) | \
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0ccf9d71_2139c9b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3c9197fb_4ae7a2b… :
PS3, Line 38: __CM_CMSIS_VERSION_SUB ) /*!< CMSIS Core(M) version number */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/573dc333_621a6e9… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0c19163f_f06afed… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/409ee172_6167cea… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f558455a_d17c8b6… :
PS3, Line 39: #endif
DOS line endings
File hpl/core/hpl_core_m4.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f457e388_9b8c459… :
PS3, Line 220: __asm(".align 3 \n"
unnecessary whitespace before a quoted newline
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9138d216_91ae327… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/94208abf_e60104a… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/49aff5d9_bb87a5d… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4d7882eb_639a808… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/55762feb_2ee5f7b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8223d806_eec24b2… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9cccae8a_25aab33… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8ceca226_8832f8c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/53d05104_3a72288… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8eecc894_c50032f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/82f89882_070d3c7… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3bafb7fd_0e843e1… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c89e029d_a564abd… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/66e2983a_e32d91c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8d32fed9_756cab7… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8bd0745b_22c8ef8… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/414c0250_55caa53… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6438a211_470426a… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/50585297_944c29b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d2f5a2d6_478be92… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/13b3b526_a58cfaf… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2bf95f3b_cc1a34f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/244328f5_a858889… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3597467b_a77abac… :
PS3, Line 140: SERCOM_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/751825c1_9634b41… :
PS3, Line 143: SERCOM_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5058a56e_ac55d89… :
PS3, Line 146: SERCOM_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1449ed3b_ef7f169… :
PS3, Line 149: SERCOM_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8276cf7e_2bf32ac… :
PS3, Line 152: SERCOM_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/db1821de_5ffaa45… :
PS3, Line 155: SERCOM_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/825b2e20_868aa81… :
PS3, Line 158: SERCOM_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/90356498_ddd908d… :
PS3, Line 161: SERCOM_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4532f264_c962c93… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/250684da_096c76a… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/67470641_a6b2c64… :
PS3, Line 218: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5dabbfa8_8fd61b1… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b10985d7_1d5db84… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/33ad60c2_4ff9f85… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/42c4ea4c_0b261b6… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/07f66332_f8d1caf… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/96080f5a_2438309… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/eab40031_0d774b5… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4835f630_cdd0714… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5472f6d8_a39bc90… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5de67255_1f294ff… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/aaeebb16_8ad4265… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4d66c075_2be65e1… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c3c7d606_026da79… :
PS3, Line 565: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/250d483f_ed4ce1e… :
PS3, Line 565: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f223db45_cb01ab6… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/20bb131e_8053e27… :
PS3, Line 641: if (hw == SERCOM0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/842f9821_6d6ed31… :
PS3, Line 645: if (hw == SERCOM1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/14554c21_4e9196f… :
PS3, Line 649: if (hw == SERCOM2) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1b777547_f975716… :
PS3, Line 653: if (hw == SERCOM3) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/720421b8_0295e11… :
PS3, Line 657: if (hw == SERCOM4) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f579c879_05efa50… :
PS3, Line 661: if (hw == SERCOM5) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2b2e0867_2977ee5… :
PS3, Line 665: if (hw == SERCOM6) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1d7d8e10_6af0b43… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c109ec2b_3571561… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bfd191d1_f3e481c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/234238a4_f4ec47f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3df9ed5c_ef7dcda… :
PS3, Line 740: if (USART_BAUDRATE_SYNCH == mode) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ee5e0d74_e46a9ce… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6b6efce8_9870bb6… :
PS3, Line 871: if (enabled) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b1101977_b251917… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fc7277be_700605b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6b760db1_f1098ac… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a49c2b31_ed4eefb… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a6b7e2dd_b9dbe66… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/07105ba1_89149c1… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9a93e69d_2fd2892… :
PS3, Line 966: I2CM_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/35ede16c_30a1cab… :
PS3, Line 969: I2CM_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f5b2ea63_1c618fc… :
PS3, Line 972: I2CM_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d486ba0b_956de34… :
PS3, Line 975: I2CM_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/37cef0ea_47ea8a6… :
PS3, Line 978: I2CM_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7b5f7015_352b378… :
PS3, Line 981: I2CM_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/690b3910_ff224a1… :
PS3, Line 984: I2CM_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f52eb8b9_9707df5… :
PS3, Line 987: I2CM_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/062d0fe3_844202c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4cbf4767_ef80e68… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/94532f90_e808ed1… :
PS3, Line 1039: } else {
else is not generally useful after a break or return
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b02e2f6b_55392ac… :
PS3, Line 1043: if (msg->len > 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0618057e_1a46361… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4e6d08ba_e84fa43… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7b29b18c_a1f62b3… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/34948212_a870c28… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ac90f250_942fd69… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/83b563b7_02e5450… :
PS3, Line 1153: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/868e62d0_d1f3f3b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f1623925_8ea09e3… :
PS3, Line 1164: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/72622059_67e64ca… :
PS3, Line 1168: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bbc535de_1f43033… :
PS3, Line 1203: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/91d291cf_e2f0df1… :
PS3, Line 1240: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/de13e68e_83db3d1… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1bc22688_0169fc4… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cebf2b88_673d598… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/59d8af84_d865166… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fbcba23b_1dc6f18… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/071850e1_43b0c73… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/39aaf07c_97ba736… :
PS3, Line 1288: if (msg->len == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ee3e6944_2af1e26… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9bac2837_6557e95… :
PS3, Line 1319: FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2cb087c5_e1400e1… :
PS3, Line 1319: FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dcd0f3d9_1230dec… :
PS3, Line 1345: * \retval I2C_OK Operation was successfull
'successfull' may be misspelled - perhaps 'successful'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a80046d9_275daf9… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c67cb69a_c7c80dc… :
PS3, Line 1441: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cde0b22d_2244347… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/37206a74_1658469… :
PS3, Line 1452: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/799ed168_51d6a2e… :
PS3, Line 1456: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b07e26cf_21287c0… :
PS3, Line 1473: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5733b49e_6dcd63e… :
PS3, Line 1473: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/30a56169_93bf088… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f3f7c9ea_111ff84… :
PS3, Line 1496: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1d935d14_c8533c9… :
PS3, Line 1501: if (timeout-- == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/95bf6465_e324108… :
PS3, Line 1516: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a07a8912_00f5d8a… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fcf15938_34cdb8c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2d123607_8ed3724… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ace03568_984bee4… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/21974bb7_cc29c12… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4a579041_e923e94… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7e0e3843_33b1af4… :
PS3, Line 1562: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/55806e2c_6380a70… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/de73a844_0ebf669… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e5ac38a3_c8bd9fb… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3f53dfa8_d920acf… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3d15ed16_f712c6c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8e1cf1c0_45b7a8d… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6016a777_f8d3da3… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ca2b9ada_0f8b788… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/656914a7_e574e93… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bcf6c673_e353165… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/71c8ac66_c919c7d… :
PS3, Line 1745: I2CS_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f900e872_1993bf6… :
PS3, Line 1748: I2CS_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/154ca887_f95accb… :
PS3, Line 1751: I2CS_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c0178af4_b8c6ba0… :
PS3, Line 1754: I2CS_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/09a8e555_7dcaee5… :
PS3, Line 1757: I2CS_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b69c829f_954498f… :
PS3, Line 1760: I2CS_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9a29f64d_23ea251… :
PS3, Line 1763: I2CS_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fab90d5e_efd2cfc… :
PS3, Line 1766: I2CS_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e2f939ca_b8e83ec… :
PS3, Line 1781: if (status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e58cb3fd_6f98ffe… :
PS3, Line 1799: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d7fc331a_fc176dc… :
PS3, Line 1992: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/98e8d071_2798f96… :
PS3, Line 1992: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/969c8da6_1c2a5a9… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/08e50739_8da0174… :
PS3, Line 2006: * \internal Initalize i2c slave hardware
'Initalize' may be misspelled - perhaps 'Initialize'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3438eb41_001a4fe… :
PS3, Line 2015: if (i == -1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/443e9727_c8d2861… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0ae3ff82_d21e2d7… :
PS3, Line 2086: if (enabled) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fca40a38_8b32bfa… :
PS3, Line 2179: SERCOMSPI_REGS(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ba944a18_4dc68e2… :
PS3, Line 2182: SERCOMSPI_REGS(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c63e6d77_c496927… :
PS3, Line 2185: SERCOMSPI_REGS(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8b41c758_21a7116… :
PS3, Line 2188: SERCOMSPI_REGS(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c08557a3_603d7bd… :
PS3, Line 2191: SERCOMSPI_REGS(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b1f4fcde_b416c8e… :
PS3, Line 2194: SERCOMSPI_REGS(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1b05d862_4d86666… :
PS3, Line 2197: SERCOMSPI_REGS(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a1af5b60_c1b5795… :
PS3, Line 2200: SERCOMSPI_REGS(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b9fa34e8_36a3341… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8a5ce915_8bd6d73… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0acd0e81_b4e6960… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/894b482b_c3243c3… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b1e81f13_7b747fc… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2c9123bb_ff5859c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/df81682b_6785607… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5ba3a72e_49a8fbe… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d254b4ee_6bef9c8… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1a42ede2_1708970… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a0559766_4a39746… :
PS3, Line 2396: | (SERCOM_SPI_CTRLB_RXEN));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b006f232_3bf3d39… :
PS3, Line 2413: (regs->ctrlb & ~(SERCOM_SPI_CTRLB_MSSEN))
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c25266f6_b6c652c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7a3e749f_81982e6… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0e8f3273_1809473… :
PS3, Line 2648: if (regs == NULL) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5e464429_031ad7f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2b7efa03_c489968… :
PS3, Line 2689: if (rc < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/86a04814_18f3029… :
PS3, Line 2829: if (clk == 0 || baud == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/72820e9e_00fae25… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d5f7602d_e0bc716… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/81142201_310fbd0… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/60ed9a14_2a71374… :
PS3, Line 2949: if (ctrl->char_size > 1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6986b06f_82eaa15… :
PS3, Line 2961: uint16_t dummy)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c8842212_9033cd8… :
PS3, Line 2961: uint16_t dummy)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6da81781_9fe8d36… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5655f849_53681b0… :
PS3, Line 2998: void * hw = dev->prvt;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d396d192_36839fb… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e1ca1606_78c0d10… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c75467d8_13bc2f0… :
PS3, Line 3021: if (ctrl.rxcnt >= ctrl.txcnt) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3db189d8_5704f74… :
PS3, Line 3028: if (rc < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/834b10be_8c36316… :
PS3, Line 3048: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/db2c2c2e_772c831… :
PS3, Line 3069: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ecefcab1_0e1f7bd… :
PS3, Line 3087: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/58a185d6_5a2c7e8… :
PS3, Line 3150: const FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/662d5c32_9017170… :
PS3, Line 3150: const FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b7ebf311_44401b5… :
PS3, Line 3164: const FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/897e20d0_2dd7bd0… :
PS3, Line 3164: const FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/42b076dd_0075a4b… :
PS3, Line 3218: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0a30249a_6c5d8d6… :
PS3, Line 3218: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c14c1ec5_ae51c9b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dfdccbd7_b07b20e… :
PS3, Line 3235: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/590d156d_20fa91f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d64e897d_c169287… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/301a118f_e794a47… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/546f567a_9d2c391… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8a302c48_f88022d… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bc969597_6a173d6… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e6021a87_0f8e9cf… :
PS3, Line 44: while (!NVMCTRL->STATUS.bit.READY) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/baec7497_02927bd… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b92ffe6e_40f2f6b… :
PS3, Line 48: while (!NVMCTRL->STATUS.bit.READY) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6f41cbba_f7ff898… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f23d8d69_34f960f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/81e9e94a_e2643ce… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ec0391b3_f5a3832… :
PS3, Line 54: dst[i] = src[i];
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fee82271_85940ec… :
PS3, Line 54: dst[i] = src[i];
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/11efac67_dfad031… :
PS3, Line 55: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6da35598_9bed411… :
PS3, Line 57: NVMCTRL->ADDR.reg = address;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/db816d32_808ba1c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0062f717_67b44e7… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b87df682_a395d35… :
PS3, Line 59: while (!NVMCTRL->INTFLAG.bit.DONE) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e12906a8_0b014bf… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7278882f_9d35ae5… :
PS3, Line 60: while (!NVMCTRL->STATUS.bit.READY) {}
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d5bdd222_189d086… :
PS3, Line 63: __attribute__((section(".ramfunc"),OPTNONE))
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f864001e_1a6815e… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bc61e4bc_5c61848… :
PS3, Line 64: void do_update() {
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a9e23ced_cfc2f57… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1196c4ae_d21984e… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a12d663c_37de73c… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/71590dcf_84cd3ad… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/706641dc_3316604… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0bce4df5_dd50f1d… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/045a0277_6ceeecc… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c4e22385_e66b464… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f5c15f1d_1c9ccee… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/385bb886_7b5f48d… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a41d80de_1dd9b3f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/71082617_cde78b9… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f1714998_e931d3b… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9f3627bc_86ee7d8… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f2de0201_d59565a… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/69cda634_f22c6c7… :
PS3, Line 105: uint32_t address = 0;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f3e16efb_d8f5149… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/acba2dde_7983b37… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/21f407ae_233ac49… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dd7b767a_c00946e… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f5067eda_a7f22f2… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2de069c2_1426df0… :
PS3, Line 111: address += NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/39089e2f_cf669de… :
PS3, Line 111: address += NVMCTRL_PAGE_SIZE;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/209c5d04_93f1a22… :
PS3, Line 112: data_ptr += NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/43db17bf_096be9f… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/92b0b25d_fcc8ad4… :
PS3, Line 113: remaining -= NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d57cb61f_ec97117… :
PS3, Line 113: remaining -= NVMCTRL_PAGE_SIZE;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/132e55ef_e468d40… :
PS3, Line 114: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0a34717f_e9082ba… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1213b03b_4d06a83… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9a324a46_e66a1e7… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ce948b9e_a690ed5… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5eccf132_5f964c9… :
PS3, Line 123:
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/521b7465_6544c0e… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/674fe904_19b5a97… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5d350165_fd62401… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ae091a43_f25e9fe… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/35ea9043_568d25e… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4de5655b_8061024… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4519593a_6bb2435… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e70ef6e3_cdb2727… :
PS3, Line 33: static volatile uint32_t* dfu_magic = (uint32_t*)HSRAM_ADDR; // magic value should be written at start of RAM
"(foo*)" should be "(foo *)"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/40c7067c_3b42fb3… :
PS3, Line 33: static volatile uint32_t* dfu_magic = (uint32_t*)HSRAM_ADDR; // magic value should be written at start of RAM
"foo* bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f02e610a_bc01fdf… :
PS3, Line 109: static int str_to_usb_desc(char* in, uint8_t in_sz, uint8_t* out, uint8_t out_sz){
"foo* bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/095e3464_4edcb2c… :
PS3, Line 109: static int str_to_usb_desc(char* in, uint8_t in_sz, uint8_t* out, uint8_t out_sz){
"foo* bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d58a5918_d7bef9c… :
PS3, Line 109: static int str_to_usb_desc(char* in, uint8_t in_sz, uint8_t* out, uint8_t out_sz){
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b8435e8e_d61b401… :
PS3, Line 109: static int str_to_usb_desc(char* in, uint8_t in_sz, uint8_t* out, uint8_t out_sz){
space required before the open brace '{'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a1380c7e_f7826fb… :
PS3, Line 116: for (int i= 2; i < out_sz; i+=2)
spaces required around that '=' (ctx:VxW)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/55b6bfb3_87b98e2… :
PS3, Line 116: for (int i= 2; i < out_sz; i+=2)
spaces required around that '+=' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8c17b198_f18a6a5… :
PS3, 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-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/da431cf6_3fe2f2b… :
PS3, Line 138: if (hri_nvmctrl_read_STATUS_BOOTPROT_bf(FLASH_0.dev.hw) != 13) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21270):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a30606b5_7253201… :
PS3, Line 139: _user_area_write_bits((void*)NVMCTRL_FUSES_BOOTPROT_ADDR, NVMCTRL_FUSES_BOOTPROT_Pos, 13, 4);
"(foo*)" should be "(foo *)"
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I1403d35497cdea41ea5bed2c94b9e729ca352772
Gerrit-Change-Number: 39434
Gerrit-PatchSet: 3
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 18:39:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0
[XL] Change in osmo-asf4-dfu[master]: bl updater app
by Hoernchen
Attention is currently required from: Hoernchen.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder
Change subject: bl updater app
......................................................................
bl updater app
This builds a second binary that will be flashed like the main firmware
and will then update the bootloader, just like the st2. It additionaly
aligns the cmsis + config with the main app, and ensures the hw is
mostly completely initalized before the main app starts, which allows
skipping the init in the main fw.
Main changes:
* bl freq 12/48Mhz -> 120Mhz
* ext 32khz disabled
* "proper" init time for xosc
* no usb init unless DFU mode is actually used
* errata 2.13.1 clock init/lock fixups
* no guessing according to f++&%! bootprog flags where the main app is
Change-Id: I1403d35497cdea41ea5bed2c94b9e729ca352772
---
A CMSIS/Include/cmsis_version.h
M atmel_start.c
M atmel_start_pins.h
M config/hpl_cmcc_config.h
M config/hpl_gclk_config.h
M config/hpl_osc32kctrl_config.h
M config/hpl_oscctrl_config.h
A config/hpl_rtc_config.h
A config/hpl_sercom_config.h
M config/peripheral_clk_config.h
M gcc/Makefile
M gcc/system_same54.c
M hpl/core/hpl_core_m4.c
M hpl/oscctrl/hpl_oscctrl.c
A hpl/sercom/hpl_sercom.c
M usb_dfu_main.c
A usb_flash_main.c
M usb_start.c
18 files changed, 6,225 insertions(+), 303 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-asf4-dfu refs/changes/34/39434/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I1403d35497cdea41ea5bed2c94b9e729ca352772
Gerrit-Change-Number: 39434
Gerrit-PatchSet: 3
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
4 months, 3 weeks
- 1
- 0
[S] Change in osmo-asf4-dfu[master]: errata 2.6.10
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435?usp=email )
Change subject: errata 2.6.10
......................................................................
Patch Set 2:
(17 comments)
File gcc/gcc/startup_same54.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/f9513c64_380102b… :
PS2, Line 671: __attribute__((naked,noreturn)) void Reset_Handler(void)
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/b9e523d4_a7bbfa7… :
PS2, Line 675: __asm volatile (
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/9f3450e9_d3b59e8… :
PS2, Line 676: "movs r0, #0\n"
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/f1bc6e7a_2a27b9c… :
PS2, Line 676: "movs r0, #0\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/b9f4216f_a41c6e3… :
PS2, Line 677: "ldr r1, =0x40002000\n"
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/f4f6196e_f64c161… :
PS2, Line 677: "ldr r1, =0x40002000\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/0084195f_cb5e315… :
PS2, Line 678: "strb r0, [r1]\n"
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/95e8ba0a_acb7e4c… :
PS2, Line 678: "strb r0, [r1]\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/0ddf0129_0394e01… :
PS2, Line 679: "bl _Reset_Handler\n"
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/e22faafd_8a63f62… :
PS2, Line 679: "bl _Reset_Handler\n"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/12d30ccf_1ac897c… :
PS2, Line 680: ::: "r0", "r1", "memory"
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/61b684c9_b6b9cd1… :
PS2, Line 680: ::: "r0", "r1", "memory"
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/3457724e_e796768… :
PS2, Line 681: );
please, no spaces at the start of a line
File usb_flash_main.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/5c93adda_d7369bd… :
PS2, Line 136: bool startup_wdt_inactive = _user_area_read_bits((void*)NVMCTRL_USER, 62, 1) != 1;
"(foo*)" should be "(foo *)"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/f1567b46_2014376… :
PS2, Line 137: if (chiprev_lower_revG && startup_wdt_inactive)
that open brace { should be on the previous line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/eb80e406_f70db58… :
PS2, Line 139: _user_area_write_bits((void*)NVMCTRL_USER, 50, 0, 4);
"(foo*)" should be "(foo *)"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21267):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/07d9faaa_eca33b7… :
PS2, Line 140: _user_area_write_bits((void*)NVMCTRL_USER, 62, 1, 1);
"(foo*)" should be "(foo *)"
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: Ifbc6a3dfe91462029fce50ed42f20440debaa552
Gerrit-Change-Number: 39435
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 18:36:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0
[XL] Change in osmo-asf4-dfu[master]: bl updater app
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434?usp=email )
Change subject: bl updater app
......................................................................
Patch Set 2:
(313 comments)
File CMSIS/Include/cmsis_version.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0ca2d4ce_6c81d3f… :
PS2, Line 1: /**************************************************************************//**
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2c54571a_70d7b76… :
PS2, Line 2: * @file cmsis_version.h
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d88c7536_9085d70… :
PS2, Line 3: * @brief CMSIS Core(M) Version definitions
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/84acadd1_249671a… :
PS2, Line 4: * @version V5.0.2
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c9e610e5_fd7bbcf… :
PS2, Line 5: * @date 19. April 2017
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/53764735_54b1af2… :
PS2, Line 6: ******************************************************************************/
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9a9572ba_1984af9… :
PS2, Line 7: /*
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/afda904a_7be0e2b… :
PS2, Line 8: * Copyright (c) 2009-2017 ARM Limited. All rights reserved.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/92437c4a_16f0bcb… :
PS2, Line 9: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/005782fe_01ea9c7… :
PS2, Line 10: * SPDX-License-Identifier: Apache-2.0
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5ff3c6ba_3cfe795… :
PS2, Line 11: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/64c6daa7_12bf399… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/40ac3406_1d82945… :
PS2, Line 13: * not use this file except in compliance with the License.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a98e3695_0995dfb… :
PS2, Line 14: * You may obtain a copy of the License at
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/156008c0_e7ab63c… :
PS2, Line 15: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/50f60e8c_83ea577… :
PS2, Line 16: * www.apache.org/licenses/LICENSE-2.0
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/241f6239_a11d258… :
PS2, Line 17: *
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4594f12d_57cb9b1… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4a428b2e_8fbdead… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8cbaa82d_39c6a3e… :
PS2, Line 20: * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1c0402e8_e329806… :
PS2, Line 21: * See the License for the specific language governing permissions and
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d323e9d5_07597e2… :
PS2, Line 22: * limitations under the License.
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ec681fdc_88c4c5d… :
PS2, Line 23: */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/53240e91_8034204… :
PS2, Line 24:
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d1674773_3f29307… :
PS2, Line 25: #if defined ( __ICCARM__ )
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/020153c6_108bf91… :
PS2, Line 25: #if defined ( __ICCARM__ )
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/96fa4605_fb06876… :
PS2, Line 25: #if defined ( __ICCARM__ )
space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5f154243_720c2bb… :
PS2, Line 25: #if defined ( __ICCARM__ )
space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/033afc64_b6c284a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0a2e8a81_9717b0d… :
PS2, Line 27: #elif defined (__clang__)
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a20f3eb2_1f75d72… :
PS2, Line 27: #elif defined (__clang__)
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/737949c4_8cc7a30… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/770d78a6_37cf6bd… :
PS2, Line 29: #endif
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fbda05e8_6d7f2a3… :
PS2, Line 30:
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a79ec78c_9894b59… :
PS2, Line 31: #ifndef __CMSIS_VERSION_H
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/592e0418_5e2a77b… :
PS2, Line 32: #define __CMSIS_VERSION_H
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/720bf7db_f1ea17e… :
PS2, Line 33:
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e516cd91_ebfe816… :
PS2, Line 34: /* CMSIS Version definitions */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e2c23296_9fa6373… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/93fb13e0_f6aba12… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/da7bb9f5_324df56… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a5c652a5_2a75607… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0e445c47_856c54d… :
PS2, Line 37: #define __CM_CMSIS_VERSION ((__CM_CMSIS_VERSION_MAIN << 16U) | \
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b2d20370_57a3b3d… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fe501f20_cf82a04… :
PS2, Line 38: __CM_CMSIS_VERSION_SUB ) /*!< CMSIS Core(M) version number */
DOS line endings
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/255439e5_fc20f4a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/705d246d_c7d1846… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c01103e7_8a83fcd… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1be4bf1b_a74cbbd… :
PS2, Line 39: #endif
DOS line endings
File hpl/core/hpl_core_m4.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b5a8b215_1b58400… :
PS2, Line 220: __asm(".align 3 \n"
unnecessary whitespace before a quoted newline
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/722f584e_bef4fcc… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d953137f_a1579c4… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a3a1f9cc_5e072cf… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/eaf2b4a0_89db5bb… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/df96c864_d9c8054… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7eeaaaf9_243b8fa… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cd4372f3_e96c20f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/89633bec_b69b740… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3e810f96_205f797… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bedf83d0_72dab74… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/90267b52_53c1f3f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7e801a29_fe43093… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/705b8af6_9de50ef… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f7126e55_8600d1e… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/44cd90cb_ba821db… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/447c2225_a81bf27… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1d7e497a_b3e8701… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/577c4236_a15ff16… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/65cd211c_88ab91a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/49be78f4_b9133bc… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d1f5fd02_acc4202… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d36a6c5a_f44e1bc… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/74068ec0_b0c8add… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/933f18fd_9841e53… :
PS2, Line 140: SERCOM_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/02b21c12_c01c5b6… :
PS2, Line 143: SERCOM_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/08eb9b27_6d722bf… :
PS2, Line 146: SERCOM_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/28fd9485_8a683c4… :
PS2, Line 149: SERCOM_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4eaf4ef4_955076e… :
PS2, Line 152: SERCOM_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6f3bc176_f0a9d16… :
PS2, Line 155: SERCOM_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d86c0e46_fec163b… :
PS2, Line 158: SERCOM_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/72d2c3ae_5f1faa6… :
PS2, Line 161: SERCOM_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ec9913a4_33904ba… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8e9f1289_a028c25… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/604c91c6_ae85002… :
PS2, Line 218: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9cf188f5_7ffb17a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d1623e29_a74fb8f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a1fa46ad_d294a1a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/13049995_b66db08… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/98682062_8895005… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9325a2de_dd6f021… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/eeb21267_fb4c9ee… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9ea183d2_c260508… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c9aaf526_2132d18… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/21c2ee31_98adb36… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1ab50b6a_df8da02… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7238225e_bb251fe… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a94461e8_19b97d2… :
PS2, Line 565: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b890896f_4419499… :
PS2, Line 565: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/88a4bdf1_f9a653f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/24e7b620_e8c86dd… :
PS2, Line 641: if (hw == SERCOM0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fa518181_af863db… :
PS2, Line 645: if (hw == SERCOM1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/728f3989_27a64ef… :
PS2, Line 649: if (hw == SERCOM2) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a7cf1308_b62d5ee… :
PS2, Line 653: if (hw == SERCOM3) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2b9c88f9_9b90b98… :
PS2, Line 657: if (hw == SERCOM4) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cf3d8f6a_73098bf… :
PS2, Line 661: if (hw == SERCOM5) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6b8db720_5343628… :
PS2, Line 665: if (hw == SERCOM6) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bccb4634_e8a6d8a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/33fb7d40_65a22b6… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/80c9269f_75064ec… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a396e0e0_947d98d… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/83890881_291a089… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/985fd422_844f849… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/17770cd5_6901db8… :
PS2, Line 871: if (enabled) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0e823e5f_1daf5a7… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/34f364b2_3103a43… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3a94694d_363e059… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/812d6d8f_102c83e… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e96a0567_09d1d99… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bfe68e62_728c886… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a1908d22_221a14d… :
PS2, Line 966: I2CM_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/84a064fe_a4212c2… :
PS2, Line 969: I2CM_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a848f51d_2f569e2… :
PS2, Line 972: I2CM_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5e63309e_1dd548f… :
PS2, Line 975: I2CM_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c376f2e7_b4f4176… :
PS2, Line 978: I2CM_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4c4d00e4_cf6468d… :
PS2, Line 981: I2CM_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/942d5491_da312ab… :
PS2, Line 984: I2CM_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5e61e46a_75a24cb… :
PS2, Line 987: I2CM_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/faf21488_1be784d… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fa917922_4cfdac0… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3a7c9669_853b337… :
PS2, Line 1039: } else {
else is not generally useful after a break or return
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7dc36a25_46e6887… :
PS2, Line 1043: if (msg->len > 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0e1b8cc9_dae9d94… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/23915650_dc96e57… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/16c98160_2aa443f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/349071ed_897b0b3… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/04ed21d5_888e575… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/94cf4c0f_8368d1f… :
PS2, Line 1153: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/83be45a4_752e1ab… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f357de2b_091382c… :
PS2, Line 1164: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dc8bf8d6_f88e058… :
PS2, Line 1168: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/021df178_016ac78… :
PS2, Line 1203: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b8020d16_279bc76… :
PS2, Line 1240: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/daaa7d76_9b0c520… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1a21ea0a_c2862bf… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dfda8077_ca82c46… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6a220694_786ccac… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dcbfa2b3_6456eee… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/069b952f_b339857… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/113785da_75520c6… :
PS2, Line 1288: if (msg->len == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5da9ff9d_bada2aa… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/435c26ed_9a521e7… :
PS2, Line 1319: FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1799594f_22a0e6b… :
PS2, Line 1319: FUNC_PTR func)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/81f1e664_1cce6a1… :
PS2, Line 1345: * \retval I2C_OK Operation was successfull
'successfull' may be misspelled - perhaps 'successful'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/62c5a9b2_d6d437b… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/89c75dc1_86cf472… :
PS2, Line 1441: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/74532910_2808c3f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/47bc44e2_f84430f… :
PS2, Line 1452: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3136eabe_219354c… :
PS2, Line 1456: / (2 * baudrate));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/79a0c6e6_021d50d… :
PS2, Line 1473: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/17fed941_fc3fdf0… :
PS2, Line 1473: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/727a510e_7c3e407… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7963a8ca_ad1d673… :
PS2, Line 1496: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3908b27c_2593beb… :
PS2, Line 1501: if (timeout-- == 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9f2c4d52_8596bbc… :
PS2, Line 1516: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8d6aa1c9_f421ef4… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/23c1fe70_b513e57… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2f87f870_a31a78c… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ce314dcd_7291cd6… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/be8862ca_76058f6… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f10b4175_ce82b69… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3b07e53d_82e087e… :
PS2, Line 1562: void * hw = i2c_dev->hw;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/601cdcf7_e045059… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/06b1e712_79a330c… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8589710f_f8e5581… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fd5cbd57_100e8d3… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/72da752d_4dabec8… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6e29995e_9d2e3cf… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fd794899_ebe0c7c… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/dca221d3_4413f04… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9eceb208_aa30802… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/007ed652_c2e4885… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8fe6e03f_f06965b… :
PS2, Line 1745: I2CS_CONFIGURATION(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/62bccb71_ea58bb4… :
PS2, Line 1748: I2CS_CONFIGURATION(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2f4773ff_a1cc524… :
PS2, Line 1751: I2CS_CONFIGURATION(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7ee6bcbf_4f9726f… :
PS2, Line 1754: I2CS_CONFIGURATION(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c4de7d67_f00c444… :
PS2, Line 1757: I2CS_CONFIGURATION(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9316dc05_7aecda7… :
PS2, Line 1760: I2CS_CONFIGURATION(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f9350128_800f057… :
PS2, Line 1763: I2CS_CONFIGURATION(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/84e1cbef_3eea029… :
PS2, Line 1766: I2CS_CONFIGURATION(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/96d19d28_ff3d413… :
PS2, Line 1781: if (status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4416f253_5be6bf1… :
PS2, Line 1799: if (init_status) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/de95c970_e10d5ef… :
PS2, Line 1992: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/605798dc_5a96aff… :
PS2, Line 1992: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/af103543_dda6d80… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/845c6ef0_1407902… :
PS2, Line 2006: * \internal Initalize i2c slave hardware
'Initalize' may be misspelled - perhaps 'Initialize'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b1249637_6791fb8… :
PS2, Line 2015: if (i == -1) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d7a0af4a_f4c02ef… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6da1bb18_e6dbe18… :
PS2, Line 2086: if (enabled) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/20151c16_06ba4f2… :
PS2, Line 2179: SERCOMSPI_REGS(0),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b9565f5e_17b61ce… :
PS2, Line 2182: SERCOMSPI_REGS(1),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f4697365_e131338… :
PS2, Line 2185: SERCOMSPI_REGS(2),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b4857106_f48df8a… :
PS2, Line 2188: SERCOMSPI_REGS(3),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cde8c654_adba8f8… :
PS2, Line 2191: SERCOMSPI_REGS(4),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d3a09a18_012b171… :
PS2, Line 2194: SERCOMSPI_REGS(5),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4d3d8021_31d0e99… :
PS2, Line 2197: SERCOMSPI_REGS(6),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8ecf2e7e_327be13… :
PS2, Line 2200: SERCOMSPI_REGS(7),
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/387b8269_0e8d00a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5558b11e_accc9a0… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2a1f419d_ec95e46… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e1ad7d9b_24456c7… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b633b73b_f510e27… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b40670ca_2907f8c… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5382ba2b_c06a2d3… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/59bb2636_cee39a9… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f24ac921_efaddbd… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/02afb656_d07566b… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/87becf7a_eff22a8… :
PS2, Line 2396: | (SERCOM_SPI_CTRLB_RXEN));
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/77a82402_3711cd8… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f5b5d5ec_a0ca494… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cea8f7fb_df448ce… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f308bfd3_d83080d… :
PS2, Line 2648: if (regs == NULL) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6f170891_c909829… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/df61a410_cd90519… :
PS2, Line 2689: if (rc < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b1fe3294_44121d9… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/299a938e_1c508a5… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6803476f_26a4509… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b448b884_82460c1… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/22a9c2cf_ab8fd41… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/fecc9ac8_946bb94… :
PS2, Line 2961: uint16_t dummy)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/48d25162_7bafc75… :
PS2, Line 2961: uint16_t dummy)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1d58cf9f_c55bfd1… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/990bed68_1fd8d29… :
PS2, Line 2998: void * hw = dev->prvt;
"foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a3a712df_19a60ed… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/edaff38a_457b9e5… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0d89567d_d1c5ab6… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/243dbee2_27875c8… :
PS2, Line 3028: if (rc < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/bc51e6d6_db64721… :
PS2, Line 3048: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/084b4f77_37f4f7c… :
PS2, Line 3069: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/1dde575a_cb1c7b0… :
PS2, Line 3087: if (state) {
braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/450d424c_8307f08… :
PS2, Line 3150: const FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a09c9276_db0dd13… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/87d2f168_ef6440c… :
PS2, Line 3164: const FUNC_PTR func)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e3fcff2c_b3c670a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/637d2c5c_a656b97… :
PS2, Line 3218: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/188e1027_2d02918… :
PS2, Line 3218: const bool state)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e70317fc_3c57541… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/365c3b2d_74927c6… :
PS2, Line 3235: const bool state)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/8fa57859_f92d6e7… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/594f1e31_2451124… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/db6264a2_8d22ae4… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/88ca5d03_86edd46… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7cba6bcf_45a06af… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7584f63f_1dc11a4… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0bc8bde1_6a3e229… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a4243adf_7747a6d… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2fce7b48_2e9098c… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/abbade3e_e24e102… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2643c11f_c732a8f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0db8a225_08dac96… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/110c1119_2383b16… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a48e2cc1_83bab9e… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/40f0e7f3_326cb6a… :
PS2, Line 54: dst[i] = src[i];
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e223f7dc_e536935… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/72c966ff_314b71e… :
PS2, Line 55: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ca87132f_b33d237… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/75a61362_b0967d7… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5e6dc410_ba95bed… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3d947401_ce30efb… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/012694d0_5ce5729… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/07468335_9b7e8bf… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5fbd3fe1_4fb6a60… :
PS2, Line 63: __attribute__((section(".ramfunc"),OPTNONE))
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3ada82cb_7557049… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/671ece7d_d56a55f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/2f7b70a8_ff499a7… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/f2911e9a_af44176… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/487b6d28_ab7fc07… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d4f4a5dc_d4791d9… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/d372b872_6cd3971… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/34ec2d8a_1755a10… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/32f8a5db_fa65031… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6591c3ed_b1ecf6f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ee09eb67_b418376… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/cea859af_2a07d6e… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3120ebb4_2fe512f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/7556f17f_c3da5a2… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6a1d60a2_4c5949a… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/05f94fd0_395b2cf… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/b8f1c887_b5dd76d… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e1b23111_34ad038… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9855eb81_9b2cf72… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a1b50427_bb78250… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3fc49749_ab6fb10… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4eca39c2_49bdfe2… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/e78a0ed8_5efa0fd… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6f7f9bf7_07ddeee… :
PS2, Line 111: address += NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/51eeb06a_5692f97… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/5cc00b90_935951a… :
PS2, Line 112: data_ptr += NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c6026de2_3bc68c3… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/01b23803_35b92cf… :
PS2, Line 113: remaining -= NVMCTRL_PAGE_SIZE;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/52749f8b_f764dec… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/a4f0f839_7dc058d… :
PS2, Line 114: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/4d3b66b7_eb4291f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/13fe90b5_0f4cf6b… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3ecf64d5_dba1940… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/9ebab1bd_05def38… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/052cf409_c3e58dc… :
PS2, Line 123:
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21265):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/6d5f78cb_ea39dd4… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/ab38544d_cc06a29… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c6bddadf_b8648bf… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/3a0a2cf4_fd4887f… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/0b882f70_675bf5e… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/46a3785f_c1c1dd6… :
PS2, 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):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434/comment/c2faa28d_648f51c… :
PS2, Line 139: _user_area_write_bits((void*)NVMCTRL_FUSES_BOOTPROT_ADDR, NVMCTRL_FUSES_BOOTPROT_Pos, 13, 4);
"(foo*)" should be "(foo *)"
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39434?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I1403d35497cdea41ea5bed2c94b9e729ca352772
Gerrit-Change-Number: 39434
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 18:36:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0
[S] Change in osmo-asf4-dfu[master]: ver in usb prod desc
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39438?usp=email )
Change subject: ver in usb prod desc
......................................................................
Patch Set 2:
(2 comments)
File usb_flash_main.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21264):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39438/comment/063f2f8b_49f8d0c… :
PS2, Line 124: uint8_t __attribute__ ((aligned (16))) sernr_buf_descr[1+1+16*2*2];
space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21264):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39438/comment/5bc21f67_37d8717… :
PS2, Line 125: uint8_t __attribute__ ((aligned (16))) product_buf_descr[1+1+(sizeof(product_buf)-1)*2];
space prohibited between function name and open parenthesis '('
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39438?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I1f183695a3e458db2668e9950f859c936a821dc8
Gerrit-Change-Number: 39438
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 18:36:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0
[L] Change in osmo-asf4-dfu[master]: add fw update crcstub
by Jenkins Builder
Attention is currently required from: Hoernchen.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437?usp=email )
Change subject: add fw update crcstub
......................................................................
Patch Set 2:
(77 comments)
File crc_code.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/15f90874_759170a… :
PS2, Line 15: static uint32_t
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1aee2197_6d3c3e1… :
PS2, Line 17: uint32_t crc = 0xFFFFFFFF;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/37544bda_d177ad3… :
PS2, Line 18: for (uint32_t i = 0; i < length; i++) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/f99076b1_d7c5a84… :
PS2, Line 18: for (uint32_t i = 0; i < length; i++) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4e361820_f3884bb… :
PS2, Line 19: crc ^= data[i];
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/b0796bf9_63a9043… :
PS2, Line 20: for (uint8_t j = 0; j < 8; j++) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/c8c42d30_8705450… :
PS2, Line 20: for (uint8_t j = 0; j < 8; j++) {
suspect code indent for conditional statements (4, 6)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/98e338c0_2d82e5e… :
PS2, Line 20: for (uint8_t j = 0; j < 8; j++) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/2950362d_0c9b59a… :
PS2, Line 22: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/c480cb1b_4c03ae7… :
PS2, Line 23: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/949aaf98_fefe17b… :
PS2, Line 24: return ~crc;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/2781e21b_4e98752… :
PS2, Line 28: __attribute__((used, section(".crc_code"), OPTNONE)) void Reset_Handler(void) {
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/33a76edb_4ad3d1c… :
PS2, Line 29: extern uint32_t __CRC_Checksum;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/640a7939_d71b285… :
PS2, Line 30: extern uint32_t __CRC_Start;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/b67b240b_c822530… :
PS2, Line 31: extern uint32_t __CRC_End;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/28f1652c_027227c… :
PS2, Line 32: extern int main(void);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1f1efaec_c197923… :
PS2, Line 39: USB->DEVICE.CTRLA.reg = 0x1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/01f3505d_3bcbfdd… :
PS2, Line 41: uint32_t length = (uint32_t)&__CRC_End - (uint32_t)&__CRC_Start;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/43ff9bed_aed1e27… :
PS2, Line 42: uint32_t calculated_crc = calculate_crc((uint8_t *)&__CRC_Start, length);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/09d993cd_fbfcd1c… :
PS2, Line 43: uint32_t expected_crc = *(uint32_t *)&__CRC_Checksum;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/6d4d6128_a0f9378… :
PS2, Line 45: if (calculated_crc != expected_crc) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/8798a5eb_0d39adf… :
PS2, Line 45: if (calculated_crc != expected_crc) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d08eedd8_07c42ee… :
PS2, Line 46: static volatile uint32_t *dfu_magic = (uint32_t *)HSRAM_ADDR;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/e44a047e_0879926… :
PS2, Line 47: *dfu_magic = 0x44465521;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/8f50502b_796bab3… :
PS2, Line 50: __DSB();
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/48440387_b0a550d… :
PS2, Line 51: SCB->AIRCR = (uint32_t)((0x5FAUL << SCB_AIRCR_VECTKEY_Pos) | (SCB->AIRCR & SCB_AIRCR_PRIGROUP_Msk) |
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/0a0c66e9_a9e9ecf… :
PS2, Line 53: __DSB();
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/99e8b7d3_07bc07c… :
PS2, Line 54: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/51fcd2d7_dd7f7a9… :
PS2, Line 58: _Reset_Handler();
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/61621b19_a5d677b… :
PS2, Line 61: while (1)
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/f8341e38_f4a170a… :
PS2, Line 61: while (1)
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/41a2a2d7_f8c7490… :
PS2, Line 62: ;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/65a7c2d3_a2ff3b8… :
PS2, Line 73: int main(int argc, char *argv[]) {
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/aa21fe02_a07ca3c… :
PS2, Line 74: if (argc != 4) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/9aad9f89_ba74a2b… :
PS2, Line 74: if (argc != 4) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/4bdec083_6cd559d… :
PS2, Line 75: fprintf(stderr, "Usage: %s <binary_file> <start_offset_hex> <length_hex>\n", argv[0]);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/857b0761_f530fbb… :
PS2, Line 76: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/00ce415e_b775e93… :
PS2, Line 77: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/8a5e859b_c4c10e1… :
PS2, Line 79: const char *filename = argv[1];
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/638e2aa7_6e440d0… :
PS2, Line 82: uint32_t start_offset = strtoul(argv[2], NULL, 10);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/05d750f2_937b553… :
PS2, Line 83: uint32_t length = strtoul(argv[3], NULL, 10);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/57cb1b94_1e3cad2… :
PS2, Line 85: FILE *f = fopen(filename, "rb");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/7495e10b_df6a885… :
PS2, Line 86: if (!f) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/85dd1800_03334d1… :
PS2, Line 86: if (!f) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d20cf73e_ff11ed8… :
PS2, Line 87: perror("fopen");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/71471b9e_910875c… :
PS2, Line 88: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/2ea771c0_58c6daf… :
PS2, Line 89: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/85312c46_a42d0da… :
PS2, Line 92: if (fseek(f, (long)start_offset, SEEK_SET) != 0) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1b512bd7_0addadb… :
PS2, Line 92: if (fseek(f, (long)start_offset, SEEK_SET) != 0) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/93662ad6_1334fb6… :
PS2, Line 93: perror("fseek");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/a4491659_9b4566d… :
PS2, Line 94: fclose(f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/74a6977b_8cfeeb7… :
PS2, Line 95: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/17f4e3ed_d91787b… :
PS2, Line 96: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/0bc87f6b_6020282… :
PS2, Line 98: uint8_t *buf = (uint8_t *)malloc(length);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/b71cb30d_6750a6d… :
PS2, Line 99: if (!buf) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/ef32f1e1_d8c83f7… :
PS2, Line 99: if (!buf) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/adcf7292_73a548b… :
PS2, Line 100: fprintf(stderr, "Memory allocation error\n");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/53f93042_902c2b6… :
PS2, Line 101: fclose(f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1849fe4a_f13b234… :
PS2, Line 102: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/9858cfe6_648b412… :
PS2, Line 103: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/5be97164_a638dde… :
PS2, Line 105: size_t read_bytes = fread(buf, 1, length, f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/5b5f1c6c_a48e973… :
PS2, Line 106: fclose(f);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/7183d0d7_9f3b836… :
PS2, Line 108: if (read_bytes != length) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/a63b495a_221cc1f… :
PS2, Line 108: if (read_bytes != length) {
suspect code indent for conditional statements (2, 4)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/75150913_9ee913b… :
PS2, Line 109: fprintf(stderr, "Error reading file or not enough data\n");
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/987439a4_71dbd37… :
PS2, Line 110: free(buf);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/1d481fef_8e9e8b4… :
PS2, Line 111: return 1;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/9d08c2ef_a1d0b64… :
PS2, Line 112: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d241e30c_071b2b6… :
PS2, Line 114: uint32_t crc = calculate_crc(buf, length);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d4821dc4_bdad631… :
PS2, Line 115: free(buf);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/b432828e_c4b4518… :
PS2, Line 117: printf("%08X\n", crc);
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/9fb36046_2d0717c… :
PS2, Line 118: return 0;
please, no spaces at the start of a line
File gcc/Makefile:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/76b6db23_abc939f… :
PS2, Line 2: SHELL := /bin/bash
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/7b787867_ea4685e… :
PS2, Line 52: SRC_dfu = usb_dfu_main.c crc_code.c crc_params.c
trailing whitespace
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/d412d400_b4d14b9… :
PS2, Line 81: all: $(SUB_DIRS) hosttools dfu-merge
trailing whitespace
File gcc/gcc/startup_same54.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/c4e15b2c_5b6f513… :
PS2, Line 671: __attribute__((naked,noreturn,weak)) void Reset_Handler(void)
space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21263):
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/5dea1877_8c87020… :
PS2, Line 671: __attribute__((naked,noreturn,weak)) void Reset_Handler(void)
space required after that ',' (ctx:VxV)
--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I39eae7aaafd5531db6ce48837c9499432caadbed
Gerrit-Change-Number: 39437
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jan 2025 18:36:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
4 months, 3 weeks
- 1
- 0