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

Corrected CSR addresses for HWLOOP*, FPREC, UHARTID, PRIVLV. #376

Merged

Conversation

Silabs-ArjanB
Copy link
Contributor

Corrected CSR addresses for HWLOOP*, FPREC, UHARTID, PRIVLV. Cleaned up CSR defines.

CSR                     Old address     New address     Comment
---------------------------------------------------------------
HWLoop0_START           12'h7C0         12'h800         Needs to be User accessible (now in user custom read/write space)
HWLoop0_END             12'h7C1         12'h801         Needs to be User accessible (now in user custom read/write space)
HWLoop0_COUNTER         12'h7C2         12'h802         Needs to be User accessible (now in user custom read/write space)
HWLoop1_START           12'h7C4         12'h804         Needs to be User accessible (now in user custom read/write space)
HWLoop1_END             12'h7C5         12'h805         Needs to be User accessible (now in user custom read/write space)
HWLoop1_COUNTER         12'h7C6         12'h806         Needs to be User accessible (now in user custom read/write space)
FPREC                   12'h006         12'h807         Cannot be in Standard space (now in user custom read/write space)
UHARTID                 12'h014         12'hCC0         Cannot be in Standard space; should not be R/W (now in user custom read-only space)
PRIVLV                  12'hC10         12'hCC1         Cannot be in Standard space (also clashes) (now in user custom read-only space)

This fixes issue #142

Signed-off-by: Arjan Bink [email protected]

@davideschiavone
Copy link
Contributor

As currently the HWLoop is undertest and the GCC support is not yet aligned with the new addresses, I am keeping this PR freeze until the PULP GCC is aligned.

@Silabs-ArjanB
Copy link
Contributor Author

Please note that there is a corresponding PULP toolchain request: pulp-platform/pulp-riscv-gnu-toolchain#21

Do you have an estimate for when this merge can happen?

Signed-off-by: Arjan Bink <[email protected]>
@Silabs-ArjanB
Copy link
Contributor Author

Hi @davideschiavone We can no longer wait for PULP toolchain updates. I intend to merge this pull request coming Monday afternoon.

@Silabs-ArjanB Silabs-ArjanB merged commit cc445cd into openhwgroup:master Aug 31, 2020
@Silabs-ArjanB Silabs-ArjanB deleted the ArjanB_csr_address_fix branch August 31, 2020 13:06
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.

2 participants