<p style="white-space: pre-wrap; word-wrap: break-word;"><br>> the sysmoBTS PHY (just like the PHY of the litecell 1.5 and the oc2g)<br>> still have the bit ordering as you discovered it back then.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But there is code to 'fix' that in osmo-bts itself osmo-bts-sysmo/tch.c</p><p style="white-space: pre-wrap; word-wrap: break-word;">And IMHO that's where it belongs. That weird format shouldn't appear on the wire ever.</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br>> It may be that the ETSI TS is using the same bit-ordering as the new RFC? In that case it would really be a bug.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, both the ETSI TS spec and the new RFC have the same bit ordering.<br>They both point to the same spec for the order of the parameter. Then one says "The first bit (b1) of the first parameter is placed in the most significant bit (MSB) (bit 0) of the first octet (octet 1) of the payload field;"  and the other says "Within the GSM 06.20 codec parameter bits are numbered in big-endian manner." (i.e. MSB first), so both the same thing IMHO. (and really, the opposite makes no sense at all)</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br>> However, the additional 1-byte TOC definitely is a difference between the ETSI HR RTP payload and the RFC5593.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes indeed the only difference between the two formats is the TOC byte.</p><p><a href="https://gerrit.osmocom.org/12979">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12979">change 12979</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/12979"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Iceef19e5619f8c92dfa7c8cdecb2e9b15f0a11a1 </div>
<div style="display:none"> Gerrit-Change-Number: 12979 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 20 Feb 2019 21:15:27 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>