Friday, 2020-05-08

*** tpb has joined #yosys00:00
*** Cerpin has quit IRC00:02
*** Cerpin has joined #yosys00:09
*** emeb has quit IRC00:12
*** BinaryLust has joined #yosys01:31
*** strobokopp has quit IRC01:52
*** Degi has quit IRC03:38
*** Degi has joined #yosys03:38
*** Cerpin has quit IRC04:16
*** Cerpin has joined #yosys04:17
awygleare there any good examples of throughput-based formal properties?04:21
awygletrying to phrase the requirement "assuming a sink which tries to read every cycle, this core's sink must be able to accept one transaction every clock cycle unless [list of exceptions]"04:22
awygleand i can't really figure out how to do that...04:22
awyglei guess i can do a cover trace fairly easily actually04:28
*** BinaryLust has quit IRC05:00
az0reDo you have a signal for "able to accept a transaction"?05:35
az0rethen it should be not(redor(exception_bitvector)) -> able_to_accept_transaction, no?05:36
az0reso "or(not(redor(exception_bitvector)), able_to_accept_transaction)"05:39
*** BinaryLust has joined #yosys06:09
*** Cerpin has quit IRC06:40
*** emeb_mac has quit IRC06:42
*** N2TOH has quit IRC07:01
*** N2TOH has joined #yosys07:03
*** jakobwenzel has joined #yosys07:06
*** dys has quit IRC07:08
*** N2TOH has quit IRC07:21
*** dys has joined #yosys07:23
*** N2TOH has joined #yosys07:24
*** Cerpin has joined #yosys07:51
*** Thorn has quit IRC08:05
*** vidbina has joined #yosys08:07
*** N2TOH has quit IRC08:10
*** vidbina has quit IRC08:12
*** N2TOH has joined #yosys08:15
*** Thorn has joined #yosys08:23
*** N2TOH has quit IRC08:24
*** N2TOH has joined #yosys08:26
*** N2TOH has quit IRC08:32
*** N2TOH has joined #yosys08:44
*** Asu has joined #yosys08:48
*** futarisIRCcloud has quit IRC08:55
*** N2TOH has quit IRC09:11
*** captain_morgan has quit IRC09:12
*** captain_morgan has joined #yosys09:13
*** N2TOH has joined #yosys09:18
*** N2TOH has quit IRC09:53
*** N2TOH has joined #yosys09:54
*** N2TOH has quit IRC10:02
*** N2TOH has joined #yosys10:03
*** N2TOH has quit IRC10:09
*** N2TOH has joined #yosys10:14
*** kraiskil has joined #yosys10:14
*** N2TOH has quit IRC10:28
*** N2TOH has joined #yosys10:32
*** N2TOH has quit IRC10:42
*** N2TOH has joined #yosys10:55
*** N2TOH has quit IRC11:09
*** N2TOH has joined #yosys11:20
*** BinaryLust has quit IRC11:25
*** kraiskil has quit IRC11:36
*** N2TOH has quit IRC11:42
*** N2TOH has joined #yosys11:45
*** N2TOH has quit IRC11:58
*** N2TOH has joined #yosys12:02
*** adjtm has joined #yosys12:16
*** adjtm_ has quit IRC12:18
*** N2TOH has quit IRC12:33
*** N2TOH has joined #yosys12:45
*** N2TOH has quit IRC12:50
*** vidbina has joined #yosys12:54
*** N2TOH has joined #yosys12:55
*** N2TOH has quit IRC13:00
*** rrika has quit IRC13:03
*** rrika_ has joined #yosys13:03
*** N2TOH has joined #yosys13:07
*** N2TOH_ has joined #yosys13:11
*** N2TOH has quit IRC13:13
*** adjtm_ has joined #yosys13:30
*** adjtm has quit IRC13:32
*** N2TOH has joined #yosys13:33
*** N2TOH_ has quit IRC13:34
*** N2TOH_ has joined #yosys13:34
*** N2TOH has quit IRC13:37
*** emeb has joined #yosys14:01
*** kristianpaul has quit IRC14:11
*** kristianpaul has joined #yosys14:14
*** BinaryLust has joined #yosys14:36
*** vidbina has quit IRC14:46
*** kraiskil has joined #yosys14:51
sensillewhen yosys infers BRAM, it rejects a lot of my smallish register banks with "min efficiency 5' not met (ECP5)14:54
sensilleis that only not to waste bram, or also that the overhead to use the bram would be similar to not using it at all?14:54
sensille(and others again get rejected with "Read port #0 is in clock domain !~async~.")15:04
*** citypw has joined #yosys15:24
ZirconiumXsensille: it's to not waste BRAM, but there needs to be a better heuristic for it all15:27
ZirconiumXI think it will favour LUTRAM for it though15:28
ZirconiumXsensille: (and others again get rejected with "Read port #0 is in clock domain !~async~.") <--- the ECP5 has synchronous read, but you're asking for asynchronous read, for which it would use LUTRAM.15:30
sensilleyeah, some get caught by DPR16X4 later on15:30
sensillei can't find the async read though, the output is explicitly registered15:31
ZirconiumXGenerally you want BRAM for deep but narrow memories, and LUTRAM for wide but shallow memories15:31
ZirconiumXCan I see your Verilog module for it?15:32
sensillehttps://github.com/sensille/conan_fpga/blob/master/fifo.v15:33
tpbTitle: conan_fpga/fifo.v at master · sensille/conan_fpga · GitHub (at github.com)15:33
sensillefor the async read warning15:33
*** BinaryLust has quit IRC15:37
*** jakobwenzel has quit IRC15:44
*** kristianpaul has quit IRC15:45
*** kristianpaul has joined #yosys15:49
*** jfcaron has joined #yosys15:52
ZirconiumXsensille: can you give the exact module path of the thing you want inferred as BRAM?15:56
ZirconiumXLike, conan.u_command.u_stepper.genstepdir[0].u_stepdir.u_fifo.ram15:56
*** kraiskil has quit IRC15:58
ZirconiumXSince as far as I can tell, the async read thing isn't actually a warning15:59
*** citypw has quit IRC16:00
sensilleZirconiumX: conan.u_framing.recv_ring for example16:22
sensilleand conan.u_framing.send_ring16:23
ZirconiumXThe error is actually this: Bram port B1.1 has incompatible clock type.16:24
*** BinaryLust has joined #yosys16:25
ZirconiumXBy the way, you don't need to bother with `verilog -defer` if you use sensible defaults16:26
ZirconiumXdaveshah: can you take a look at this?16:28
daveshahAre you initialising or resetting the output register?16:29
ZirconiumXDoesn't look like it16:30
daveshahI can have a look later16:31
sensilleindeed, "Bram port B1.1 has incompatible clock type." is in the next line16:36
sensilleat least my intention is that everything is driven by the one and only clk16:37
*** kraiskil has joined #yosys16:50
daveshahWith data width 8, address width 12, that file infers memory fine for me16:53
ZirconiumXThe params used are data width 6 address width 516:54
daveshahOh, then yes it seems to fail efficiency criterea16:55
sensilleefficiency=2216:55
sensillehttps://pastebin.com/vmrvT4dB16:56
tpbTitle: Processing conan.u_framing.recv_ring: Properties: ports=2 bits=4096 rports=1 - Pastebin.com (at pastebin.com)16:56
daveshahOh, not the FIFO module16:56
sensilleit is probably how i use the fifo module16:56
sensilleit is an instance of the fifo module16:57
ZirconiumX> reg [7:0] recv_ring [RING_SIZE-1:0];16:57
ZirconiumXNo, it's not16:57
daveshahThat looks like an async port?16:58
daveshahhttps://github.com/sensille/conan_fpga/blob/master/framing.v#L8416:58
tpbTitle: conan_fpga/framing.v at master · sensille/conan_fpga · GitHub (at github.com)16:58
* sensille blushes with shame17:03
sensilleZirconiumX, daveshah: sorry, i don't know where i got the idea from it's the fifo. the fifo just fails the efficiency test17:04
sensillethanks for looking at this17:05
ZirconiumXNo, I'm looking through the FIFO mappings, and it seems like all of them pass the efficiency test at 92%17:06
daveshahI think it would actually have mapped (as a transparent port, with the reg on the address) without the '= 0' here: https://github.com/sensille/conan_fpga/blob/master/framing.v#L8317:07
tpbTitle: conan_fpga/framing.v at master · sensille/conan_fpga · GitHub (at github.com)17:08
sensilleconan.u_framing.send_len_fifo.ram fails with 1%17:08
sensillewhy is initializing the rptr a problem?17:10
ZirconiumXBecause Yosys can't merge flops into memories at the moment17:10
sensillei guess i could initialize rptr with the first clk17:13
sensillealternatively i need to register the output?17:13
ZirconiumXYou already have a clear option to initialise rptr17:16
ZirconiumXReally though, I don't know much about memories here17:17
*** cr1901_modern has quit IRC17:18
*** cr1901_modern has joined #yosys17:26
*** dys has quit IRC17:27
*** Twix has quit IRC17:28
*** Twix has joined #yosys17:31
*** klotz has joined #yosys17:42
*** kraiskil has quit IRC18:04
*** klotz has quit IRC18:05
*** klotz has joined #yosys18:06
*** vidbina has joined #yosys18:13
*** klotz has quit IRC19:10
*** vidbina has quit IRC19:29
*** forrestv has quit IRC19:55
*** Kamilion has quit IRC19:55
*** az0re has quit IRC19:58
*** dys has joined #yosys20:05
*** forrestv has joined #yosys20:15
*** Kamilion has joined #yosys20:17
*** FFY00 has quit IRC20:20
*** FFY00 has joined #yosys20:21
*** Asuu has joined #yosys20:26
*** Asu has quit IRC20:27
*** emeb_mac has joined #yosys20:37
*** forrestv has quit IRC20:45
*** Kamilion has quit IRC20:45
*** forrestv has joined #yosys20:55
*** jfcaron has quit IRC21:01
*** vidbina has joined #yosys21:11
*** BinaryLust has quit IRC21:12
*** kraiskil has joined #yosys21:36
*** Kamilion has joined #yosys21:37
*** smkz is now known as __builtin_trap22:01
*** futarisIRCcloud has joined #yosys22:03
*** az0re has joined #yosys22:10
*** Cerpin has quit IRC22:10
*** Cerpin has joined #yosys22:10
*** kraiskil has quit IRC22:21
*** Cerpin has quit IRC22:28
*** Cerpin has joined #yosys22:28
*** Cerpin has quit IRC22:42
*** Cerpin has joined #yosys22:42
*** Asuu has quit IRC23:21
*** BinaryLust has joined #yosys23:21

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!