{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3246","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3246/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3246/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3246/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/3246","id":90419730,"number":3246,"title":"core: cleaned up and simplified reboot interface","user":{"login":"haukepetersen","id":620834,"avatar_url":"https://avatars.githubusercontent.com/u/620834?v=3","gravatar_id":"","url":"https://api.github.com/users/haukepetersen","html_url":"https://github.com/haukepetersen","followers_url":"https://api.github.com/users/haukepetersen/followers","following_url":"https://api.github.com/users/haukepetersen/following{/other_user}","gists_url":"https://api.github.com/users/haukepetersen/gists{/gist_id}","starred_url":"https://api.github.com/users/haukepetersen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/haukepetersen/subscriptions","organizations_url":"https://api.github.com/users/haukepetersen/orgs","repos_url":"https://api.github.com/users/haukepetersen/repos","events_url":"https://api.github.com/users/haukepetersen/events{/privacy}","received_events_url":"https://api.github.com/users/haukepetersen/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/API%20CHANGE","name":"API CHANGE","color":"e11d21"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/cleanup","name":"cleanup","color":"FF7c7c"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/core","name":"core","color":"0052cc"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Request%20for%20Comments","name":"Request for Comments","color":"23F823"}],"state":"closed","locked":false,"assignee":{"login":"cgundogan","id":739129,"avatar_url":"https://avatars.githubusercontent.com/u/739129?v=3","gravatar_id":"","url":"https://api.github.com/users/cgundogan","html_url":"https://github.com/cgundogan","followers_url":"https://api.github.com/users/cgundogan/followers","following_url":"https://api.github.com/users/cgundogan/following{/other_user}","gists_url":"https://api.github.com/users/cgundogan/gists{/gist_id}","starred_url":"https://api.github.com/users/cgundogan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cgundogan/subscriptions","organizations_url":"https://api.github.com/users/cgundogan/orgs","repos_url":"https://api.github.com/users/cgundogan/repos","events_url":"https://api.github.com/users/cgundogan/events{/privacy}","received_events_url":"https://api.github.com/users/cgundogan/received_events","type":"User","site_admin":false},"milestone":null,"comments":12,"created_at":"2015-06-23T15:28:51Z","updated_at":"2015-09-07T16:59:56Z","closed_at":"2015-09-07T16:59:56Z","pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/3246","html_url":"https://github.com/RIOT-OS/RIOT/pull/3246","diff_url":"https://github.com/RIOT-OS/RIOT/pull/3246.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/3246.patch"},"body":"I created this PR to continue the discussion from #3242 on a code example.\r\n\r\n@OlegHahm wrote:\r\n\r\n> 1. I think the parameter was originally introduced for POSIX or BSD compatibility reasons.\r\n> 2. Calling *_arch() from outside the kernel feels wrong for me.\r\n\r\nMy thoughts are the following:\r\n\r\n1. I was not able to find any usage of the reboot interface w.r.t. POSIX or BSD. So if this is needed, I would say to write a (4line) wrapper modul for reboot somewhere in `sys` and stick with the simplified reboot version for internal use\r\n2. At first it did for me, too. But by thinking about it, I find it not so bad. For one, the user always knows, that he/she is using hardware dependent functionality. But even more important, it prevents weird structures as we use them currently for the interrupt_xxx functions (see `core/arch/irq_arch.h`).\r\n\r\nAre there any ideas on solving this in a nicer way?"},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/138340612","html_url":"https://github.com/RIOT-OS/RIOT/pull/3246#issuecomment-138340612","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3246","id":138340612,"user":{"login":"cgundogan","id":739129,"avatar_url":"https://avatars.githubusercontent.com/u/739129?v=3","gravatar_id":"","url":"https://api.github.com/users/cgundogan","html_url":"https://github.com/cgundogan","followers_url":"https://api.github.com/users/cgundogan/followers","following_url":"https://api.github.com/users/cgundogan/following{/other_user}","gists_url":"https://api.github.com/users/cgundogan/gists{/gist_id}","starred_url":"https://api.github.com/users/cgundogan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cgundogan/subscriptions","organizations_url":"https://api.github.com/users/cgundogan/orgs","repos_url":"https://api.github.com/users/cgundogan/repos","events_url":"https://api.github.com/users/cgundogan/events{/privacy}","received_events_url":"https://api.github.com/users/cgundogan/received_events","type":"User","site_admin":false},"created_at":"2015-09-07T16:59:56Z","updated_at":"2015-09-07T16:59:56Z","body":"I close this because the discussion stalled or there is no need anymore for a discussion"}}