Project

General

Profile

Bug #3729

osmo-trx-lms: Support building against LimeSuite>18.10

Added by pespin 6 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
12/13/2018
Due date:
% Done:

100%

Spec Reference:

Description

looks like soon we'll have an API break in limesuite and we need to take care on that in osmo-trx-lms:
https://discourse.myriadrf.org/t/error-building-osmotrx-for-limesdr-from-source/3839/3

APIs were changed in LimeSuite dc124e4e2ed9b549b142410af172f0592f9f0c23, currently in master and not yet released. We should add some configure.ac test to detect the version of LimeSuite by trying to compile against that API.

+ * @param   memoty      if true also saves value to non-volatile storage   
-API_EXPORT int CALL_CONV LMS_VCTCXOWrite(lms_device_t * dev, uint16_t val);
+API_EXPORT int CALL_CONV LMS_VCTCXOWrite(lms_device_t * dev, uint16_t val, bool memoty);

+ * @param[in]   memory  if true obtains VCTCXO value from non-volatile storage
-API_EXPORT int CALL_CONV LMS_VCTCXORead(lms_device_t * dev, uint16_t *val);
+API_EXPORT int CALL_CONV LMS_VCTCXORead(lms_device_t * dev, uint16_t *val, bool memory);

History

#1 Updated by pespin 6 months ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 90

Commit submitted to gerrit:
https://gerrit.osmocom.org/#/c/osmo-trx/+/12303 lms: Fix build against LimeSuite > 18.10

Let's announce it in Myriadrf once it's merged.

#2 Updated by pespin 6 months ago

  • Status changed from Feedback to Resolved
  • % Done changed from 90 to 100

Merged, closing.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)