Attention is currently required from: Hoernchen, Timur Davydov, fixeria, 
laforge, osmith.

pespin has posted comments on this change by Timur Davydov. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email )

Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr)
......................................................................


Patch Set 7:

(5 comments)

File Transceiver52M/device/usdr/USDRDevice.h:

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/b35a4c7c_3fb1ea8b?usp=email
 :
PS4, Line 12:     This program is distributed in the hope that it will be 
useful,
> Fixed indentation.
Done


File Transceiver52M/device/usdr/USDRDevice.cpp:

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/a4dd6af4_b9045e5d?usp=email
 :
PS4, Line 475: std::string USDRDevice::getTxAntenna(size_t chan )
> Now removed :)
Done


File debian/osmo-trx-usdr.install:

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/1f38ce73_bf0c0523?usp=email
 :
PS4, Line 3: /usr/bin/osmo-trx-usdr
> Removed the leading slashes to make it consistent with the other `*.install` 
> files.
Done


File doc/manuals/chapters/trx-backends.adoc:

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/70aa0278_2ca91958?usp=email
 :
PS3, Line 86: where multiple small SDRs are combined to present one or more TRX 
instances
> Correct. So far, I've only tested one `osmo-trx-usdr` instance with one 
> `osmo-bts-trx`. […]
An osmo-trx can serve multiple osmo-bts-trx transceivers (TRX) (see "chan" VTY 
ndoe and or "multi-arfcn enable") in the same BTS.

You cannot serve the same BTS from multiple osmo-trx processes nor service 
multiple BTS from the same osmot-rx process.


File doc/manuals/chapters/trx-devices.adoc:

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/efce6c50_9f4d7ab6?usp=email
 :
PS3, Line 102: scale the number of TRX instances.
> I’ve adjusted the documentation in this file.
Done



--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df
Gerrit-Change-Number: 42198
Gerrit-PatchSet: 7
Gerrit-Owner: Timur Davydov <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: Hoernchen <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-CC: osmith <[email protected]>
Gerrit-Attention: osmith <[email protected]>
Gerrit-Attention: Hoernchen <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Attention: Timur Davydov <[email protected]>
Gerrit-Comment-Date: Mon, 02 Mar 2026 15:09:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <[email protected]>
Comment-In-Reply-To: pespin <[email protected]>
Comment-In-Reply-To: Timur Davydov <[email protected]>

Reply via email to