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/.
osmith gerrit-no-reply at lists.osmocom.orgosmith has uploaded this change for review. ( https://gerrit.osmocom.org/12245
Change subject: LMSDevice: make use of dev-args in osmo-trx.cfg
......................................................................
LMSDevice: make use of dev-args in osmo-trx.cfg
Allow selecting a specific LimeSDR device by setting dev-args in the
config file. Split up the given dev-args address by comma and select
the device where all substrings can be found.
I could not test this with real hardware, but I have added a test case
to make sure this works as expected.
Related: OS#3654
Change-Id: Ib9aaa066a01bf9de3f78234d7ada884d6f28c852
---
M .gitignore
M Transceiver52M/device/lms/LMSDevice.cpp
A tests/Transceiver52M/LMSDeviceTest.cpp
M tests/Transceiver52M/Makefile.am
M tests/testsuite.at
5 files changed, 103 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/45/12245/1
diff --git a/.gitignore b/.gitignore
index 92fc723..68ca4b1 100644
--- a/.gitignore
+++ b/.gitignore
@@ -18,6 +18,7 @@
tests/CommonLibs/VectorTest
tests/CommonLibs/PRBSTest
tests/Transceiver52M/ConvolveTest
+tests/Transceiver52M/LMSDeviceTest
# automake/autoconf
*.in
diff --git a/Transceiver52M/device/lms/LMSDevice.cpp b/Transceiver52M/device/lms/LMSDevice.cpp
index 5e21894..a727d30 100644
--- a/Transceiver52M/device/lms/LMSDevice.cpp
+++ b/Transceiver52M/device/lms/LMSDevice.cpp
@@ -22,6 +22,7 @@
#include "Threads.h"
#include "LMSDevice.h"
+#include <boost/algorithm/string.hpp>
#include <lime/LimeSuite.h>
#include <osmocom/core/utils.h>
@@ -95,6 +96,35 @@
<< " Step=" << range->step;
}
+/*! Find the device string that matches all filters from \a args.
+ * \param[in] info_list device addresses found by LMS_GetDeviceList()
+ * \param[in] count length of info_list
+ * \param[in] args dev-args value from osmo-trx.cfg, containing comma separated key=value pairs
+ * \return index of first matching item or -1 (no match) */
+int info_list_find(lms_info_str_t* info_list, unsigned int count, const std::string &args)
+{
+ unsigned int i, j;
+ vector<string> filters;
+
+ boost::split(filters, args, boost::is_any_of(","));
+
+ /* iterate over device addresses */
+ for (i=0; i < count; i++) {
+ /* check if all filters match */
+ bool match = true;
+ for (j=0; j < filters.size(); j++) {
+ if (!strstr(info_list[i], filters[j].c_str())) {
+ match = false;
+ break;
+ }
+ }
+
+ if (match)
+ return i;
+ }
+ return -1;
+}
+
int LMSDevice::open(const std::string &args, int ref, bool swap_channels)
{
//lms_info_str_t dev_str;
@@ -103,7 +133,7 @@
float_type sr_host, sr_rf, lpfbw_rx, lpfbw_tx;
uint16_t dac_val;
unsigned int i, n;
- int rc;
+ int rc, dev_id;
LOGC(DDEV, INFO) << "Opening LMS device..";
@@ -123,7 +153,17 @@
for (i = 0; i < n; i++)
LOGC(DDEV, INFO) << "Device [" << i << "]: " << info_list[i];
- rc = LMS_Open(&m_lms_dev, info_list[0], NULL);
+ dev_id = info_list_find(info_list, n, args);
+
+ if (dev_id != -1)
+ LOGC(DDEV, INFO) << "Using device: " << info_list[dev_id];
+ else {
+ LOGC(DDEV, ERROR) << "No LMS device found with address '" << args << "'";
+ delete [] info_list;
+ return -1;
+ }
+
+ rc = LMS_Open(&m_lms_dev, info_list[dev_id], NULL);
if (rc != 0) {
LOGC(DDEV, ERROR) << "LMS_GetDeviceList() failed)";
delete [] info_list;
diff --git a/tests/Transceiver52M/LMSDeviceTest.cpp b/tests/Transceiver52M/LMSDeviceTest.cpp
new file mode 100644
index 0000000..bca12df
--- /dev/null
+++ b/tests/Transceiver52M/LMSDeviceTest.cpp
@@ -0,0 +1,45 @@
+#include <stdio.h>
+#include <string>
+#include <string.h>
+#include <stdlib.h>
+#include <assert.h>
+
+#include <lime/LimeSuite.h>
+
+extern "C"
+{
+ size_t osmo_strlcpy(char *dst, const char *src, size_t siz);
+}
+
+int info_list_find(lms_info_str_t* info_list, unsigned int count, const std::string &args);
+
+using namespace std;
+
+int main(void)
+{
+ unsigned int count;
+ lms_info_str_t* info_list;
+ std::string args;
+
+ /* two fake entries for info_list */
+ count = 2;
+ info_list = new lms_info_str_t[count];
+ osmo_strlcpy(info_list[0], "LimeSDR Mini, addr=24607:1337, serial=FAKESERIAL0001", sizeof(lms_info_str_t));
+ osmo_strlcpy(info_list[1], "LimeSDR Mini, addr=24607:1338, serial=FAKESERIAL0002", sizeof(lms_info_str_t));
+
+ /* find second entry by args filter */
+ args = "serial=FAKESERIAL0002,LimeSDR Mini";
+ assert(info_list_find(info_list, count, args) == 1);
+
+ /* empty args -> first entry */
+ args = "";
+ assert(info_list_find(info_list, count, args) == 0);
+
+ /* not matching args -> -1 */
+ args = "serial=NOTMATCHING";
+ assert(info_list_find(info_list, count, args) == -1);
+
+ /* clean up */
+ delete [] info_list;
+ return 0;
+}
diff --git a/tests/Transceiver52M/Makefile.am b/tests/Transceiver52M/Makefile.am
index a9d3992..2668dc4 100644
--- a/tests/Transceiver52M/Makefile.am
+++ b/tests/Transceiver52M/Makefile.am
@@ -16,3 +16,12 @@
if HAVE_SSE4_1
ConvolveTest_CFLAGS += $(SIMD_FLAGS)
endif
+
+if DEVICE_LMS
+noinst_PROGRAMS += LMSDeviceTest
+LMSDeviceTest_SOURCES = LMSDeviceTest.cpp
+LMSDeviceTest_LDFLAGS = $(LIBOSMOCORE_LIBS) $(LMS_LIBS)
+LMSDeviceTest_LDADD = \
+ $(top_builddir)/Transceiver52M/device/lms/.libs/LMSDevice.o \
+ $(COMMON_LA)
+endif
diff --git a/tests/testsuite.at b/tests/testsuite.at
index 1968481..d8cdd70 100644
--- a/tests/testsuite.at
+++ b/tests/testsuite.at
@@ -51,3 +51,9 @@
cat $abs_srcdir/Transceiver52M/ConvolveTest.ok > expout
AT_CHECK([$abs_top_builddir/tests/Transceiver52M/ConvolveTest], [], [expout], [])
AT_CLEANUP
+
+AT_SETUP([LMSDeviceTest])
+AT_KEYWORDS([LMSDeviceTest])
+AT_SKIP_IF([! test -e $abs_top_builddir/tests/Transceiver52M/LMSDeviceTest])
+AT_CHECK([$abs_top_builddir/tests/Transceiver52M/LMSDeviceTest], [], [], [])
+AT_CLEANUP
--
To view, visit https://gerrit.osmocom.org/12245
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib9aaa066a01bf9de3f78234d7ada884d6f28c852
Gerrit-Change-Number: 12245
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181211/7027c735/attachment.htm>