-
Notifications
You must be signed in to change notification settings - Fork 9
Branch: Quicklogic : Multiplier primitive instance should be qlal4s3_mult_cell_macro #65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
The multiplier is added to Yosys and will soon be released on Github along with other changes |
The multiplier is released |
@rakeshm75 all the mults macros are also supported in arch-defs |
@kgugala, I have pulled the latest code. When I run the design with multiplier, I get the following error: 25.2.2. Analyzing design hierarchy.. |
hi @rakeshm75 I added your test to arch-defs repository antmicro/f4pga-arch-defs@7670933 I ran it and it passes to the bitstream. It looks like you don't have the latest Yosys code. Support for the There is one more issue with the design you attached - it get almost entirely optimized out during synthesis. |
Hi Karol, Its the same design which I had filed issue with Yosys that it optimizes the design completely. We should have only qlal4s3_mult_cell_macro multiplier primitive not qlal4s3_mult_32x32_cell nor qlal4s3_mult_16x16_cell. 32x32 or 16x16 multiplier should be built from the base qlal4s3_mult_cell_macro. (* blackbox *) |
Multiplier primitive instance should be qlal4s3_mult_cell_macro, so the black box instance should be the following in cells_sim.v file:
(* blackbox *)
module qlal4s3_mult_cell_macro (
input [31:0] Amult,
input [31:0] Bmult,
input [1:0] Valid_mult,
input sel_mul_32x32,
output [63:0] Cmult );
endmodule
User should be configuring the multiplier into x32 or x16 and should be connecting the signals appropriately.
Attached the primitive files ( cells_sim.v & cells_map.v) , design example (rtl) and EDIF file for reference.
Multipliers.zip
The text was updated successfully, but these errors were encountered: