-
Notifications
You must be signed in to change notification settings - Fork 17
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 SoC Mercury XU5 on base board ST1 #67
Merged
Conversation
This file contains 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
9b4401b
to
4144a5e
Compare
glennchid
approved these changes
Nov 8, 2021
thomascobb
reviewed
Nov 8, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand half of this PR, but as long as @glennchid understands it then the bits I know about look fine
Now it is possible to have folders hdl_zynq and hdl_zynqmp in folder common as well as in a module folder. The folder is optional and will not generate an error if it doesn't exist
In Zynq UltraScale+, some parameters are mandatory or have conflicting defaults. The default fifo implentation is common clock native, which does not support having a data count output. Another difference is that the 'asymetric port width' flag is needed when the input data width and the output data width don't match.
The extension server expects a function
It provides readings of on-board temperatures and voltages. The VHDL part is a dummy entity to make the server consider this block.
IDDR is not available in zynqmp, therefore a IDDRE1 must be used, which has a slightly different interface.
ODDR is not available in zynqmp, ODDRE1 is used instead.
Target implementations might prefer to specify only the fpga part and not the board part (which in most cases is not available)
ATF is required for platform zynqmp.
PMU firmware is mandatory and the PS system will not boot without it.
Zynq only requires FSBL and u-boot, while zynqmp requires FSBL, PMU firmware, arm trusted firmware and u-boot.
The pin number must be encoded as a string to simplify parsing, as the led_daemon is a shell script running in a limited environment.
Two apps were added: - xu5_st1-no-fmc: provides the standard soft blocks - xu5_st1-fmc_acq430: provides the standard soft blocks and support for D-TACQ ACQ430 FMC module There is one physical modification needed to use FMCs, given that the I2C connected to the FMC is not the same as the PS associated I2C, we need to join both from the base board connectors: - Connect pin 38 from Anios IO connector 0 to pin 38 from Anios IO Connector 1 - Connect pin 40 from Anios IO connector 0 to pin 40 from Anios IO Connector 1
EmilioPeJu
added a commit
that referenced
this pull request
Mar 15, 2024
At the moment only 2 targets defines it: PandABlocks: MAC from a qspi memory PAndABrick: MAC from a secure EEPROM This is part of a solution for issue #67
EmilioPeJu
added a commit
that referenced
this pull request
Mar 15, 2024
At the moment only 2 targets defines it: PandABlocks: MAC from a qspi memory PAndABrick: MAC from a secure EEPROM This is part of a solution for issue #67
EmilioPeJu
added a commit
that referenced
this pull request
Mar 15, 2024
At the moment only 2 targets define it: PandABlocks: MAC from a qspi memory PAndABrick: MAC from a secure EEPROM This is part of a solution for issue #67
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.
This was successfully tested on a ME-XU5-4EV-1I-D11E-G1 mounted on a ST1.