Attention is currently required from: laforge, pespin.
Hoernchen has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-trx/+/33930 )
Change subject: transceiver: add experimental viterbi equalizer support
......................................................................
Patch Set 11:
(1 comment)
File CommonLibs/trx_vty.c:
https://gerrit.osmocom.org/c/osmo-trx/+/33930/comment/ad2d4b72_c860704a
PS10, Line 344: "Use viterbi equalizer for gmsk (default=disable)\n"
if it's intended to be only for GSMK and not 8PSK,
does it makes sense to add such part in the comma […]
Not really, there is already
code that checks the config and bails if the setting does not match. It can be extended to
work for 8psk, too, but that is not required for the current proejct.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-trx/+/33930
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I9d7a4ff72e323832a94d885d5714fcde01ceeb3d
Gerrit-Change-Number: 33930
Gerrit-PatchSet: 11
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 27 Jul 2023 12:05:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment