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

AXI write interface liable to fail if user_clk < axi_clk #126

Open
jack-h opened this issue May 28, 2020 · 3 comments
Open

AXI write interface liable to fail if user_clk < axi_clk #126

jack-h opened this issue May 28, 2020 · 3 comments

Comments

@jack-h
Copy link
Member

jack-h commented May 28, 2020

The clock-domain crossing double-registers used to connect AXI registers to the user's simulink design are liable to miss write events if the axi clock is faster than the simulink clock (user_clk) since writable registers from the AXI interconnect subsystem only hold the IP_BUS_VALID signal high for one cycle.

Should there be some more sophisticated handshaking (as in the WB software registers)?

@jack-h
Copy link
Member Author

jack-h commented May 28, 2020

@AdamI75 -- FYI^. I discovered this when using the ADM-PCIE-9H7 with a 125 MHz AXI clock generated by the PCI core, along with a user_clk=sys_clk=100MHz. For now I'm avoiding the issue by turning the AXI clock down to 62.5 MHz. I'm not sure what the standard configuration is with the other AXIfied platforms, and whether this is going to be an issue for them too.

@wnew
Copy link
Contributor

wnew commented May 29, 2020

@AdamI75 FYI. Ill take a look at this next week.

@AdamI75
Copy link
Contributor

AdamI75 commented Jun 1, 2020

@jack-h and @wnew. Ahhh, yes. We typically use a much lower clock frequency (user_clk > axi_clk) for AXI comms. It also assists with timing closure. At the moment I am just using AXI for comms, but at some point when we stream then we may need to up the clock frequency for streaming interfaces. Although streaming interfaces will not use this IP_BUS_VALID signal in my mind. I am not sure if it is worth changing it now since a lower clock frequency is desirable, in my opinion. We can add this as a requirement to the future toolflow optimisation in the mean time?

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

No branches or pull requests

3 participants