{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3751","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3751/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3751/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3751/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/3751","id":104077681,"number":3751,"title":"samd21/i2c: add feedback in case transfer failed","user":{"login":"daniel-k","id":224128,"avatar_url":"https://avatars.githubusercontent.com/u/224128?v=3","gravatar_id":"","url":"https://api.github.com/users/daniel-k","html_url":"https://github.com/daniel-k","followers_url":"https://api.github.com/users/daniel-k/followers","following_url":"https://api.github.com/users/daniel-k/following{/other_user}","gists_url":"https://api.github.com/users/daniel-k/gists{/gist_id}","starred_url":"https://api.github.com/users/daniel-k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniel-k/subscriptions","organizations_url":"https://api.github.com/users/daniel-k/orgs","repos_url":"https://api.github.com/users/daniel-k/repos","events_url":"https://api.github.com/users/daniel-k/events{/privacy}","received_events_url":"https://api.github.com/users/daniel-k/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Device%20driver","name":"Device driver","color":"0052cc"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/enhancement","name":"enhancement","color":"f17070"}],"state":"open","locked":false,"assignee":{"login":"bapclenet","id":4921425,"avatar_url":"https://avatars.githubusercontent.com/u/4921425?v=3","gravatar_id":"","url":"https://api.github.com/users/bapclenet","html_url":"https://github.com/bapclenet","followers_url":"https://api.github.com/users/bapclenet/followers","following_url":"https://api.github.com/users/bapclenet/following{/other_user}","gists_url":"https://api.github.com/users/bapclenet/gists{/gist_id}","starred_url":"https://api.github.com/users/bapclenet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bapclenet/subscriptions","organizations_url":"https://api.github.com/users/bapclenet/orgs","repos_url":"https://api.github.com/users/bapclenet/repos","events_url":"https://api.github.com/users/bapclenet/events{/privacy}","received_events_url":"https://api.github.com/users/bapclenet/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2015-08-31T15:38:33Z","updated_at":"2015-09-04T16:47:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/3751","html_url":"https://github.com/RIOT-OS/RIOT/pull/3751","diff_url":"https://github.com/RIOT-OS/RIOT/pull/3751.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/3751.patch"},"body":"According to the I2C interface the return value of any read or write function is either 'the bytes transfered' or `-1` in case of undefined device.\r\n\r\nWith this PR those functions now return `0` in case of general failure, so the user/driver code can react accordingly."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/137788349","html_url":"https://github.com/RIOT-OS/RIOT/pull/3751#issuecomment-137788349","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3751","id":137788349,"user":{"login":"daniel-k","id":224128,"avatar_url":"https://avatars.githubusercontent.com/u/224128?v=3","gravatar_id":"","url":"https://api.github.com/users/daniel-k","html_url":"https://github.com/daniel-k","followers_url":"https://api.github.com/users/daniel-k/followers","following_url":"https://api.github.com/users/daniel-k/following{/other_user}","gists_url":"https://api.github.com/users/daniel-k/gists{/gist_id}","starred_url":"https://api.github.com/users/daniel-k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniel-k/subscriptions","organizations_url":"https://api.github.com/users/daniel-k/orgs","repos_url":"https://api.github.com/users/daniel-k/repos","events_url":"https://api.github.com/users/daniel-k/events{/privacy}","received_events_url":"https://api.github.com/users/daniel-k/received_events","type":"User","site_admin":false},"created_at":"2015-09-04T16:47:31Z","updated_at":"2015-09-04T16:47:31Z","body":"The Riot I2C interface doesn't provide detailed feedback in case of failure, but internally the driver has some additional feedback. It was just that you could use this information maybe somewhere, but then it turned out that it's not the case. I don't have an opinion on this as long as the I2C interface doesn't provide more detailed feedback."}}