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/.
laforge gerrit-no-reply at lists.osmocom.orglaforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/14646 ) Change subject: Transceiver: Support SETFORMAT command ...................................................................... Patch Set 1: Code-Review+1 (1 comment) https://gerrit.osmocom.org/#/c/14646/1/Transceiver52M/Transceiver.cpp File Transceiver52M/Transceiver.cpp: https://gerrit.osmocom.org/#/c/14646/1/Transceiver52M/Transceiver.cpp@872 PS1, Line 872: if (version_recv > TRX_DATA_FORMAT_VER) : version_used = TRX_DATA_FORMAT_VER; so if somebody requests version 3, but the TRX can only do 0, 1 and 2, then version 2 will be selected? I'm not sure if this is the semantics we'd like? One could also argue that the existing version (whatever is mVersionTRXD at that point) should be retained if an unsupported version is received. It just looks a bit strange to me if "SETVERSION 3" can still succeed even if '2' is chosen. A lazy client might just check for success/failure -> boom. -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/14646 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5786dd44b076202c6f1a6e82405670e8605797ed Gerrit-Change-Number: 14646 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge at gnumonks.org> Gerrit-Comment-Date: Tue, 02 Jul 2019 04:45:35 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190702/b4ec91a2/attachment.htm>