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/.
pespin gerrit-no-reply at lists.osmocom.orgpespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-trx/+/17475 ) Change subject: radioDevice: Drop unused RSSI param from readSamples API ...................................................................... radioDevice: Drop unused RSSI param from readSamples API The out "RSSI" parameter is only filled by USRPDevice, and not used at all by any user of the API (radioInterface*.cpp). RSSI seems to be computed nowadays in the common path in Transceiver::pullRadioVector(). Change-Id: I06c2ea5a9891d170bc468f952bbf2a7e64d95784 --- M Transceiver52M/device/common/radioDevice.h M Transceiver52M/device/lms/LMSDevice.cpp M Transceiver52M/device/lms/LMSDevice.h M Transceiver52M/device/uhd/UHDDevice.cpp M Transceiver52M/device/uhd/UHDDevice.h M Transceiver52M/device/usrp1/USRPDevice.cpp M Transceiver52M/device/usrp1/USRPDevice.h 7 files changed, 11 insertions(+), 11 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/75/17475/1 diff --git a/Transceiver52M/device/common/radioDevice.h b/Transceiver52M/device/common/radioDevice.h index 26903e8..7ca3f00 100644 --- a/Transceiver52M/device/common/radioDevice.h +++ b/Transceiver52M/device/common/radioDevice.h @@ -81,8 +81,7 @@ @return The number of samples actually read */ virtual int readSamples(std::vector<short *> &bufs, int len, bool *overrun, - TIMESTAMP timestamp = 0xffffffff, bool *underrun = 0, - unsigned *RSSI = 0) = 0; + TIMESTAMP timestamp = 0xffffffff, bool *underrun = 0) = 0; /** Write samples to the radio. @param buf Contains the data to be written. diff --git a/Transceiver52M/device/lms/LMSDevice.cpp b/Transceiver52M/device/lms/LMSDevice.cpp index 884cc28..0cd8002 100644 --- a/Transceiver52M/device/lms/LMSDevice.cpp +++ b/Transceiver52M/device/lms/LMSDevice.cpp @@ -739,7 +739,7 @@ // NOTE: Assumes sequential reads int LMSDevice::readSamples(std::vector < short *>&bufs, int len, bool * overrun, - TIMESTAMP timestamp, bool * underrun, unsigned *RSSI) + TIMESTAMP timestamp, bool * underrun) { int rc, num_smpls, expect_smpls; ssize_t avail_smpls; diff --git a/Transceiver52M/device/lms/LMSDevice.h b/Transceiver52M/device/lms/LMSDevice.h index 755f6c4..60b88fa 100644 --- a/Transceiver52M/device/lms/LMSDevice.h +++ b/Transceiver52M/device/lms/LMSDevice.h @@ -111,7 +111,7 @@ */ int readSamples(std::vector < short *>&buf, int len, bool * overrun, TIMESTAMP timestamp = 0xffffffff, bool * underrun = - NULL, unsigned *RSSI = NULL); + NULL); /** Write samples to the LMS. @param buf Contains the data to be written. diff --git a/Transceiver52M/device/uhd/UHDDevice.cpp b/Transceiver52M/device/uhd/UHDDevice.cpp index 5d267c8..ddb6631 100644 --- a/Transceiver52M/device/uhd/UHDDevice.cpp +++ b/Transceiver52M/device/uhd/UHDDevice.cpp @@ -721,7 +721,7 @@ } int uhd_device::readSamples(std::vector<short *> &bufs, int len, bool *overrun, - TIMESTAMP timestamp, bool *underrun, unsigned *RSSI) + TIMESTAMP timestamp, bool *underrun) { ssize_t rc; uhd::time_spec_t ts; diff --git a/Transceiver52M/device/uhd/UHDDevice.h b/Transceiver52M/device/uhd/UHDDevice.h index d485a8e..4809b25 100644 --- a/Transceiver52M/device/uhd/UHDDevice.h +++ b/Transceiver52M/device/uhd/UHDDevice.h @@ -74,7 +74,7 @@ enum TxWindowType getWindowType() { return tx_window; } int readSamples(std::vector<short *> &bufs, int len, bool *overrun, - TIMESTAMP timestamp, bool *underrun, unsigned *RSSI); + TIMESTAMP timestamp, bool *underrun); int writeSamples(std::vector<short *> &bufs, int len, bool *underrun, TIMESTAMP timestamp, bool isControl); diff --git a/Transceiver52M/device/usrp1/USRPDevice.cpp b/Transceiver52M/device/usrp1/USRPDevice.cpp index 766a228..5eaca07 100644 --- a/Transceiver52M/device/usrp1/USRPDevice.cpp +++ b/Transceiver52M/device/usrp1/USRPDevice.cpp @@ -365,7 +365,7 @@ // NOTE: Assumes sequential reads int USRPDevice::readSamples(std::vector<short *> &bufs, int len, bool *overrun, - TIMESTAMP timestamp, bool *underrun, unsigned *RSSI) + TIMESTAMP timestamp, bool *underrun) { #ifndef SWLOOPBACK if (!m_uRx) @@ -433,8 +433,10 @@ *underrun = true; LOGC(DDEV, DEBUG) << "UNDERRUN in TRX->USRP interface"; } - if (RSSI) *RSSI = (word0 >> 21) & 0x3f; - +#if 0 + /* FIXME: Do something with this ? */ + unsigned RSSI = (word0 >> 21) & 0x3f; +#endif if (!isAligned) continue; unsigned cursorStart = pktTimestamp - timeStart + dataStart; diff --git a/Transceiver52M/device/usrp1/USRPDevice.h b/Transceiver52M/device/usrp1/USRPDevice.h index 47f2a43..35dcd55 100644 --- a/Transceiver52M/device/usrp1/USRPDevice.h +++ b/Transceiver52M/device/usrp1/USRPDevice.h @@ -121,8 +121,7 @@ @return The number of samples actually read */ int readSamples(std::vector<short *> &buf, int len, bool *overrun, - TIMESTAMP timestamp = 0xffffffff, bool *underrun = NULL, - unsigned *RSSI = NULL); + TIMESTAMP timestamp = 0xffffffff, bool *underrun = NULL); /** Write samples to the USRP. @param buf Contains the data to be written. -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/17475 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I06c2ea5a9891d170bc468f952bbf2a7e64d95784 Gerrit-Change-Number: 17475 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin at sysmocom.de> Gerrit-MessageType: newchange -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200312/27fb14dc/attachment.htm>