Skip to content

MCI Spec Updates for Debug, Straps, and Register Bank#139

Merged
clayton8 merged 16 commits intomainfrom
ckuchta-misc-changes
Mar 19, 2025
Merged

MCI Spec Updates for Debug, Straps, and Register Bank#139
clayton8 merged 16 commits intomainfrom
ckuchta-misc-changes

Conversation

@clayton8
Copy link
Collaborator

  • Change DEBUG_INTENT to sample once per cold boot
  • Add MCI register bank access restriction details
  • Add MCI strap details
  • MCU SRAM section reword to make more clear the access restrictions and debug access

@clayton8 clayton8 requested a review from Copilot March 17, 2025 16:38
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.

Pull Request Overview

This PR updates the CaliptraSS Hardware Specification to include new details on MCI debug behavior, register bank access restrictions, and strap sampling. Key changes include:

  • Adding a new section for MCI CSR Access Restrictions.
  • Introducing a detailed section on strap types and their sampling configuration.
  • Rewording the MCU SRAM section to clarify access restrictions and providing an illustrative diagram.

clayton8 and others added 7 commits March 17, 2025 09:42
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@clayton8 clayton8 marked this pull request as ready for review March 17, 2025 16:49
@clayton8 clayton8 requested a review from calebofearth March 17, 2025 18:47
bharatpillilli
bharatpillilli previously approved these changes Mar 19, 2025
@clayton8 clayton8 merged commit 5191d8d into main Mar 19, 2025
6 checks passed
@clayton8 clayton8 deleted the ckuchta-misc-changes branch March 19, 2025 22:14
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.

4 participants