{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10045","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10045/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10045/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10045/events","html_url":"https://github.com/RIOT-OS/RIOT/issues/10045","id":363968514,"node_id":"MDU6SXNzdWUzNjM5Njg1MTQ=","number":10045,"title":"Add other radios support for OpenThread pkg","user":{"login":"jia200x","id":1260616,"node_id":"MDQ6VXNlcjEyNjA2MTY=","avatar_url":"https://avatars3.githubusercontent.com/u/1260616?v=4","gravatar_id":"","url":"https://api.github.com/users/jia200x","html_url":"https://github.com/jia200x","followers_url":"https://api.github.com/users/jia200x/followers","following_url":"https://api.github.com/users/jia200x/following{/other_user}","gists_url":"https://api.github.com/users/jia200x/gists{/gist_id}","starred_url":"https://api.github.com/users/jia200x/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jia200x/subscriptions","organizations_url":"https://api.github.com/users/jia200x/orgs","repos_url":"https://api.github.com/users/jia200x/repos","events_url":"https://api.github.com/users/jia200x/events{/privacy}","received_events_url":"https://api.github.com/users/jia200x/received_events","type":"User","site_admin":false},"labels":[{"id":23399100,"node_id":"MDU6TGFiZWwyMzM5OTEwMA==","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/enhancement","name":"enhancement","color":"f17070","default":true},{"id":86443543,"node_id":"MDU6TGFiZWw4NjQ0MzU0Mw==","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/network","name":"network","color":"8f00cc","default":false},{"id":114180965,"node_id":"MDU6TGFiZWwxMTQxODA5NjU=","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/pkg","name":"pkg","color":"006b75","default":false},{"id":1071205470,"node_id":"MDU6TGFiZWwxMDcxMjA1NDcw","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/tracking","name":"tracking","color":"ffff66","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-09-26T10:50:54Z","updated_at":"2018-10-05T08:34:09Z","closed_at":null,"author_association":"MEMBER","body":"The current OpenThread package only supports `at86rf2xx` radios, although the port is done on top of `netdev`.\r\n\r\nThis is why:\r\n1. The radio setup is in the [openthread_bootstrap function](https://github.com/RIOT-OS/RIOT/blob/master/pkg/openthread/contrib/openthread.c#L72). \r\n2. This is the only radio that [implements the NETDEV_EVENT_TX_COMPLETE_DATA_PENDING](https://github.com/RIOT-OS/RIOT/blob/master/drivers/at86rf2xx/at86rf2xx_netdev.c#L664) for handling the frame pending bit in the IEEE802.15.4 ACK\r\n\r\n\r\nI open this issue to keep track of the OT support for other radios. For adding a new radio to OpenThread, do 1. (we should probably move it to auto_init though!) and add the netdev event NETDEV_EVENT_TX_COMPLETE_DATA_PENDING. It should be simple since it should only involve reading a register :)\r\n\r\nTODO:\r\n- [x] at86rf2xx\r\n- [ ] cc2538\r\n- [ ] cc2420\r\n- [ ] mrf24j40 (@bergzand) \r\n- [ ] kw2x\r\n- [ ] kw41zrf (#7107)\r\n\r\n(Am I missing one?)\r\n\r\nAnd when we update/finish this:\r\n- [ ] Reflect changes in pkg documentation (#10048)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/427288202","html_url":"https://github.com/RIOT-OS/RIOT/issues/10045#issuecomment-427288202","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10045","id":427288202,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNzI4ODIwMg==","user":{"login":"jia200x","id":1260616,"node_id":"MDQ6VXNlcjEyNjA2MTY=","avatar_url":"https://avatars3.githubusercontent.com/u/1260616?v=4","gravatar_id":"","url":"https://api.github.com/users/jia200x","html_url":"https://github.com/jia200x","followers_url":"https://api.github.com/users/jia200x/followers","following_url":"https://api.github.com/users/jia200x/following{/other_user}","gists_url":"https://api.github.com/users/jia200x/gists{/gist_id}","starred_url":"https://api.github.com/users/jia200x/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jia200x/subscriptions","organizations_url":"https://api.github.com/users/jia200x/orgs","repos_url":"https://api.github.com/users/jia200x/repos","events_url":"https://api.github.com/users/jia200x/events{/privacy}","received_events_url":"https://api.github.com/users/jia200x/received_events","type":"User","site_admin":false},"created_at":"2018-10-05T08:34:09Z","updated_at":"2018-10-05T08:34:09Z","author_association":"MEMBER","body":"> Sure, only a register read, how hard could this be on a bunch of different radios, the manufacturer probably thought about exposing this information in an easily accessible way :)\r\n\r\nOh ja :)\r\n\r\n> @jia200x To ask the dumb question, do we also have to implement something to configure ACK transmission with frame pending bit?\r\n\r\noh, I totally forgot that. Good catch.\r\nYes, it has to be implemented (it won't have any effects now since pending bit handling is not supported in routers, but it will be added soon).\r\n"}}
{"actor":{"display_login":"jia200x"}}