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

Change subject: trau_frame.c cosmetic: fix misleading comment ......................................................................

trau_frame.c cosmetic: fix misleading comment

The 4 static const ubit_t arrays ft_hr_up_bits[5], ft_hr_down_bits[5], ft_data_hr_up_bits[5] and ft_data_hr_down_bits[5] contain frame type bit patterns that are defined in GSM 08.61 for half-rate speech and data channels with 16 kbit/s Abis transport, and are used as such in the code. However, the comment next to them stated incorrectly that these frame type bit patterns are for 8 kbit/s submultiplexing - fix it.

Change-Id: Ic2f818a7f8bab4e82f239464511ab430906d4680 --- M src/trau/trau_frame.c 1 file changed, 19 insertions(+), 2 deletions(-)

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

diff --git a/src/trau/trau_frame.c b/src/trau/trau_frame.c index 9c9baa7..0be005a 100644 --- a/src/trau/trau_frame.c +++ b/src/trau/trau_frame.c @@ -82,7 +82,7 @@ TS48060_AMR_FC_NO_SPEECH = 0x0, };

-/* 16k sub-slots */ +/* GSM 08.60 frame types, 16k sub-slots */ static const ubit_t ft_fr_up_bits[5] = { 0, 0, 0, 1, 0 }; static const ubit_t ft_fr_down_bits[5] = { 1, 1, 1, 0, 0 }; static const ubit_t ft_data_up_bits[5] = { 0, 1, 0, 0, 0 }; @@ -96,7 +96,7 @@ static const ubit_t ft_d145s_bits[5] = { 1, 0, 1, 0, 0 }; static const ubit_t ft_edata_bits[5] = { 1, 1, 1, 1, 1 };

-/* 8k sub-slots */ +/* GSM 08.61 frame types, still on 16k sub-slots */ static const ubit_t ft_hr_up_bits[5] = { 0, 0, 0, 1, 1 }; static const ubit_t ft_hr_down_bits[5] = { 1, 1, 1, 0, 1 }; static const ubit_t ft_data_hr_up_bits[5] = { 0, 1, 0, 0, 1 };