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

Ftd207 ultra ram #52

Merged
merged 6 commits into from
Apr 21, 2023
Merged

Ftd207 ultra ram #52

merged 6 commits into from
Apr 21, 2023

Conversation

andrewvanderbyl
Copy link

Added URAM option to delay_wideband_prog block.

…rog' block.

Known Issue: Memory has to be 'dual port' for BRAM and Ultra RAM selection. Chnaging to 'single port' generates an error.
Also - no support for 'Distributed memory' added - this would require modification of other scripts. If 'Distributed memory'
is required, it would be necessary to comment out the init script and manually set the memory type in the block.
Copy link

@amartens amartens left a comment

Choose a reason for hiding this comment

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

The Ultra RAM option does not find its way down when a single port RAM option is chosen (yet the single port RAM block does have it as an option). It does work as expected for the Dual Port RAM option though, so I think this is useful as-is.

Merging with -X ours as casper_library_delays won't merge right now as
m2021a doesn't have delay_bram changes. I have saved these changes
separately and will re-add it manually after this.
slx files are binary, so previous merge did not work as branch was
made pre delay_bram update. So re-did the UltraRAM option to delay_bram.

delay_wideband_prog and fft_wideband_real still seem to initialise correctly.
@amartens
Copy link

I had to re-add the UltraRAM option added to the delay_bram block as casper_library_delays wouldn't merge.

@amartens amartens merged commit a5c6f9d into m2021a Apr 21, 2023
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