{"push_id":2780633307,"size":38,"distinct_size":1,"ref":"refs/heads/mpu-interface","head":"324c0f23d64c95d7de6f1a676a6412cc260e3f5f","before":"433ef418b97f38b9a2b6a69eeb90d764e09ebcab","commits":[{"sha":"1a8c5df38e9bb55d2c780637c7f9adf256d043eb","author":{"name":"Pat Pannuto","email":"6364fafe2fb638c2c195524856b7e850805ec613@gmail.com"},"message":"nrf51: update clock to new register interface","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/1a8c5df38e9bb55d2c780637c7f9adf256d043eb"},{"sha":"e966d07b6f40991d387d58516bc3d51fa7988da4","author":{"name":"Pat Pannuto","email":"6364fafe2fb638c2c195524856b7e850805ec613@gmail.com"},"message":"nrf51: update radio to new register interface","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/e966d07b6f40991d387d58516bc3d51fa7988da4"},{"sha":"926d47bbad102295f66293486e197344ad049039","author":{"name":"Pat Pannuto","email":"6364fafe2fb638c2c195524856b7e850805ec613@gmail.com"},"message":"fixup documentation such that doctests pass","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/926d47bbad102295f66293486e197344ad049039"},{"sha":"27e1420ffe2529ef72b8da0c2e29c5429d53e3d0","author":{"name":"Pat Pannuto","email":"6364fafe2fb638c2c195524856b7e850805ec613@gmail.com"},"message":"ci: run doctests on kernel crate","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/27e1420ffe2529ef72b8da0c2e29c5429d53e3d0"},{"sha":"8ebe6a983b1b87765718591f24af1040b7493bed","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"debug: refactor not as process","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/8ebe6a983b1b87765718591f24af1040b7493bed"},{"sha":"06c03cacecef8242b957c1c9742834e922e9fa00","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"boards: update to new debug!() setup","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/06c03cacecef8242b957c1c9742834e922e9fa00"},{"sha":"72cca9072b11edb6ed2ce35d0852eb2902e1a6c1","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"kernel: simplify type in Allocator\n\nRather than use an option and wait for something to use the allocator to\npanic, just panic if an allocator is created for a nonexistent process.","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/72cca9072b11edb6ed2ce35d0852eb2902e1a6c1"},{"sha":"6b481a7facc4d3a195b6266b6a76f525fdcd37bb","author":{"name":"bors[bot]","email":"31e8831a045c193c917fdcb487f388eedbd9ad8e@users.noreply.github.com"},"message":"Merge #1046\n\n1046: Refactor debug.rs to not be implemented as a kernel process r=alevy a=bradjc\n\n### Pull Request Overview\r\n\r\nThis pull request changes the debug module to use a normal `uart::UART` interface rather than acting as a process. This simplifies some kernel data structures as they no longer have to handle the case where an app can be a normal app or a kernel app. It also allows boards to easily use a different uart interface (like segger RTT) without having to implement the `Driver` interface as well.\r\n\r\n~~This is blocked on #1045 but you can help test anyway!~~\r\n\r\n\r\n### Testing Strategy\r\n\r\nThis pull request was tested by running hail app and doing debug!()s from a capsule.\r\n\r\n\r\n### TODO or Help Wanted\r\n\r\nn/a\r\n\r\n\r\n### Documentation Updated\r\n\r\n- [x] Updated the relevant files in `/docs`, or no updates are required.\r\n\r\n### Formatting\r\n\r\n- [x] Ran `make formatall`.\r\n\n\nCo-authored-by: Brad Campbell <bradjc5@gmail.com>","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/6b481a7facc4d3a195b6266b6a76f525fdcd37bb"},{"sha":"3d579a7bf62334cec9cf1e71409f468f42984fb5","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"kernel: move procs array to Kernel struct\n\nThis removes the need for the static PROCS array.","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/3d579a7bf62334cec9cf1e71409f468f42984fb5"},{"sha":"eed1b358dff83afebde45650afb041caa54a7de9","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"kernel: define Eq, PartialEq, Debug for AppId\n\nThese were derived before but since the AppId struct had to be expanded\nthey have to be explicitely defined before.","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/eed1b358dff83afebde45650afb041caa54a7de9"},{"sha":"c8f75500a9dcd83ccd0f182e8110a9fe4334da57","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"kernel: debug: processes array must be passed in\n\nThere is no longer a static reference to it in the kernel crate.","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/c8f75500a9dcd83ccd0f182e8110a9fe4334da57"},{"sha":"fde31a9c52b924d1f36260c13652cd25731db548","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"boards: update to new kernel & panic!() interface","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/fde31a9c52b924d1f36260c13652cd25731db548"},{"sha":"0cb4875d0684ed5c0101f73b0166d4e69981c031","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"kernel: do not need &mut Process references","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/0cb4875d0684ed5c0101f73b0166d4e69981c031"},{"sha":"757f1e671a11f11a2de4ef853564708a80236c35","author":{"name":"bors[bot]","email":"31e8831a045c193c917fdcb487f388eedbd9ad8e@users.noreply.github.com"},"message":"Merge #1128\n\n1128: Kernel doctests r=niklasad1 a=ppannuto\n\n### Pull Request Overview\r\n\r\nThis adds CI checks to validate that our example code in documentation is correct by running doctests. This also fixes up some broken documentation :)\r\n\r\nThis PR is just for the kernel crate.\r\n\r\nSome notes:\r\n\r\n - Lines prefixed with `# ` in doc comments are not rendered\r\n - All fenced blocks are assumed to be rust unless stated otherwise, so rustdoc repurposes the code identifier a bit, e.g. a block with ` ```ignore ` will render as rust code but will not be tested\r\n - The peripherals docs are a bit messy to look at in source code form. Personally I think this is a shortcoming of rustdoc, but I think it's good to see an example of how you handle situations where there's a sort of \"continuing example\" throughout the documentation (you have to keep repeating the \"hidden\" code with `# `)\r\n\r\n### Testing Strategy\r\n\r\nCompiling.\r\n\r\n### Documentation Updated\r\n\r\n- [x] Updated the relevant files in `/docs`, or no updates are required.\r\n\r\n### Formatting\r\n\r\n- [x] Ran `make formatall`.\r\n\n\nCo-authored-by: Pat Pannuto <pat.pannuto@gmail.com>","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/757f1e671a11f11a2de4ef853564708a80236c35"},{"sha":"70f53cafa2f51b3051e21051392b3d19695a508a","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"kernel: add `kernel.create_grant()`\n\nThis is beneficial because:\n\n1. It makes the API a little more consistent, and makes it clear that\nboard_kernel (the &kernel::Kernel) is the main access to the kernel API.\n.kernel_loop() is also called from the Kernel struct.\n\n2. It simplifies where the kernel API is defined. What operations does the\nkernel provide? Look at the pub functions in the impl Kernel block. Note: this\nis different than the interfaces and tools the kernel provides (like StaticRef,\nComponent, etc., and the HIL interfaces).\n\n3. It would be straightforward to move the static variable CONTAINER_COUNTER to\nthe Kernel struct. Doing this would allow the kernel to verify that no grants\nare created after processes are created (which would be invalid if any code\ntried to do that because no space would be reserved for them in the process).","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/70f53cafa2f51b3051e21051392b3d19695a508a"},{"sha":"da05189ed81729a2ba46c7b51000f1f759427219","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"boards: update to `kernel.create_grant()`","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/da05189ed81729a2ba46c7b51000f1f759427219"},{"sha":"c89dd686cc5db0dc5e06a8eacefd7c1e9e68c0e6","author":{"name":"Brad Campbell","email":"165a4e9905a46f36529522106af3f2d9610ad239@gmail.com"},"message":"doc: general updates","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/c89dd686cc5db0dc5e06a8eacefd7c1e9e68c0e6"},{"sha":"6715c6f1bf3f957c5b5b12201b30f584b9f0a2df","author":{"name":"bors[bot]","email":"31e8831a045c193c917fdcb487f388eedbd9ad8e@users.noreply.github.com"},"message":"Merge #1130\n\n1130: docs: Collection of small updates r=ppannuto a=bradjc\n\n### Pull Request Overview\r\n\r\nThis pull request makes a few changes to the docs to update details which have changed over time or could be a little more clear.\r\n\r\n\r\n### Testing Strategy\r\n\r\nn/a\r\n\r\n\r\n### TODO or Help Wanted\r\n\r\nn/a\r\n\r\n\r\n### Documentation Updated\r\n\r\n- [x] Updated the relevant files in `/docs`, or no updates are required.\r\n\r\n### Formatting\r\n\r\n- [x] Ran `make formatall`.\r\n\n\nCo-authored-by: Brad Campbell <bradjc5@gmail.com>","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/6715c6f1bf3f957c5b5b12201b30f584b9f0a2df"},{"sha":"39135c1b6ba3901bef06e81a3a79ca9d40ebdef7","author":{"name":"Amit Aryeh Levy","email":"76b1832478b510fa51d4b2ef52dfd59ca0400db1@amitlevy.com"},"message":"Use proper pin for CC26xx UART RX\n\n`configure` was incorrectly using the stored `tx_pin` twice, for\nconfiguring the RX and TX pin. So UART wasn't actually working at all...","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/39135c1b6ba3901bef06e81a3a79ca9d40ebdef7"},{"sha":"ba8e9974d7c1ddae657b085a6b9f39b1602f6537","author":{"name":"bors[bot]","email":"31e8831a045c193c917fdcb487f388eedbd9ad8e@users.noreply.github.com"},"message":"Merge #1131\n\n1131: Use proper pin for CC26xx UART RX r=ppannuto a=alevy\n\n### Pull Request Overview\r\n\r\nThis fixes a regression in the CC26xx UART introduced by a tiny oversight in #1073. The implementation of `configure` was using the stored `tx_pin` twice, for configuring both TX and RX pins. Since RX is configured second, this overrode resulted in the user-chosen TX pin being configured to act as the RX side of the UART, and no TX configured at all.\r\n\r\nThe result is that the console or debug appeared not to work.\r\n\r\n### Testing Strategy\r\n\r\nI programmed the launchxl board with a kernel from current `master` as well as the `c_hello` app. Running that results in no output on the UART.\r\n\r\nWith the fix in this PR, and the same app, output is \"Hello World!\", as expected.\r\n\r\nI ran similar small tests with `panic!` and `debug!`.\r\n\r\n### TODO or Help Wanted\r\n\r\n_For after this PR_\r\n\r\nI'm concerned about the duplication of the PCRM module---the UART module refers to the CC26xx PCRM module, while other drivers refer to the CC26x2 PCRM module. I originally assumed this was the problem until running `git bisect`. I really think we should just get rid of CC26xx for now. But that's for another PR.\r\n\r\n### Documentation Updated\r\n\r\n- [x] ~~Updated the relevant files in `/docs`, or no updates are required.~~\r\n\r\n### Formatting\r\n\r\n- [x] Ran `make formatall`.\r\n\n\nCo-authored-by: Amit Aryeh Levy <amit@amitlevy.com>","distinct":false,"url":"https://api.github.com/repos/dverhaert/tock/commits/ba8e9974d7c1ddae657b085a6b9f39b1602f6537"}]}
{"actor":{"display_login":"cmcavity"}}