pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/15656 )

Change subject: scheduler: fix handling of PTCCH/U and PTCCH/D logical channels
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

https://gerrit.osmocom.org/#/c/15656/1/src/common/l1sap.c
File src/common/l1sap.c:

https://gerrit.osmocom.org/#/c/15656/1/src/common/l1sap.c@1320
PS1, Line 1320:                 pcu_tx_rach_ind(bts, 
rach_ind->acc_delay_256bits >> 6,
> Because 6 is a power of 2: 2 ^ 6 is 64. […]
Yes I know that part, but not why 64. But now I figured out seeing the other 
commit. 256/64 -> 4 so ranges 0..4



--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/15656
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I232e5f514fbad2c51daaa59ff516004aba97c8a3
Gerrit-Change-Number: 15656
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilira...@gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilira...@gmail.com>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lyn...@fe80.eu>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-Comment-Date: Wed, 02 Oct 2019 16:54:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: fixeria <axilira...@gmail.com>
Comment-In-Reply-To: pespin <pes...@sysmocom.de>
Gerrit-MessageType: comment

Reply via email to