{"action":"created","issue":{"url":"https://api.github.com/repos/sergev/pic32prog/issues/20","labels_url":"https://api.github.com/repos/sergev/pic32prog/issues/20/labels{/name}","comments_url":"https://api.github.com/repos/sergev/pic32prog/issues/20/comments","events_url":"https://api.github.com/repos/sergev/pic32prog/issues/20/events","html_url":"https://github.com/sergev/pic32prog/pull/20","id":108666302,"number":20,"title":"Added support for device ID probing through STK500V2 parameters","user":{"login":"majenkotech","id":3812892,"avatar_url":"https://avatars.githubusercontent.com/u/3812892?v=3","gravatar_id":"","url":"https://api.github.com/users/majenkotech","html_url":"https://github.com/majenkotech","followers_url":"https://api.github.com/users/majenkotech/followers","following_url":"https://api.github.com/users/majenkotech/following{/other_user}","gists_url":"https://api.github.com/users/majenkotech/gists{/gist_id}","starred_url":"https://api.github.com/users/majenkotech/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/majenkotech/subscriptions","organizations_url":"https://api.github.com/users/majenkotech/orgs","repos_url":"https://api.github.com/users/majenkotech/repos","events_url":"https://api.github.com/users/majenkotech/events{/privacy}","received_events_url":"https://api.github.com/users/majenkotech/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-09-28T14:14:49Z","updated_at":"2015-09-28T15:09:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/sergev/pic32prog/pulls/20","html_url":"https://github.com/sergev/pic32prog/pull/20","diff_url":"https://github.com/sergev/pic32prog/pull/20.diff","patch_url":"https://github.com/sergev/pic32prog/pull/20.patch"},"body":"Code in the bootloader can now report the device ID as parameters 0x44 through 0x47.  Legacy bootloaders just store the parameters you send and regurgitate them verbatim, so this patch also tries to set the device ID to 0xDEAFB00B first before reading it back - if it gets 0xDEAFB00B back then it's legacy, if it gets anything else back then it's a real device ID.\r\n\r\nAlso I found a rather critical bug where the stk500v2 protocol seemed to be sending absolute addresses in the LOAD_ADDRESS command, whereas the bootloader is expecting addresses relative to the start of the flash (0x1D000000).  There is now a little bit that looks to see if the address is >= 0x1D000000 and if it is it subtracts 0x1D000000 from it (all right shifted one bit of course)."},"comment":{"url":"https://api.github.com/repos/sergev/pic32prog/issues/comments/143770089","html_url":"https://github.com/sergev/pic32prog/pull/20#issuecomment-143770089","issue_url":"https://api.github.com/repos/sergev/pic32prog/issues/20","id":143770089,"user":{"login":"majenkotech","id":3812892,"avatar_url":"https://avatars.githubusercontent.com/u/3812892?v=3","gravatar_id":"","url":"https://api.github.com/users/majenkotech","html_url":"https://github.com/majenkotech","followers_url":"https://api.github.com/users/majenkotech/followers","following_url":"https://api.github.com/users/majenkotech/following{/other_user}","gists_url":"https://api.github.com/users/majenkotech/gists{/gist_id}","starred_url":"https://api.github.com/users/majenkotech/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/majenkotech/subscriptions","organizations_url":"https://api.github.com/users/majenkotech/orgs","repos_url":"https://api.github.com/users/majenkotech/repos","events_url":"https://api.github.com/users/majenkotech/events{/privacy}","received_events_url":"https://api.github.com/users/majenkotech/received_events","type":"User","site_admin":false},"created_at":"2015-09-28T15:09:07Z","updated_at":"2015-09-28T15:09:07Z","body":"Hum... I have a problem with this now...  The writing is failing because it's using the chip's page size (128 bytes) instead of the programmer's page size (1024 bytes) so isn't uploading the data properly.  Maybe an adapter-based page size override is needed."}}