{"action":"created","issue":{"url":"https://api.github.com/repos/contiki-os/contiki/issues/1042","labels_url":"https://api.github.com/repos/contiki-os/contiki/issues/1042/labels{/name}","comments_url":"https://api.github.com/repos/contiki-os/contiki/issues/1042/comments","events_url":"https://api.github.com/repos/contiki-os/contiki/issues/1042/events","html_url":"https://github.com/contiki-os/contiki/pull/1042","id":71826084,"number":1042,"title":"CC2538: added support for SSI1","user":{"login":"uknoblic","id":11543394,"avatar_url":"https://avatars.githubusercontent.com/u/11543394?v=3","gravatar_id":"","url":"https://api.github.com/users/uknoblic","html_url":"https://github.com/uknoblic","followers_url":"https://api.github.com/users/uknoblic/followers","following_url":"https://api.github.com/users/uknoblic/following{/other_user}","gists_url":"https://api.github.com/users/uknoblic/gists{/gist_id}","starred_url":"https://api.github.com/users/uknoblic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/uknoblic/subscriptions","organizations_url":"https://api.github.com/users/uknoblic/orgs","repos_url":"https://api.github.com/users/uknoblic/repos","events_url":"https://api.github.com/users/uknoblic/events{/privacy}","received_events_url":"https://api.github.com/users/uknoblic/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/contiki-os/contiki/labels/Platform%20/%20CPU","name":"Platform / CPU","color":"fbca04"}],"state":"open","locked":false,"assignee":{"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},"milestone":null,"comments":6,"created_at":"2015-04-29T09:27:05Z","updated_at":"2015-05-03T13:17:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/contiki-os/contiki/pulls/1042","html_url":"https://github.com/contiki-os/contiki/pull/1042","diff_url":"https://github.com/contiki-os/contiki/pull/1042.diff","patch_url":"https://github.com/contiki-os/contiki/pull/1042.patch"},"body":"Added support for two SPI ports for cc2538 similar to uart modul.\r\nThis is a re-pull request for the already approved pull request based on a new branch in my own fork. Hopefully i don't create to much confusion..."},"comment":{"url":"https://api.github.com/repos/contiki-os/contiki/issues/comments/98480308","html_url":"https://github.com/contiki-os/contiki/pull/1042#issuecomment-98480308","issue_url":"https://api.github.com/repos/contiki-os/contiki/issues/1042","id":98480308,"user":{"login":"bthebaudeau","id":5755442,"avatar_url":"https://avatars.githubusercontent.com/u/5755442?v=3","gravatar_id":"","url":"https://api.github.com/users/bthebaudeau","html_url":"https://github.com/bthebaudeau","followers_url":"https://api.github.com/users/bthebaudeau/followers","following_url":"https://api.github.com/users/bthebaudeau/following{/other_user}","gists_url":"https://api.github.com/users/bthebaudeau/gists{/gist_id}","starred_url":"https://api.github.com/users/bthebaudeau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bthebaudeau/subscriptions","organizations_url":"https://api.github.com/users/bthebaudeau/orgs","repos_url":"https://api.github.com/users/bthebaudeau/repos","events_url":"https://api.github.com/users/bthebaudeau/events{/privacy}","received_events_url":"https://api.github.com/users/bthebaudeau/received_events","type":"User","site_admin":false},"created_at":"2015-05-03T13:17:51Z","updated_at":"2015-05-03T13:17:51Z","body":"> The discussion in #995 was in part about a new SPI API to replace the one in core.\r\n> \r\n> This pull here is not proposing a new API, it merely extends CC2538 to support multiple instances. This is a very welcome but entirely platform-specific addition, and I don't see the reason to change anything in the core spi.h. The addition of the spix_init prototype is not strictly required and all other changes seem to be merely the result of an uncrustify run.\r\n> \r\n> I think spi.h should be left alone until we have come up with a proposal for a new API. The platform code looks good to me!\r\n\r\nI think that @uknoblic's need was to use these multiple SPI instances from device drivers. Until these drivers are mainlined, I agree that changes to `spi.h` could be postponed and eventually discarded at the benefit of a new SPI API.\r\n\r\n> And by the way, I can't think how this pull could have anything to do with the webserver example breaking on travis.\r\n\r\nTravis looks good. Are you speaking about an issue that got fixed after restarting Travis?"}}