path: root/src/osmo-bts-sysmo/l1_if.c
diff options
authorHarald Welte <laforge@gnumonks.org>2017-05-28 16:00:36 +0200
committerHarald Welte <laforge@gnumonks.org>2017-05-29 21:58:51 +0200
commita8bf666a099ecc7bb8436f31b7f30e246ef50015 (patch)
treedad283fd0790c52f1442f1e34c6dacecade25543 /src/osmo-bts-sysmo/l1_if.c
parentcb5a969a4548ec70ff16a74aa834cb5ebf50dc0a (diff)
sysmobts: Re-order the bit-endianness of every HR codec parameter
The so-called "RTP mode" of the DSP contains a bug on all firmware versions < 5.3.3 which causes the bit-order within each of the non-aligned codec parameters to be wrong. Introduce a function originally written by Sylvain Munaut during 32C3 in http://git.osmocom.org/openbsc/commit/?h=sylvain/32c3_codec&id=5b4a16bbe93a7b1ace65cc23d6cce56ecf4f1275 to bring the bits into [the correct] order. This has never been seen in a "pure sysmoBTS" setup, as all BTSs would use the same (wrong) bit-ordering and thus interoperate. This patch now checks for an affected DSP firmware version and then jumbles (old DSP firmware version) or does nothing (new DSP firmware version). Change-Id: Ia0eee4f514fb8fd81c052bb44a4facba898d6afb Closes: SYS#2452
Diffstat (limited to 'src/osmo-bts-sysmo/l1_if.c')
1 files changed, 3 insertions, 0 deletions
diff --git a/src/osmo-bts-sysmo/l1_if.c b/src/osmo-bts-sysmo/l1_if.c
index 8820f083..60638e18 100644
--- a/src/osmo-bts-sysmo/l1_if.c
+++ b/src/osmo-bts-sysmo/l1_if.c
@@ -1460,6 +1460,9 @@ static int info_compl_cb(struct gsm_bts_trx *trx, struct msgb *resp,
LOGP(DL1C, LOGL_FATAL, "BTS band %s not supported by hw\n",
+ if (l1if_dsp_ver(fl1h) < L1_VER_SHIFT(5,3,3))
+ fl1h->rtp_hr_jumble_needed = true;
/* Request the activation */
l1if_activate_rf(fl1h, 1);