Change in osmo-pcu[master]: pdch: has_gprs_only_tb_attached: use m_num_tbfs

[osmith]

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/25401 )


Change subject: pdch: has_gprs_only_tb_attached: use m_num_tbfs
......................................................................

pdch: has_gprs_only_tb_attached: use m_num_tbfs

Make use of the separate EGPRS counters added in previous patch
I0c0a1121b4ae5f031782e7e63a0c28eb0b6c8b42 to shorten
has_gprs_only_tb_attached.

Related: SYS#4878
Change-Id: I1dd7df2c740ea604f07c65bebcb7c0051aebf9ae
---
M src/pdch.cpp
1 file changed, 1 insertion(+), 12 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/01/25401/1

diff --git a/src/pdch.cpp b/src/pdch.cpp
index 8b40d7a..3d386ae 100644
--- a/src/pdch.cpp
+++ b/src/pdch.cpp
@@ -1108,18 +1108,7 @@
 
 bool gprs_rlcmac_pdch::has_gprs_only_tbf_attached() const
 {
-	unsigned int i;
-	unsigned int j;
-	for (i = 0; i < sizeof(m_assigned_tfi[0]); i++) {
-		for (j = 0; j < 2; j++) {
-			if (m_assigned_tfi[j] & (1UL << i)) {
-				gprs_rlcmac_tbf *tbf = m_tbfs[j][i];
-				if (!tbf->is_egprs_enabled())
-					return true;
-			}
-		}
-	}
-	return false;
+	return m_num_tbfs[GPRS_RLCMAC_UL_TBF][TBF_TYPE_EGPRS] + m_num_tbfs[GPRS_RLCMAC_DL_TBF][TBF_TYPE_EGPRS] == 0;
 }
 
 void gprs_rlcmac_pdch::reserve(enum gprs_rlcmac_tbf_direction dir)

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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I1dd7df2c740ea604f07c65bebcb7c0051aebf9ae
Gerrit-Change-Number: 25401
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210908/92b87852/attachment.htm>