f4pga-arch-defs icon indicating copy to clipboard operation
f4pga-arch-defs copied to clipboard

Need help to verify arch.*.xml and rr_graph.bin on spartan7

Open KKtiandao opened this issue 2 years ago • 6 comments

To add spartan7 into arch_def flow, I do some modification based on this pr and ran the arch_defs flow completely. But how do i verify the outputs like arch.*.xml and rr_graph? Simulation?

KKtiandao avatar Apr 13 '22 09:04 KKtiandao

@mithro Could you give me some guidances?

KKtiandao avatar Apr 15 '22 05:04 KKtiandao

@KKtiandao There is no validation of the arch and RR graph apart from the internal checks within VTR.

If the flow completes and you are able to get a bitstream (or at least a FASM) as an output, I believe it is enough.

What can be verified is whether the output FASM file is actually correct, running fasm2bels, which allows to decompile the FASM file into the original netlist, and verifying that everything was correctly generated by the flow.

acomodi avatar Apr 15 '22 07:04 acomodi

@acomodi Thanks very much! I'll try fasm2bels.

KKtiandao avatar Apr 18 '22 02:04 KKtiandao

@acomodi I see tile-types listed in the CMakeList.txt is less than which actual in spartan7/artix7; While I try to append some new tile and pbtypes according to the info of prjxray-db, make failed. How do I correct the CMakeList.txt?

KKtiandao avatar Apr 18 '22 11:04 KKtiandao

Hi @KKtiandao. Can you please open a PR with the current WIP changes you have? This can help understanding a bit more what is going on. Also the error output is quite useful to get a picture of the problem.

In general, those tiles and pb_types correspond to the ones defined here (for artix7) and if you want to add some other ones, e.g. the DSP which is currently not present, you need to first define the DSP tile, as well as the DSP pb_type, alongside with the other pb_types here.

acomodi avatar Apr 19 '22 07:04 acomodi

Can you please open a PR with the current WIP changes you have? This can help understanding a bit more what is going on. Also the error output is quite useful to get a picture of the problem.

@acomodi Thanks for your reminder. I have created a PR.

In general, those tiles and pb_types correspond to the ones defined here (for artix7) and if you want to add some other ones, e.g. the DSP which is currently not present, you need to first define the DSP tile, as well as the DSP pb_type, alongside with the other pb_types here.

@acomodi As dsp is a usefull model and it has already in the prjxray-db, why it is not present? Is there any differences in this flow between dsp and other tiles presented?

KKtiandao avatar Apr 19 '22 10:04 KKtiandao