Skip to content
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

misc fpv fixes #88

Merged
merged 7 commits into from
Feb 14, 2025
Merged

misc fpv fixes #88

merged 7 commits into from
Feb 14, 2025

Conversation

Nitsirks
Copy link
Collaborator

@Nitsirks Nitsirks commented Feb 5, 2025

Changed msg inf so it won't latch new data if msg hold is asserted. Fixes corner case where msg cnt could increment from 0 -> 1 and we'd lose the data if first msg was held.
Changed keccak to always drive ready low if it isn't ready. Hold shouldn't be dependent on valid whenever possible.
Changed keccak pad to start keccak even if receiving a partial in the same clock cycle. FPV found that a partial immediately following the last msg of a block would cause keccak to not start until the following cycle due to precedence in if/else statements.

added assertion for checking if zeroize cleared seed. Used in zeroize during kv check in caliptra.

fixed sha3 state dv assertion. Would spuriously assert dv if controller didn't immediately send a run command after accepting the state output.

changed keccak to always drive ready low if it isn't ready
changed keccak pad to start keccak even if receiving a partial in the same clock cycle
…h updated timestamp and hash after successful run
calebofearth and others added 5 commits February 13, 2025 16:26
* Fixup UVM assertion reporting for 1.1d compatibility

* MICROSOFT AUTOMATED PIPELINE: Stamp 'cwhitehead-msft-uvm-assert-fixups' with updated timestamp and hash after successful run
* converted spec to md format

* changed the format

* updated images

* removed TOC

* converted to md file

* added ​Pseudocode

* added perf resuls

* added single sign  perf resuls

* updated the tables

* updated API

* removed docs file

* updated area results

* updated tables

* fixed API and psedocode

* signing perf

* added a note for freq

* updated sub-sections

* updated equations

* updated more equations

* MICROSOFT AUTOMATED PIPELINE: Stamp 'mojtaba-md_spec' with updated timestamp and hash after successful run
…yvault flow is running

fixed spurious state valid dv in sha3 block when controller doesn't immediately assert a new run command
…h updated timestamp and hash after successful run
@Nitsirks Nitsirks merged commit d3e1773 into main Feb 14, 2025
5 checks passed
@Nitsirks Nitsirks deleted the user/dev/michnorris/fpv_fix branch February 14, 2025 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

msg interface doesn't keep data stable when no ready for the very first packet sent to keccak
3 participants