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

MAM-WB Interface: Reduced to "Classic" Cycle only #27

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nicogtm
Copy link

@nicogtm nicogtm commented Sep 5, 2016

In current implementation, no (useful) burst accesses are possible. Therefore "Classic" single read/writes suffice until redesign for issue #26 . Simplified code to reflect that.

nicogtm added 3 commits September 5, 2016 11:09
For that, CTI_O and BTE_O are set to be always zeros.
Reduced Write states.
Removed unneccessary write states.
read_data_reg <= nxt_read_data_reg;
dat_o_reg <= nxt_dat_o_reg;
addr_o <= nxt_addr_o;
beats <= nxt_beats;
end

assign cyc_o = stb_o;

assign cti_o = 3'b0;
assign bte_o = 2'b0;
Copy link
Member

@wallento wallento Sep 5, 2016

Choose a reason for hiding this comment

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

You will also need to remove the reg type from cti and bte too

@nicogtm
Copy link
Author

nicogtm commented Sep 5, 2016

Done. Thanks.

@imphil
Copy link
Contributor

imphil commented May 25, 2018

@wallento is this still valid and should be merged?

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