Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34492?usp=email )
Change subject: ASCI: Add group receive and transmit mode support to MM layer
......................................................................
Patch Set 1:
(4 comments)
File src/host/layer23/src/mobile/gsm48_mm.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11171):
https://gerrit.osmocom.org/c/osmocom-bb/+/34492/comment/c83585cb_7a20f258
PS1, Line 277: * completely independant from the state of the MM layer.
'independant' may be misspelled - perhaps 'independent'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11171):
https://gerrit.osmocom.org/c/osmocom-bb/+/34492/comment/ccc37e3f_ad5f5bb6
PS1, Line 3822: msg_type = (mm->vgcs_group_call) ? GSM48_MMGCC_GROUP_CNF: GSM48_MMBCC_GROUP_CNF;
spaces required around that ':' (ctx:VxW)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11171):
https://gerrit.osmocom.org/c/osmocom-bb/+/34492/comment/99c5b470_08d4b251
PS1, Line 3967: msg_type = (mm->vgcs_group_call) ? GSM48_MMGCC_UPLINK_CNF: GSM48_MMBCC_UPLINK_CNF;
spaces required around that ':' (ctx:VxW)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11171):
https://gerrit.osmocom.org/c/osmocom-bb/+/34492/comment/422042c4_38c79d66
PS1, Line 3993: msg_type = (mm->vgcs_group_call) ? GSM48_MMGCC_UPLINK_REL_IND: GSM48_MMBCC_UPLINK_REL_IND;
spaces required around that ':' (ctx:VxW)
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34492?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I05957182a57423ad947ab200b52f65fde859e110
Gerrit-Change-Number: 34492
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 21 Sep 2023 09:39:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34473?usp=email )
Change subject: HACK, DON'T MERGE! strncasecmp for firmware
......................................................................
Patch Set 1:
(1 comment)
File src/target/firmware/lib/strcasecmp.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11169):
https://gerrit.osmocom.org/c/osmocom-bb/+/34473/comment/038d14c2_4cbed01e
PS1, Line 12: if (*s1++=='\0')
spaces required around that '==' (ctx:OxV)
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34473?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ia8c201fcfcc6ad1766dcff032f0cac76f53f2fb3
Gerrit-Change-Number: 34473
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 21 Sep 2023 09:39:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34491?usp=email )
Change subject: ASCI: Add group receive and transmit mode support to RR layer
......................................................................
Patch Set 1:
(5 comments)
File src/host/layer23/src/mobile/gsm48_rr.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11164):
https://gerrit.osmocom.org/c/osmocom-bb/+/34491/comment/0a85918b_6c805ed3
PS1, Line 510: while((msg = msgb_dequeue(&rr->downqueue)))
space required before the open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11164):
https://gerrit.osmocom.org/c/osmocom-bb/+/34491/comment/181b3ab1_a88dd160
PS1, Line 1529: if (bitvec_get_bit_high(&bv) == H) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11164):
https://gerrit.osmocom.org/c/osmocom-bb/+/34491/comment/47e0d30d_0665a654
PS1, Line 1963: /* Note: Other informations are not used. */
'informations' may be misspelled - perhaps 'information'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11164):
https://gerrit.osmocom.org/c/osmocom-bb/+/34491/comment/2be13072_755c2368
PS1, Line 5501: reject:
labels should not be indented
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11164):
https://gerrit.osmocom.org/c/osmocom-bb/+/34491/comment/a15c330a_469edcb7
PS1, Line 6389: /* Leave uplink, wait for uplink beeing free, channel release or channel/link failure. */
'beeing' may be misspelled - perhaps 'being'?
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34491?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ia55c182a1b4cde777a0e16dcfe0cd32e0f2e38eb
Gerrit-Change-Number: 34491
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 21 Sep 2023 09:39:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34479?usp=email )
Change subject: ASCI: Add a flag to turn transmitter off or on
......................................................................
Patch Set 1:
(2 comments)
File src/host/layer23/src/common/l1ctl.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11163):
https://gerrit.osmocom.org/c/osmocom-bb/+/34479/comment/cde6d862_9bff186e
PS1, Line 569: uint8_t chan_nr, uint8_t tsc, uint8_t tch_mode, uint8_t audio_mode, uint8_t tch_flags)
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11163):
https://gerrit.osmocom.org/c/osmocom-bb/+/34479/comment/5a8a8034_34cb028a
PS1, Line 569: uint8_t chan_nr, uint8_t tsc, uint8_t tch_mode, uint8_t audio_mode, uint8_t tch_flags)
please, no spaces at the start of a line
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34479?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I20133523adc3b204cd2181bfe664fe66020a10e3
Gerrit-Change-Number: 34479
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 21 Sep 2023 09:39:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment