-
Notifications
You must be signed in to change notification settings - Fork 9
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
Moved techmaps from Yosys to SymbiFlow #8
Conversation
…f the "synth_xilinx" macro. Updated manual accordingly. Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
…inx" flow accordingly. Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
af63dbd
to
de7c405
Compare
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
9ddd742
to
ea9d46c
Compare
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
endmodule | ||
|
||
`ifndef _EXPLICIT_DRAM | ||
|
||
module RAM64X1D ( |
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.
Does the RAM64X1D_1 trick work here?
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.
What trick are you referring to ? RAM64X1D is recognized by Vivado. And it can be also mapped later to explicit instantation of DPRAM and SPRAM. But this map was moved to SymbiFlow.
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.
I believe RAM64X1D_1 is a valid Vivado primative that is "DRAM64X1D with clock inversion". Can Yosys emit a DRAM with invert clock? If so, it should map to RAM64X1D_1.
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.
Might be worth looking at this spreadsheet.
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.
Seems that I don't have permissions to that document... Anyway, I'll check if Vivado accepts DRAMS with _1.
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.
I converted the spreadsheet to a markdown document here -> f4pga/f4pga-arch-defs#403
I've added techmaps that allow to synthesize for Vivado P&R using Yosys. I've also moved what was possible of VPR related techmaps according to: f4pga/f4pga-arch-defs#294
The macro code for
synth_xilinx
was modified to accomodate changes. I've also added additional flags that allow to disable DRAM and/or BRAM inference.