{"action":"created","issue":{"url":"https://api.github.com/repos/wisp/wisp5/issues/22","labels_url":"https://api.github.com/repos/wisp/wisp5/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/wisp/wisp5/issues/22/comments","events_url":"https://api.github.com/repos/wisp/wisp5/issues/22/events","html_url":"https://github.com/wisp/wisp5/pull/22","id":65997626,"number":22,"title":"Get rid of RTCAL_OFFS to get exact RTCAL value.","user":{"login":"jettan","id":1465311,"avatar_url":"https://avatars.githubusercontent.com/u/1465311?v=3","gravatar_id":"","url":"https://api.github.com/users/jettan","html_url":"https://github.com/jettan","followers_url":"https://api.github.com/users/jettan/followers","following_url":"https://api.github.com/users/jettan/following{/other_user}","gists_url":"https://api.github.com/users/jettan/gists{/gist_id}","starred_url":"https://api.github.com/users/jettan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jettan/subscriptions","organizations_url":"https://api.github.com/users/jettan/orgs","repos_url":"https://api.github.com/users/jettan/repos","events_url":"https://api.github.com/users/jettan/events{/privacy}","received_events_url":"https://api.github.com/users/jettan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-02T19:44:04Z","updated_at":"2015-04-13T12:42:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/wisp/wisp5/pulls/22","html_url":"https://github.com/wisp/wisp5/pull/22","diff_url":"https://github.com/wisp/wisp5/pull/22.diff","patch_url":"https://github.com/wisp/wisp5/pull/22.patch"},"body":"With this pull request, I have (partially) rewritten/modified RX_ISR and Timer0A0_ISR  and documented it with the correct amount of cpu cycles each instruction takes. More importantly, it gets rid of the RTCAL_OFFS define and variable which was used to add an estimate value to the measured RTCAL since it was missing a PulseWidth (PW). With this pull request, Timer0A0 fires immediately after setting up Timer0A0, but returns after recording the falling edge (since it was detected in data-0 after the delimiter). Consequently, we also don't need FORCE_INTO_RTCAL delay anymore since we make Timer0A0 fire.\r\n\r\nEDIT: Monday April 13:\r\nAdditionally the modifications for a working BlockWrite is also here. As a demo, I have also edited simpleAckDemo's my_blockWriteCallback to echo what exactly was written BlockWrite.\r\nIn the main(), you have to allocate some memory for RWData.bwrBufPtr. I have done this by allocating an array of 6 words to it. Of course this can be more, but for the sake of a demo I have chosen 6 because the EPC has 12 bytes = 6 words.\r\n\r\nNote: This PR also fixes issue #13 \r\n\r\nIf you have any questions, feel free to ask :)"},"comment":{"url":"https://api.github.com/repos/wisp/wisp5/issues/comments/92339658","html_url":"https://github.com/wisp/wisp5/pull/22#issuecomment-92339658","issue_url":"https://api.github.com/repos/wisp/wisp5/issues/22","id":92339658,"user":{"login":"jettan","id":1465311,"avatar_url":"https://avatars.githubusercontent.com/u/1465311?v=3","gravatar_id":"","url":"https://api.github.com/users/jettan","html_url":"https://github.com/jettan","followers_url":"https://api.github.com/users/jettan/followers","following_url":"https://api.github.com/users/jettan/following{/other_user}","gists_url":"https://api.github.com/users/jettan/gists{/gist_id}","starred_url":"https://api.github.com/users/jettan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jettan/subscriptions","organizations_url":"https://api.github.com/users/jettan/orgs","repos_url":"https://api.github.com/users/jettan/repos","events_url":"https://api.github.com/users/jettan/events{/privacy}","received_events_url":"https://api.github.com/users/jettan/received_events","type":"User","site_admin":false},"created_at":"2015-04-13T12:42:31Z","updated_at":"2015-04-13T12:42:31Z","body":"Added BlockWrite support in this PR."}}