search

xiang-rc / cfntt_ref

MIT License
24 stars 5 forks source link

Missing implementation of top module (`top_poly_mul`) for the radix-4 design #3

Open kammoh opened 4 months ago

kammoh commented 4 months ago

There are no top modules included for the radix-4 implementation. Your testbench tb_top references a module named top_poly_mul as the top UUT, but the corresponding implementation is missing for the radix-4 hardare. An implementation for radix-2 top_poly_mul exists in hardware_code_radix-2/top_poly_mul.v, it is not parameterized and would require non-trivial changes for radix-4. Additionally, the file hardware_code_radix-4/fsm.v (finite state machine of the radix-4 design?) is empty. I'd appreciate if you could please add the missing files!

xiang-rc commented 3 months ago

Thanks for your feedback. The file has been updated.