{"action":"created","issue":{"url":"https://api.github.com/repos/Nuand/bladeRF/issues/385","labels_url":"https://api.github.com/repos/Nuand/bladeRF/issues/385/labels{/name}","comments_url":"https://api.github.com/repos/Nuand/bladeRF/issues/385/comments","events_url":"https://api.github.com/repos/Nuand/bladeRF/issues/385/events","html_url":"https://github.com/Nuand/bladeRF/issues/385","id":62399457,"number":385,"title":"libbladeRF: TX PLL Instability @ 1.8602 GHz","user":{"login":"jynik","id":3046210,"avatar_url":"https://avatars.githubusercontent.com/u/3046210?v=3","gravatar_id":"","url":"https://api.github.com/users/jynik","html_url":"https://github.com/jynik","followers_url":"https://api.github.com/users/jynik/followers","following_url":"https://api.github.com/users/jynik/following{/other_user}","gists_url":"https://api.github.com/users/jynik/gists{/gist_id}","starred_url":"https://api.github.com/users/jynik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jynik/subscriptions","organizations_url":"https://api.github.com/users/jynik/orgs","repos_url":"https://api.github.com/users/jynik/repos","events_url":"https://api.github.com/users/jynik/events{/privacy}","received_events_url":"https://api.github.com/users/jynik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-17T13:35:44Z","updated_at":"2015-03-18T20:19:43Z","closed_at":null,"body":"It has been reported that the TX PLL loses lock when varying over temperature @ 1.8602 GHz.\r\n\r\nThis frequency is close to a FREQSEL change, and based upon [5.24 of the LMS6002D FAQ](http://www.limemicro.com/download/FAQ_v1.0r10.pdf), the VCO selected by the FREQSEL value is operating near its extremes (hence the lack of stability).  The following change would allow the switchover to a more appropriate FREQSEL value earlier:\r\n\r\n```\r\ndiff --git a/host/libraries/libbladeRF/src/lms.c b/host/libraries/libbladeRF/src/lms.c\r\nindex afcc57b..87485e9 100644\r\n--- a/host/libraries/libbladeRF/src/lms.c\r\n+++ b/host/libraries/libbladeRF/src/lms.c\r\n@@ -112,8 +112,8 @@ static const struct freq_range {\r\n     FREQ_RANGE(930000000,               1142500000,             0x25),\r\n     FREQ_RANGE(1142500000,              1347500000,             0x2d),\r\n     FREQ_RANGE(1347500000,              1620000000,             0x35),\r\n-    FREQ_RANGE(1620000000,              1860000000,             0x3d),\r\n-    FREQ_RANGE(1860000000u,             2285000000u,            0x24),\r\n+    FREQ_RANGE(1620000000,              1900000000,             0x3d),\r\n+    FREQ_RANGE(1900000000u,             2285000000u,            0x24),\r\n     FREQ_RANGE(2285000000u,             2695000000u,            0x2c),\r\n     FREQ_RANGE(2695000000u,             3240000000u,            0x34),\r\n     FREQ_RANGE(3240000000u,             BLADERF_FREQUENCY_MAX,  0x3c),\r\n```\r\n\r\nThis tracker items has been opened for the purposes of:\r\n* Evaluating the above change\r\n* Reviewing the remainder of the FREQSEL table and comparing it to the VCO ranges presented in FAQ item 5.24.\r\n\r\nA follow-up tracker item will likely be required, as the updated LMS6002D documentation proposes a new algorithm for tuning, which differs from the FREQSEL table-based method.\r\n"},"comment":{"url":"https://api.github.com/repos/Nuand/bladeRF/issues/comments/83152721","html_url":"https://github.com/Nuand/bladeRF/issues/385#issuecomment-83152721","issue_url":"https://api.github.com/repos/Nuand/bladeRF/issues/385","id":83152721,"user":{"login":"jynik","id":3046210,"avatar_url":"https://avatars.githubusercontent.com/u/3046210?v=3","gravatar_id":"","url":"https://api.github.com/users/jynik","html_url":"https://github.com/jynik","followers_url":"https://api.github.com/users/jynik/followers","following_url":"https://api.github.com/users/jynik/following{/other_user}","gists_url":"https://api.github.com/users/jynik/gists{/gist_id}","starred_url":"https://api.github.com/users/jynik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jynik/subscriptions","organizations_url":"https://api.github.com/users/jynik/orgs","repos_url":"https://api.github.com/users/jynik/repos","events_url":"https://api.github.com/users/jynik/events{/privacy}","received_events_url":"https://api.github.com/users/jynik/received_events","type":"User","site_admin":false},"created_at":"2015-03-18T20:19:43Z","updated_at":"2015-03-18T20:19:43Z","body":"The Yate team has indicated that other values operate too close to VCO extrema. They have tested and provided the following patch:\r\n\r\n```\r\n--- ./host/libraries/libbladeRF/src/lms.c.orig\t2014-04-07 14:00:34.000000000 +0300\r\n+++ ./host/libraries/libbladeRF/src/lms.c\t2015-03-17 12:23:23.980424916 +0200\r\n@@ -87,16 +87,16 @@ static const struct freq_range {\r\n     FREQ_RANGE(232500000,   285625000,   0x27),\r\n     FREQ_RANGE(285625000,   336875000,   0x2f),\r\n     FREQ_RANGE(336875000,   405000000,   0x37),\r\n-    FREQ_RANGE(405000000,   465000000,   0x3f),\r\n-    FREQ_RANGE(465000000,   571250000,   0x26),\r\n+    FREQ_RANGE(405000000,   475000000,   0x3f),\r\n+    FREQ_RANGE(475000000,   571250000,   0x26),\r\n     FREQ_RANGE(571250000,   673750000,   0x2e),\r\n     FREQ_RANGE(673750000,   810000000,   0x36),\r\n-    FREQ_RANGE(810000000,   930000000,   0x3e),\r\n-    FREQ_RANGE(930000000,   1142500000,  0x25),\r\n-    FREQ_RANGE(1142500000,  1347500000,  0x2d),\r\n-    FREQ_RANGE(1347500000,  1620000000,  0x35),\r\n-    FREQ_RANGE(1620000000,  1860000000,  0x3d),\r\n-    FREQ_RANGE(1860000000u, 2285000000u, 0x24),\r\n+    FREQ_RANGE(810000000,   945000000,   0x3e),\r\n+    FREQ_RANGE(945000000,   1142500000,  0x25),\r\n+    FREQ_RANGE(1142500000,  1350000000,  0x2d),\r\n+    FREQ_RANGE(1350000000,  1620000000,  0x35),\r\n+    FREQ_RANGE(1620000000,  1890000000,  0x3d),\r\n+    FREQ_RANGE(1890000000u, 2285000000u, 0x24),\r\n     FREQ_RANGE(2285000000u, 2695000000u, 0x2c),\r\n     FREQ_RANGE(2695000000u, 3240000000u, 0x34),\r\n     FREQ_RANGE(3240000000u, 3720000000u, 0x3c),\r\n```"}}