search

YosysHQ / nextpnr

nextpnr portable FPGA place and route tool
ISC License
1.32k stars 245 forks source link

Assert triggers for lp384 devices #453

Open olofk opened 4 years ago

olofk commented 4 years ago

I'm hitting

nextpnr-ice40 -q --lp384 --package cm36 --pcf tinyfpga_bx.pcf --json servant_1.0.2.json --asc servant_1.0.2_next.asc
 terminate called after throwing an instance of 'nextpnr_ice40::assertion_failure'
  what():  Assertion failure: has_clktoq (/home/olof/code/icestorm/nextpnr/ice40/arch.cc:1132)
make: *** [Makefile:16: servant_1.0.2_next.asc] Aborted (core dumped)

when targeting an lp384 device. Changing to lp1k gets rid of the assert

Attaching a minimal testcase lp384bug.tar.gz

daveshah1 commented 4 years ago

Definitely could do with a nicer error, but the problem here is that the design uses a RAM block which don't exist on the lp384, so there is no timing data for these

olofk commented 4 years ago

Aha, I see. Now I know. Feel free to close or keep open as you see fit