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

Add support for Sonata PCB #17

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

marnovandermaas
Copy link
Contributor

This commit assumes revision 0.2 of the board. It adds a pin file, a top level module, a clock generator and the appropriate changes to the core file. It also updates the getting started guide to use the Sonata PCB by default.

This commit is derived from a PR in the demo system: lowRISC/ibex-demo-system#86

Copy link
Contributor

@alees24 alees24 left a comment

Choose a reason for hiding this comment

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

Thanks.

doc/guide/getting-started.md Show resolved Hide resolved
data/pins_sonata.xdc Outdated Show resolved Hide resolved
Copy link
Member

@HU90m HU90m left a comment

Choose a reason for hiding this comment

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

This looks good and worked great for me

This commit assumes revision 0.2 of the board. It adds a pin file, a top
level module, a clock generator and the appropriate changes to the core
file. It also updates the getting started guide to use the Sonata PCB by
default.

This commit is derived from a commit in the demo system:
lowRISC/ibex-demo-system@d9ec0d0
With the following changes:
- Remove the JTAG pins from the top and the XDC file since Sonata does
not have a debug module yet.
- Rename the pins to match the names in the PCB documentation.
@marnovandermaas
Copy link
Contributor Author

In the last force push, I renamed the signals in the XDC to match the PCB specification as suggested by @alees24. I also removed the JTAG signals as these are currently not used in the Sonata system while the debug module is missing.

@marnovandermaas marnovandermaas merged commit c700fb4 into lowRISC:main Feb 2, 2024
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