-
Notifications
You must be signed in to change notification settings - Fork 27
RV2A03 #20
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
base: main
Are you sure you want to change the base?
RV2A03 #20
Conversation
reg example_interrupt; | ||
reg last_ui_in_6; | ||
|
||
always @(posedge clk) begin | ||
if (!rst_n) begin | ||
example_interrupt <= 0; | ||
end | ||
|
||
if (ui_in[6] && !last_ui_in_6) begin | ||
example_interrupt <= 1; | ||
end else if (address == 6'h8 && data_write_n != 2'b11 && data_in[0]) begin | ||
example_interrupt <= 0; | ||
end | ||
last_ui_in_6 <= ui_in[6]; | ||
end | ||
|
||
assign user_interrupt = example_interrupt; |
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.
It looks like this is left over from the example - please could you remove it? Set user_interrupt to 0 if you don't need it - thank you!
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.
Will do :)
Using peripheral index 15 to avoid conflicts was a good idea, but there's currently test of the example peripheral that runs on that index. Could you switch to one of the other free ones? Currently 10, 12, 13 or 14. Thanks! |
Sure! |
The failing ledstrip tests can be ignored, however two of your tests are failing in the gate level test. This is often due to missing resets. Please could you investigate? |
I'm on it, just didn't know about the resets :) Will double check |
RV2A03 - Nintendo NES' (Ricoh) RP2A03, but RV controlled
Link to your peripheral repo: https://github.com/fjpolo/tinyqv-rv2a03