Change in osmo-trx[master]: add support for xtrx

Pau Espin Pedrol gerrit-no-reply at
Mon May 13 16:27:58 UTC 2019

Pau Espin Pedrol has posted comments on this change. ( )

Change subject: add support for xtrx

Patch Set 1:


Some comments after quick review.
File Transceiver52M/device/xtrx/XTRXDevice.cpp:
PS1, Line 44: 	LOG(INFO) << "creating XTRX device:"
You should be using category DEV here, not MAIN (see other devices).
PS1, Line 162: 	return NORMAL;
This NORMAL thing here makes no sense in the context of this function afaik.
PS1, Line 167: 	if (device) {
Drop {}
PS1, Line 175: 	if (started) {
Drop {}
PS1, Line 212: 	if (loopback) {
Drop {}
PS1, Line 231: 	if (started) {
early return: if (!started) reutrn false;
PS1, Line 278: 	LOG(NOTICE) << "Setting TX gain to " << dB << " dB.";
PS1, Line 282: 		LOG(ERR) << "Error setting TX gain res: " << res;
LOGCHAN. LOGCHAN everywhere where "chan" is passed as parameter in this file.
PS1, Line 319: 	int res = xtrx_recv_sync_ex(device, &ri);
Looks like we need to add smpl_buf usage in here (see my latest commits merged).
File debian/osmo-trx-xtrx.install:
PS1, Line 3: /usr/bin/osmo-trx-xtrx
why some strt with / and some doesn't? Unify style if possible.
File doc/examples/osmo-trx-xtrx/osmo-trx-xtrx.cfg:
PS1, Line 21:   tx-path BAND1
Are you sure this belongs here? looks copied from LMS file.

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iad73e0611e7951d5bcfcc918063cc3778cb1dd8f
Gerrit-Change-Number: 14018
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis at>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-CC: Pau Espin Pedrol <pespin at>
Gerrit-Comment-Date: Mon, 13 May 2019 16:27:58 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list