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

Timur Davydov 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 6:

(4 comments)

File Transceiver52M/device/usdr/USDRDevice.h:

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

and the indentation still differs from the the lines above.

Fixed indentation.

File Transceiver52M/device/usdr/USDRDevice.cpp:

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/8a485113_237ed47f?usp=... : PS4, Line 475: std::string USDRDevice::getTxAntenna(size_t chan )

Not removed ;)

Now removed :)

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

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/940f6378_bb71d198?usp=... : PS3, Line 86: where multiple small SDRs are combined to present one or more TRX instances

SO IIUC that means you didn't actually yet try/succeed to run multi-trx BTS with these devices, corr […]

Correct. So far, I've only tested one `osmo-trx-usdr` instance with one `osmo-bts-trx`. Initially, I assumed that one `osmo-bts-trx` could connect to multiple `osmo-trx-usdr` instances (one for each SDR), based on the structure of the `trx` configuration in `osmo-bts-trx`, where `trx` has an index, so it's possible to create multiple `trx` instance configurations.

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

https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/9a0f3ddd_631f53df?usp=... : PS3, Line 102: scale the number of TRX instances.

As mentioned in the other comment, multi-trx for a given BTS running multiple osmo-trx processes is […]

I’ve adjusted the documentation in this file.