Attention is currently required from: dexter, pespin.
Hello pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-pcu/+/34575?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by pespin
Change subject: gprs_rlcmac_sched: check if we really use direct phy
......................................................................
gprs_rlcmac_sched: check if we really use direct phy
In gprs_rlcmac_sched we have a flag to skip idle frames in case we do not
use the so called "direct phy access". A similar mechanism also exists
in pcu_l1_if.cpp in function pcu_rx_rts_req_ptcch().
Unfortunately the check is not done correctly. The flag gets set when
the ENABLE_DIRECT_PHY define constant is set. However, this does not say
much about whether we access a PHY directly or not. The define constant
is intended to be used to disable direct phy related code in
on platforms where no direct phy code is used or cannot be used.
In order to know if we actually accessing a PHY directly we must check
the presence of fl1h on the related TRX. (see also pcu_l1_if.cpp,
function pcu_l1if_tx_pdtch)
Related: OS#6191
Change-Id: I0808950b1154bbb9a789c3f706ad9fb6618764ec
---
M src/gprs_rlcmac_sched.cpp
M src/pcu_l1_if.cpp
2 files changed, 34 insertions(+), 6 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/75/34575/2
--
To view, visit
https://gerrit.osmocom.org/c/osmo-pcu/+/34575?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I0808950b1154bbb9a789c3f706ad9fb6618764ec
Gerrit-Change-Number: 34575
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newpatchset