laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-abis/+/38323?usp=email )

Change subject: TRAU frame output EDATA: set sync pattern bits
......................................................................

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(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified




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 https://gerrit.osmocom.org/c/libosmo-abis/+/38323?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Iacd391cb6d389d8320c8ed767878eed144ee9d69
Gerrit-Change-Number: 38323
Gerrit-PatchSet: 2
Gerrit-Owner: falconia <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to