-
Notifications
You must be signed in to change notification settings - Fork 71
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
Fix for SBA 'be' when reading (fixes #22). #70
Conversation
Change-Id: I6d8ddf47c3152326bb9f804ee91af9ac3e1690f2 Signed-off-by: John Eric Martin <John.Martin@emmicro-us.com>
I want to make this a parameter, so that we stay compatible with the current way we do it. |
Do you have a naming preference for this parameter? Perhaps ReadByteEnable=1 to default to the new behavior. Bring it out of dm_top with the other parameters? |
Change-Id: If416f3489d46b1e22f680f5283ad78462bcc72e4
…store the old behavior if desired. Change-Id: Ic97ce3f77664360822ecb6834baf180e98b14fcf Signed-off-by: John Eric Martin <John.Martin@emmicro-us.com>
@bluewww I added a parameter at your request. Please review when you get a chance. |
Thanks for the PR Ill look into it |
Could you please make an entry in the CHANGELOG.md, then it is good to go. |
Change-Id: Id38e03fbfec0570dd7edf47686690dc313b29085 Signed-off-by: John Eric Martin <John.Martin@emmicro-us.com>
@bluewww Added to the changelog. Thanks for looking at it. |
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.
lgtm
Fixes #22
@bluewww It looks like you have a branch that addressed this with a copy & paste of the combinatorial logic but it never got merged in.