Double Clocked FFT Core

longbimpy error with iverilog #1

Closed jessed81 opened this issue over 7 years ago

First let me thank you for all the work you put into this core. I like your method of generating a configurable core with a C++ program that has no dependencies!

I'm not sure if this is a bug or not but with iverilog there is an error from longbimpy.v. The error given is: "longbimpy.v:142: error: Concatenation operand "(LUTB)*((k)+('sd2))" has indefinite width.". Iverilog seems very strict with this. The core does compile with another simulator (Active-HDL) and seems to work correctly.

I think the syntax that causes the error is actually the replication operator (line 1218 of the c program) "genp, {{(LUTB(k+2))}{1'b0}} };" should be "genp, {(LUTB(k+2)){1'b0}} };" After making this change it compiles and simulates on both of my simulators.

Best regards

Thanks for the ticket, and for finding the problem. From your post, It looks like only a simple syntax problem. I'm surprised neither Vivado nor <A HREF="https://www.veripool.org/wiki/verilator">Verilator</A> caught it, but so be it. With the change, it continues to pass my bench tests, and so I'm going to close this ticket. I've updated the repository, both here and on <A HREF="https://github.com/ZipCPU/dblclockfft">GitHub</A> with your proposed fix. Thanks for the feedback! I've only had a couple people come back to say thanks, and ... it helps to encourage my work. Please feel free to browse any of the other work I've posted here on OpenCores, or equivalently the (often more up to date repositories) on <A HREF="https://github.com/ZipCPU?tab=repositories/">GitHub</A>. Thanks, again! Dan

dgisselq closed this over 7 years ago

Assignee

No one

Labels

Bug

Browse

Double Clocked FFT Core