Jenkins Builder has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/32093 )
Change subject: Replace explicit gsm_lchan_name() calls with LOGPLCHAN
......................................................................
Patch Set 1:
(4 comments)
File src/common/rsl.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5484):
https://gerrit.osmocom.org/c/osmo-bts/+/32093/comment/be654ccb_6fd417be
PS1, Line 3298: gsm_pchan_name(ts->pchan), pdch_act? "PDCH" :
"TCH/F", ts->flags);
spaces required around that '?' (ctx:VxW)
File src/osmo-bts-lc15/oml.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5484):
https://gerrit.osmocom.org/c/osmo-bts/+/32093/comment/ba488bd5_770ecb3d
PS1, Line 937: LOGPLCHAN(lchan, DL1C, LOGL_INFO, "%s tch_mode=0x%02x\n",
__FUNCTION__, lchan->tch_mode);
__func__ should be used instead of gcc specific __FUNCTION__
File src/osmo-bts-oc2g/oml.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5484):
https://gerrit.osmocom.org/c/osmo-bts/+/32093/comment/4ba0608a_8b1db3b7
PS1, Line 952: LOGPLCHAN(lchan, DL1C, LOGL_INFO, "%s tch_mode=0x%02x\n",
__FUNCTION__, lchan->tch_mode);
__func__ should be used instead of gcc specific __FUNCTION__
File src/osmo-bts-sysmo/oml.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5484):
https://gerrit.osmocom.org/c/osmo-bts/+/32093/comment/a551c51d_46b38895
PS1, Line 945: __FUNCTION__, lchan->tch_mode);
__func__ should be used instead of gcc specific __FUNCTION__
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bts/+/32093
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: If4f4f555f5ca61dfa624b298805f5375efc0b137
Gerrit-Change-Number: 32093
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Tue, 28 Mar 2023 01:29:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment