This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.
Andreas Eversberg andreas at eversberg.euThomas Tsou wrote:
> 3GPP TS 05.03 "Channel coding" specifies the puncturing matrix (1,0,1)
> for class 1 information bits and tail bits valued u(0) to u(103) for a
> maximum puncturing index of 311. The puncturing index 313 exceeds the
> maximum index and causes osmo_conv_get_output_length() to output the
> improper length of 210 instead of 211.
>
> Signed-off-by: Thomas Tsou <tom at tsou.cc>
> ---
> src/osmo-bts-trx/gsm0503_conv.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/osmo-bts-trx/gsm0503_conv.c b/src/osmo-bts-trx/gsm0503_conv.c
> index 2a814ca..dc50e6c 100644
> --- a/src/osmo-bts-trx/gsm0503_conv.c
> +++ b/src/osmo-bts-trx/gsm0503_conv.c
> @@ -139,7 +139,7 @@ static const int conv_tch_hr_puncture[] = {
> 253, 256, 259, 262, 265, 268, 271, 274, 277, 280, 283,
>
> /* Tail bits */
> - 295, 298, 301, 304, 307, 310, 313,
> + 295, 298, 301, 304, 307, 310,
>
> /* End */
> -1,
hi thomas,
thanx for the patch. i blindly applied it, since it look reasonable but
i have no time to test/review it. it is in the branch jolly/trx_rebased.
please let me know if it works or not. did you test it with a half rate
v1 call?
best regards,
andreas