laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20072 )

Change subject: [cosmetic] radioInterfaceMulti: More comments
......................................................................

[cosmetic] radioInterfaceMulti: More comments

Change-Id: If608627a77c39b5faabc72c7dd72d00fae8697a9
---
M Transceiver52M/radioInterfaceMulti.cpp
1 file changed, 9 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/Transceiver52M/radioInterfaceMulti.cpp 
b/Transceiver52M/radioInterfaceMulti.cpp
index 29f85ca..e654184 100644
--- a/Transceiver52M/radioInterfaceMulti.cpp
+++ b/Transceiver52M/radioInterfaceMulti.cpp
@@ -38,6 +38,7 @@
 /* Universal resampling parameters */
 #define NUMCHUNKS                              24

+/* number of narrow-band virtual ARFCNs in this wide-band multi-ARFCN device */
 #define MCHANS                                 4

 RadioInterfaceMulti::RadioInterfaceMulti(RadioDevice *radio, size_t tx_sps,
@@ -79,6 +80,10 @@
        RadioInterface::close();
 }

+/*! we re-map the physical channels from the filter bank to logical per-TRX 
channels
+ *  \param[in] pchan physical channel number within the channelizer
+ *  \param[in] chans total number of narrow-band ARFCN channels
+ *  \returns logical (TRX) channel number, or -1 in case there is none */
 static int getLogicalChan(size_t pchan, size_t chans)
 {
        switch (chans) {
@@ -113,6 +118,9 @@
        return -1;
 }

+/*! do we need to frequency shift our spectrum or not?
+ *  \param chans total number of channels
+ *  \returns 1 if we need to shift; 0 if not; -1 on error */
 static int getFreqShift(size_t chans)
 {
        switch (chans) {
@@ -154,6 +162,7 @@
        tx_freq_state.resize(mChans);
        active.resize(MCHANS, false);

+       /* 4 == sps */
        inchunk = RESAMP_INRATE * 4;
        outchunk = RESAMP_OUTRATE * 4;


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

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: If608627a77c39b5faabc72c7dd72d00fae8697a9
Gerrit-Change-Number: 20072
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-MessageType: merged

Reply via email to