{"action":"created","issue":{"url":"https://api.github.com/repos/contiki-os/contiki/issues/1233","labels_url":"https://api.github.com/repos/contiki-os/contiki/issues/1233/labels{/name}","comments_url":"https://api.github.com/repos/contiki-os/contiki/issues/1233/comments","events_url":"https://api.github.com/repos/contiki-os/contiki/issues/1233/events","html_url":"https://github.com/contiki-os/contiki/pull/1233","id":104447080,"number":1233,"title":"CC26xx/CC13xx fixes","user":{"login":"adamdunkels","id":589970,"avatar_url":"https://avatars.githubusercontent.com/u/589970?v=3","gravatar_id":"","url":"https://api.github.com/users/adamdunkels","html_url":"https://github.com/adamdunkels","followers_url":"https://api.github.com/users/adamdunkels/followers","following_url":"https://api.github.com/users/adamdunkels/following{/other_user}","gists_url":"https://api.github.com/users/adamdunkels/gists{/gist_id}","starred_url":"https://api.github.com/users/adamdunkels/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adamdunkels/subscriptions","organizations_url":"https://api.github.com/users/adamdunkels/orgs","repos_url":"https://api.github.com/users/adamdunkels/repos","events_url":"https://api.github.com/users/adamdunkels/events{/privacy}","received_events_url":"https://api.github.com/users/adamdunkels/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-02T10:15:18Z","updated_at":"2015-09-02T14:32:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/contiki-os/contiki/pulls/1233","html_url":"https://github.com/contiki-os/contiki/pull/1233","diff_url":"https://github.com/contiki-os/contiki/pull/1233.diff","patch_url":"https://github.com/contiki-os/contiki/pull/1233.patch"},"body":"This fixes a few bugs in the latest cc26xx/cc13xx code:\r\n\r\n# Tickless clock\r\n\r\nThe tickless clock interrupt interrupt timing was not computed from the previous interrupt's trigger time, but from the current time (`RTIMER_NOW()`), which may have progressed significantly, causing the next timer to occur with a delay\r\n\r\nAlso, the `clock_time()` function should return an increasing time even if there hasn't been any clock interrupts. This patch recomputes the `count` variable inside the `clock_time()` call instead of on each interrupt.\r\n\r\n# IEEE radio\r\n\r\nA few minor fixes to the RF driver. We also have a workaround for the `lpm_sleep()` issue in #1231, which is not part of this pull request, but maybe that will be taken care of by #1231?"},"comment":{"url":"https://api.github.com/repos/contiki-os/contiki/issues/comments/137104001","html_url":"https://github.com/contiki-os/contiki/pull/1233#issuecomment-137104001","issue_url":"https://api.github.com/repos/contiki-os/contiki/issues/1233","id":137104001,"user":{"login":"g-oikonomou","id":705207,"avatar_url":"https://avatars.githubusercontent.com/u/705207?v=3","gravatar_id":"","url":"https://api.github.com/users/g-oikonomou","html_url":"https://github.com/g-oikonomou","followers_url":"https://api.github.com/users/g-oikonomou/followers","following_url":"https://api.github.com/users/g-oikonomou/following{/other_user}","gists_url":"https://api.github.com/users/g-oikonomou/gists{/gist_id}","starred_url":"https://api.github.com/users/g-oikonomou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/g-oikonomou/subscriptions","organizations_url":"https://api.github.com/users/g-oikonomou/orgs","repos_url":"https://api.github.com/users/g-oikonomou/repos","events_url":"https://api.github.com/users/g-oikonomou/events{/privacy}","received_events_url":"https://api.github.com/users/g-oikonomou/received_events","type":"User","site_admin":false},"created_at":"2015-09-02T14:32:08Z","updated_at":"2015-09-02T14:32:08Z","body":"I meant ContikiMAC, yes. It turns us on/off explicitly, so the idea was that we just linger in the on state in anticipation of next strobe. As you say, it is different logic. CPE startup overhead longer than other radios, since we have to power the domain, boot the thing, patch it and fire up the OSC - not sure if other drivers have to do so many things. By the way this logic was there pre-cc1310.\n\nI also do not understand the change in lpm.c. Will it not schedule the wakeup earlier than desired?"}}