osmo-bts[master]: DTX DL: use FSM for AMR

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/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Wed Oct 12 23:38:02 UTC 2016


Patch Set 6:

(1 comment)

It looks like not too much effort to first fix the regression in a separate commit, and only then do the DTX re-implementation. It was just the couple of 'if (lchan->ts->trx->bts->dtxd)' inserted, right?

Well, the effort might not be necessary when this is anyway basically a completely new implementation of DTX. Harald/Holger, what are your opinions?

https://gerrit.osmocom.org/#/c/1030/6//COMMIT_MSG
Commit Message:

Line 14: - sync with corresponding changes in OpenBSC's
what "corresponding changes"? I don't enjoy having to be intelligent when reading commit logs...


-- 
To view, visit https://gerrit.osmocom.org/1030
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I74a0b42cb34d525b8a70d264135e82994ca70d31
Gerrit-PatchSet: 6
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-HasComments: Yes



More information about the gerrit-log mailing list