components: Update components used by OpenTitan #3280
Merged
Conversation
This file contains hidden or 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
alistair23
approved these changes
Oct 9, 2022
There was a problem hiding this comment.
I don't know enough about the key/data/dest buffers of the digest capsule to know whether any of those lengths need to be made configurable, @alistair23 does this look ok to you?
|
Looks fine to me |
hudson-ayers
approved these changes
Oct 11, 2022
|
bors r+ |
bors bot
added a commit
that referenced
this pull request
Oct 12, 2022
3274: components: several updated components with complex macros r=hudson-ayers a=bradjc ### Pull Request Overview This pull request is a collection of components where the macro was doing a lot more than just allocating static memory. I removed all of the extras and moved them to either new() or finalize(), and made the macros just the static declarations. ### Testing Strategy travis ### TODO or Help Wanted n/a ### Documentation Updated - [x] Updated the relevant files in `/docs`, or no updates are required. ### Formatting - [x] Ran `make prepush`. 3280: components: Update components used by OpenTitan r=hudson-ayers a=bradjc ### Pull Request Overview This updates the components that only OT uses. Somewhat of an arbitrary group. ### Testing Strategy travis ### TODO or Help Wanted n/a ### Documentation Updated - [x] Updated the relevant files in `/docs`, or no updates are required. ### Formatting - [x] Ran `make prepush`. 3281: component: update alarm r=hudson-ayers a=bradjc ### Pull Request Overview This pull request updates the alarm component. Since practically every board uses timers this ended up as its own PR. ### Testing Strategy travis ### TODO or Help Wanted n/a ### Documentation Updated - [x] Updated the relevant files in `/docs`, or no updates are required. ### Formatting - [x] Ran `make prepush`. 3284: component: update ieee802154 r=hudson-ayers a=bradjc ### Pull Request Overview This component update is somewhat involved. ### Testing Strategy travis ### TODO or Help Wanted n/a ### Documentation Updated - [x] Updated the relevant files in `/docs`, or no updates are required. ### Formatting - [x] Ran `make prepush`. 3285: component: update remaining sensors r=hudson-ayers a=bradjc ### Pull Request Overview This pull request is an update of a collection of a few sensor components. ### Testing Strategy travis ### TODO or Help Wanted n/a ### Documentation Updated - [x] Updated the relevant files in `/docs`, or no updates are required. ### Formatting - [x] Ran `make prepush`. 3286: component: update various bus components r=hudson-ayers a=bradjc ### Pull Request Overview This pull request updates the bus/i2c/spi components. This needs other component PRs to be merged. ### Testing Strategy travis ### TODO or Help Wanted n/a ### Documentation Updated - [x] Updated the relevant files in `/docs`, or no updates are required. ### Formatting - [x] Ran `make prepush`. Co-authored-by: Brad Campbell <[email protected]>
|
Build failed (retrying...): |
|
Build succeeded:
|
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
This updates the components that only OT uses.
Somewhat of an arbitrary group.
Testing Strategy
travis
TODO or Help Wanted
n/a
Documentation Updated
/docs, or no updates are required.Formatting
make prepush.