Skip to content

Ckuchta ss mci trace buffer#102

Merged
clayton8 merged 29 commits intomainfrom
ckuchta-ss-mci-trace-buffer
Feb 21, 2025
Merged

Ckuchta ss mci trace buffer#102
clayton8 merged 29 commits intomainfrom
ckuchta-ss-mci-trace-buffer

Conversation

@clayton8
Copy link
Collaborator

@clayton8 clayton8 commented Feb 21, 2025

  • Add MCU Trace Buffer to spec
  • Add MCU trace buffer to MCI and connect to MCU
  • Fix AXI Debug User connection in SS TB
  • Change default size MCU SRAM size to 512KB per FW feedback
  • Chance default MCI MBOX0 SRAM size to 128KB per FW feedback
  • Enable partial STRB writes to MCI Register Bank

clayton8 and others added 23 commits January 30, 2025 15:24
- Remove strap_prod_debug_unlock_pk_hash
- Add Debug intent strap
- Remove datavault and lockable scratch registers as we might not be able to add these in time for 2.0
- Add all MCI interrupts/error to MCI register bank
- Add reset scan_mode control
- Add MBOX AXI User parameters to match caliptra implementation
- Add MBOX AXI USER filtering
- Add debug AXI access to MCI
…eedback. Clean up other parts of the MCU Trace Buffer section.
- Rearrange DMI addresses to add MCU_TRACE_BUFFER registers
- Connect DMI write to MCU Trace Buffer
- Fix size of trace_buffer_depth CSR
- Connect MCU Trace Buffer DMI interface to main DMI interface
@clayton8 clayton8 requested a review from Copilot February 21, 2025 00:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@clayton8 clayton8 requested a review from Copilot February 21, 2025 00:38
@clayton8 clayton8 marked this pull request as ready for review February 21, 2025 00:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 26 out of 26 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (1)

docs/CaliptraSSHardwareSpecification.md:1134

  • Formatting issue: unexpected 'D' in the register access field. It likely should read as 'RO' without the extra character.
| HW_ERROR_NON_FATAL | 0x65 D RO | Yes |  |  |

clayton8 and others added 2 commits February 20, 2025 16:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
clayton8 and others added 3 commits February 20, 2025 16:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@clayton8 clayton8 merged commit ae8af96 into main Feb 21, 2025
1 check passed
@clayton8 clayton8 deleted the ckuchta-ss-mci-trace-buffer branch February 21, 2025 19:15
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.

3 participants