Attention is currently required from: fixeria, jolly.

pespin has posted comments on this change by jolly. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/41361?usp=email )

Change subject: Add ER-GSM band support
......................................................................


Patch Set 1:

(2 comments)

File src/osmo-bsc/system_information.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/41361/comment/b6cafa78_612bde93?usp=email
 :
PS1, Line 548:  /* Check presence of R-GSM / E-GSM ARFCNs 940..1023 */
(E)R-GSM


https://gerrit.osmocom.org/c/osmo-bsc/+/41361/comment/e12a0208_9fda10f3?usp=email
 :
PS1, Line 583:          if (i >= 940 && min < 940)
Not mandatory, but sounds like we may want to have all 940 hardcoded in this 
function into some "const unsigned int min_arfcn = 940 ;)



--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/41361?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I0a9a2f64d9db65967699453c26ae282c4ddf4558
Gerrit-Change-Number: 41361
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: jolly <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Thu, 06 Nov 2025 15:13:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to