Attention is currently required from: jolly.
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33611 )
Change subject: ASCI: Add FSM for VGCS/VBS ......................................................................
Patch Set 2:
(4 comments)
File include/osmocom/bsc/vgcs_fsm.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/6ada7a28_7de88b8f PS2, Line 59: /* Channel link failed. (Talker faild.) */ 'faild' may be misspelled - perhaps 'failed'?
File src/osmo-bsc/vgcs_fsm.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/5aaafc56_77d02d48 PS2, Line 373: LOG_CALL(conn, LOGL_DEBUG, "Calling subscriber assiged and now on uplink.\n"); 'assiged' may be misspelled - perhaps 'assigned'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/f64aba1b_6cd34c9d PS2, Line 668: no_aoip: labels should not be indented
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/cb4735c0_c12579b8 PS2, Line 1155: strcpy(conn->vgcs_chan.msc_rtp_addr, req.msc_rtp_addr); Instead of strncpy()/strcpy(), use osmo_strlcpy() or OSMO_STRLCPY_ARRAY()
-
Jenkins Builder