chipsalliance / fpga-tool-perf

FPGA tool performance profiling
https://chipsalliance.github.io/fpga-tool-perf
Apache License 2.0
102 stars 30 forks source link

Daisho USB design failing #271

Open acomodi opened 4 years ago

acomodi commented 4 years ago

With the latest database from conda, the daisho-usb3 design fails in symbiflow with the following error:

prjxray.fasm_assembler.FasmInconsistentBits: FASM line "CMT_TOP_R_UPPER_T_X8Y200.PLL_CLK_FREQ_BB1_NS_ACTIVE" wanted to set bit (156, 74, 1) but was cleared by FASM line "CMT_TOP_R_UPPER_T_X8Y200.CMT_TOP_R_UPPER_T_PLLE2_CLKIN2.CMT_TOP_R_UPPER_T_PLLE2_CLK_IN2_INT"
Makefile:43: recipe for target 'top.bit' failed
litghost commented 4 years ago

This is likely a bug in the PLL pip fuzzer. The MMCM pip fuzzer does a better job of decoupling the bits. @mkurc-ant is this something you can help with?

mkurc-ant commented 4 years ago

I can take a look at that. Besides that some PLL PIPs are missing in the issue: https://github.com/SymbiFlow/symbiflow-arch-defs/issues/1750. So anyways PLL PIP fuzzer is worth revisiting.

mkurc-ant commented 4 years ago

@acomodi This is weird. These two FASM features do not even refer to the same segbits. Can you point me out to the design?

acomodi commented 4 years ago

@mkurc-ant This is the top level.

You can run that with the following, prior to having set up the toolperf environment:

./fpgaperf.py --toolchain vpr --board nexys-video --project daisho-usb3
mkurc-ant commented 4 years ago

Ok I've found the issue. These features conflict for segbits_cmt_top_r_upper_t but not for segbits_cmt_top_l_upper_t.