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 1:

(17 comments)

File gcc/gcc/startup_same54.c:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/c5813e67_567c6bd8... : PS1, Line 671: __attribute__((naked,noreturn)) void Reset_Handler(void) space required after that ',' (ctx:VxV)

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/93dff4cd_41772371... : PS1, Line 675: __asm volatile ( please, no spaces at the start of a line

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/9373fc81_8e64a375... : PS1, Line 676: "movs r0, #0\n" code indent should use tabs where possible

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/2b7d17c3_0779c8e8... : PS1, Line 676: "movs r0, #0\n" please, no spaces at the start of a line

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/6816a398_4f7c4179... : PS1, Line 677: "ldr r1, =0x40002000\n" code indent should use tabs where possible

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/8a9b5a11_e37c0aad... : PS1, Line 677: "ldr r1, =0x40002000\n" please, no spaces at the start of a line

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/40b45f7c_3d22779e... : PS1, Line 678: "strb r0, [r1]\n" code indent should use tabs where possible

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/0574c5d5_fc5a4fb9... : PS1, Line 678: "strb r0, [r1]\n" please, no spaces at the start of a line

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/f55a6fab_770c5585... : PS1, Line 679: "bl _Reset_Handler\n" code indent should use tabs where possible

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/2365f18b_5a863eb7... : PS1, Line 679: "bl _Reset_Handler\n" please, no spaces at the start of a line

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/d710b350_a601a77f... : PS1, Line 680: ::: "r0", "r1", "memory" code indent should use tabs where possible

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/248d2c37_893c0f19... : PS1, Line 680: ::: "r0", "r1", "memory" please, no spaces at the start of a line

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/5fa96671_e7967db9... : PS1, 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-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/af46e7a9_7fd82282... : PS1, 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-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/eeb8c7f1_f3a89240... : PS1, 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-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/b6f308b1_a1722608... : PS1, 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-21260): https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39435/comment/3d45facf_e307e61e... : PS1, Line 140: _user_area_write_bits((void*)NVMCTRL_USER, 62, 1, 1); "(foo*)" should be "(foo *)"