-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ccm #1
Merged
Merged
Ccm #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
simple ble interface
Also better comments and i2c naming
Since this was refactored to use subscribes, just let the default allow implementation handle things.
Max17205 RomID
Using https://github.com/jameysharp/corrode, automatically convert the old crt1.c to Rust. In contrast to tock#369, which sought to update this file to newer Tock conventions, this is a blind translation, but should hopefully unblock tock#367.
tools: build rustfmt with Tock Rust version
* ble_advertising: print progress It's nice to see progress and most importantly to know that the chip thinks it's advertising when running this example * Print what BLE device name to look for
Corrode nrf51
Wrong Driver Number!
[nRF5X] sensors app work
It is too slow and is broken, plus we are running the same tests on mac, and have no mac specific code.
Fix broken travis builds
Add nrf51 serialization driver for imix
Copying over from tock#664 to hail.
add imix test app
hail: add small delay to nrf51 reset
userland: add gitignore for libg++ build artifacts
Cleanup userland_generic.ld
Removed long ago!
Remove unnecessary comment
doc: add out of tree documentation
- hil::symmetric_encryption - provide distinct traits for Ctr/CBC modes so we can be sure these hardware features exist at compile time - allow distinct src/dst encryption or in-place operation - fix up sam4l::aes to match the new interface, and track input/output with the hardware properly - add TakeCell.map_or_else() - tests are in boards/imix/src/aes_test.rs, and may be run by adding this just before the call to `kernel::main()`: aes_test::run(); NOTE: this commit removes capsules::symmetric_encryption, the userspace interface to AES. Future work will have to patch that back onto the modified HIL, and perhaps also provide a mux for userspace+kernel users.
These methods now have the same semantics as put/take for a TakeCell, except that when the hardware is busy (between a call to crypt() and the following invocation of crypt_done()), they have degenerate behavior: put has no effect, and take returns None.
This changes the interface once more so that there is no opportunity for confusion over the state of buffer ownership.
* The CBC-test fails because it is not supported. I suggest to change the test structure and the trait bounds accordingly
* Because the AES tests depended on trait bounds on both AES128Cbc and AES128Ctr => both the traits needed to implemented * Since NRF5x only support CTR ended up in test failed * That's why I splitted up them (I have not verified that tests are correct though) * The test passed on NRF5x
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Overview
Hi, @bbbert
I have implemented support for NRF5x for the new
symmetric_encryption hil
and I modified the AES tests because they required both CTR and CCM (NRF5x support only CTR)Testing Strategy
The test was executed in the
reset_handler on nRF52-DK
similar to the imix test with the following outcome:TODO or Help Wanted
Have a look and let me know what to fix etc...
Rebased now!
Documentation Updated
/docs
have been updated or no updates are required.Formatting
make formatall
has been run.