{"action":"created","issue":{"url":"https://api.github.com/repos/contiki-os/contiki/issues/1196","labels_url":"https://api.github.com/repos/contiki-os/contiki/issues/1196/labels{/name}","comments_url":"https://api.github.com/repos/contiki-os/contiki/issues/1196/comments","events_url":"https://api.github.com/repos/contiki-os/contiki/issues/1196/events","html_url":"https://github.com/contiki-os/contiki/pull/1196","id":101497717,"number":1196,"title":"New platform: Intel Galileo Board","user":{"login":"aguedes","id":882998,"avatar_url":"https://avatars.githubusercontent.com/u/882998?v=3","gravatar_id":"","url":"https://api.github.com/users/aguedes","html_url":"https://github.com/aguedes","followers_url":"https://api.github.com/users/aguedes/followers","following_url":"https://api.github.com/users/aguedes/following{/other_user}","gists_url":"https://api.github.com/users/aguedes/gists{/gist_id}","starred_url":"https://api.github.com/users/aguedes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aguedes/subscriptions","organizations_url":"https://api.github.com/users/aguedes/orgs","repos_url":"https://api.github.com/users/aguedes/repos","events_url":"https://api.github.com/users/aguedes/events{/privacy}","received_events_url":"https://api.github.com/users/aguedes/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":32,"created_at":"2015-08-17T19:56:25Z","updated_at":"2015-12-20T20:38:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/contiki-os/contiki/pulls/1196","html_url":"https://github.com/contiki-os/contiki/pull/1196","diff_url":"https://github.com/contiki-os/contiki/pull/1196.diff","patch_url":"https://github.com/contiki-os/contiki/pull/1196.patch"},"body":"Hi all,\r\n\r\nThis PR is a follow-up from the Galileo port effort announced a couple of weeks ago in Contiki's mailing list (for further information check this [link](http://sourceforge.net/p/contiki/mailman/message/34350483/)).\r\n\r\nSo this PR adds support for both Intel Quark X1000 SoC (cpu layer) and the Intel Galileo Board (platform layer). In order to assist the review process, below follows some comments that summarize the patches within this PR.\r\n\r\n* Cpu layer patches are prefixed with \"x86:\" and implement the following features:\r\n  * IA initialization (GDT, IDT and IRQs)\r\n  * Programmable Interrupt Controller (PIC) driver\r\n  * Real-time Clock (RTC) driver\r\n  * Programmable Interval Timer (PIT) driver\r\n  * UART driver\r\n  * PCI driver\r\n* Platform layer patches are prefixed with \"galileo:\" and implement the following features:\r\n  * Clock module\r\n  * Etimer, Ctimer and Rtimer libraries\r\n  * Stdio library (stdout and stderr)\r\n* The patch \"examples: Add all-timers example\" introduces a Contiki sample application to validate the timers implementation. This sample is not Galileo-specific and it can be used by any port.\r\n* BSP-related files are located in platform/galileo/bsp/ directory. We provide shell scripts that downloads and build the BSP components.\r\n* platform/galileo/README.md provides detailed instructions on how to build, run and debug application for the platform.\r\n\r\nThis PR doesn't provide Travis integration. Apart from that, it addresses all others requirements described in the [New Platforms section](https://github.com/contiki-os/contiki/wiki/Code-Contributions#new-platforms) from the contribution guide.\r\n\r\nLooking forward to get your feedback!\r\n\r\nBest regards,\r\n\r\nAndre"},"comment":{"url":"https://api.github.com/repos/contiki-os/contiki/issues/comments/166152510","html_url":"https://github.com/contiki-os/contiki/pull/1196#issuecomment-166152510","issue_url":"https://api.github.com/repos/contiki-os/contiki/issues/1196","id":166152510,"user":{"login":"alignan","id":632710,"avatar_url":"https://avatars.githubusercontent.com/u/632710?v=3","gravatar_id":"","url":"https://api.github.com/users/alignan","html_url":"https://github.com/alignan","followers_url":"https://api.github.com/users/alignan/followers","following_url":"https://api.github.com/users/alignan/following{/other_user}","gists_url":"https://api.github.com/users/alignan/gists{/gist_id}","starred_url":"https://api.github.com/users/alignan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alignan/subscriptions","organizations_url":"https://api.github.com/users/alignan/orgs","repos_url":"https://api.github.com/users/alignan/repos","events_url":"https://api.github.com/users/alignan/events{/privacy}","received_events_url":"https://api.github.com/users/alignan/received_events","type":"User","site_admin":false},"created_at":"2015-12-20T20:38:29Z","updated_at":"2015-12-20T20:38:29Z","body":"Great! :+1: from me! @simonduq this can be merged as it is, or rebasing is needed due the amount of commits?"}}