dexter submitted this change.

View Change


Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
trau_pcu_ericsson: cosmetic: set hdr_good flag explictly to false

It is not necessary but we also set the data_good flags explicitly to
false so lets to the same with hdr_good

Change-Id: Ic710f7e63f4b7e16a3cbc9eb8bbf9ae1c7c5cb26
Related: OS#5198
---
M src/trau/trau_pcu_ericsson.c
1 file changed, 15 insertions(+), 0 deletions(-)

diff --git a/src/trau/trau_pcu_ericsson.c b/src/trau/trau_pcu_ericsson.c
index 0b0477b..2f2bf0e 100644
--- a/src/trau/trau_pcu_ericsson.c
+++ b/src/trau/trau_pcu_ericsson.c
@@ -1671,6 +1671,8 @@
if (trau_bits[98] == 0)
ind->u.egprs.data_good[1] = true;
} else {
+ ind->u.egprs.hdr_good = false;
+
/* A bad RLC/MAC header always means that the the data blocks
* cannot be valid. */
ind->u.egprs.data_good[0] = false;

To view, visit change 31441. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Ic710f7e63f4b7e16a3cbc9eb8bbf9ae1c7c5cb26
Gerrit-Change-Number: 31441
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged