laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
TRAU frame output EDATA: set sync pattern bits

osmo_trau_frame_encode() for OSMO_TRAU16_FT_EDATA previously failed
to set the sync pattern bits in the generated output. Fix it.

Change-Id: Iacd391cb6d389d8320c8ed767878eed144ee9d69
---
M src/trau/trau_frame.c
1 file changed, 4 insertions(+), 0 deletions(-)

diff --git a/src/trau/trau_frame.c b/src/trau/trau_frame.c
index 43b0e6b..88d0031 100644
--- a/src/trau/trau_frame.c
+++ b/src/trau/trau_frame.c
@@ -676,6 +676,10 @@
else
cbits5 = ft_edata_bits;

+ /* sync pattern */
+ memset(trau_bits, 0, 16);
+ trau_bits[16] = 1;
+
/* C1 .. C5 */
memcpy(trau_bits + 17, cbits5, 5);


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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Iacd391cb6d389d8320c8ed767878eed144ee9d69
Gerrit-Change-Number: 38323
Gerrit-PatchSet: 2
Gerrit-Owner: falconia <falcon@freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>