03:13 <hackerfoo> The FASM parser doesn't seem to accept prefixes for fasm_params, even though fasm_features does (presumably because it just copies them into the FASM output.)

03:57 <litghost> hackerfoo: Example? It should

03:57 <litghost> hackerfoo: Prefixing is handled in a generic way, no associated with fasm_params or fasm_mux, etc

03:58 <hackerfoo> Whenever I try, it fails here: https://github.com/SymbiFlow/vtr-verilog-to-routing/blob/master%2Bwip/utils/fasm/src/parameters.cpp#L9

03:58 <tpb> Title: vtr-verilog-to-routing/parameters.cpp at master+wip · SymbiFlow/vtr-verilog-to-routing · GitHub (at github.com)

03:59 <litghost> hackerfoo: That's the error for duplicate parameter definition

04:00 <litghost> hackerfoo: When I said example, I meant an example of the XML side

04:00 <hackerfoo> https://github.com/HackerFoo/symbiflow-arch-defs/commit/92dc6ea98c29fd59fe8643b05f65e3be904b0379

04:00 <tpb> Title: WIP · HackerFoo/symbiflow-arch-defs@92dc6ea · GitHub (at github.com)

04:00 <hackerfoo> So I can't have the same thing on the right side?

04:01 <litghost> hackerfoo: When I wrote the code I didn't think to support it

04:02 <litghost> hackerfoo: But that could just be seen as a limitation of the implementation, I see no reason for the right hand side to be unique

04:05 <hackerfoo> So I could change `features_` to be a std::unordered_map<std::string, std::list<FeatureParameters>>, right?

04:06 <litghost> hackerfoo: unordered_multimap is probably better?

04:10 <hackerfoo> litghost: That will work.

10:33 <sf-slack2> <acomodi> Eblif write support has been merged in VTR upstream

10:33 <sf-slack2> <acomodi> Soon I'll have Titan results for the mode selection feature, they should finish in a couple of minutes

14:22 <sf-slack2> <mkurc> @mithro @litghost Could you look at the Yosys PR again: https://github.com/SymbiFlow/yosys/pull/25

14:22 <tpb> Title: Support for attributes on parameters in Verilog by mkurc-ant · Pull Request #25 · SymbiFlow/yosys · GitHub (at github.com)

16:20 <sf-slack2> <mkurc> @mithro I do not have the ability to ask for review and re-review on Symbiflow github. I also cannot add labels to PRs which would be useful eg. if I wanted to re-run the kokoro CI. Can you grant me those permissions ?

16:39 <mithro> mkurc: I'll look at it now, poke me again if I haven't said you have access in 30 minutes

16:40 <mithro> kgugala: There is some script which patches timing values into the muxes -- I can't figure out what is doing it however...

16:46 <sf-slack2> <kgugala> @mithro In my PR? Or somewhere else

16:46 <mithro> Somewhere else...

16:46 <mithro> In current master

16:48 <sf-slack2> <acomodi> mithro: could you duplicate this doc https://docs.google.com/document/d/14ROtp46rYfhk-sJMcGIl5Lnxk99XjjW3lejZI94TZUA in the symbiflow folder? I cannot change the permission to make it publicly accessible

16:48 <tpb> Title: Google Docs - create and edit documents online, for free. (at docs.google.com)

16:49 <mithro> kgugala: It's adding the `delay_constant` lines for this mux... https://www.irccloud.com/pastebin/amZVNMjO/

16:49 <tpb> Title: Snippet | IRCCloud (at www.irccloud.com)

16:50 <sf-slack2> <kgugala> @mithro those values are hardcoded

16:50 <mithro> kgugala: Were are they coming from?

16:50 <sf-slack2> <kgugala> @mithro it's here https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/utils/lib/mux.py#L188

16:50 <tpb> Title: symbiflow-arch-defs/mux.py at master · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com)

16:50 <sf-slack2> <kgugala> I change that in my PR

16:51 <sf-slack2> <kgugala> @mithro: here https://github.com/SymbiFlow/symbiflow-arch-defs/blob/494e4082a2e2a8f87c7195ca93ce4a7846bb9f47/utils/lib/mux.py#L182

16:51 <tpb> Title: symbiflow-arch-defs/mux.py at 494e4082a2e2a8f87c7195ca93ce4a7846bb9f47 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com)

16:52 <sf-slack2> <kgugala> for now only for mux7a/b and mux8

18:09 <mithro> gah! I forgot we moved most of the mux code into utils/lib/mux.py which is why I wasn't finding anything in utils/mux_gen.py

18:14 <sf-slack2> <mkurc> @mithro I have a WIP implementation of SDF import in V2X. I based it on the most recent V2X which is in your PR (#703). Does it make sense to create a PR with my code to your branch ?

18:14 <sf-slack2> <mkurc> @mithro and BTW please look into my github permissions

18:22 <mithro> mkurc: Your permissions should be fixed...

18:24 <sf-slack2> <mkurc> @mithro Yep its fixed now. Thanks.

18:34 <litghost> In yosys, can you duplicate a module? I see the "rename" function, but I want a split synthesis flow

18:34 <litghost> Maybe copy?

18:40 <sf-slack2> <kgugala> @litghost yep, copy seems to do that http://www.clifford.at/yosys/cmd_copy.html

18:40 <tpb> Title: Yosys Open SYnthesis Suite :: Command Reference :: copy (at www.clifford.at)

18:41 <litghost> The problem with "copy" is it requires knowledge of the module top name, so I'm looking at a simpler way

18:49 <sf-slack2> <acomodi> mithro: Ping. Could make a publicly accessible copy of this document please? https://docs.google.com/document/d/14ROtp46rYfhk-sJMcGIl5Lnxk99XjjW3lejZI94TZUA

18:49 <tpb> Title: Google Docs - create and edit documents online, for free. (at docs.google.com)

18:50 <sf-slack2> <kgugala> @acomodi I think this doc is public

18:51 <sf-slack2> <pgielda> no, I cannot acces it with my private account

18:52 <sf-slack2> <kgugala> But in sharing options I see "Public on the web - Anyone on the Internet can find and view"

18:54 <sf-slack2> <kgugala> Indeed I cannot access it with my private account

23:43 <hackerfoo> Something I did in RAMB36E1 removed the delay I observed in RAMB18E1, which still has the delay.

23:51 <litghost> delay?

23:53 <hackerfoo> litghost: https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/tests/bram_shifter/bram_shifter.v#L20

23:53 <tpb> Title: symbiflow-arch-defs/bram_shifter.v at master · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com)

23:54 <hackerfoo> https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/tests/common/ram_shifter.v#L36

23:54 <tpb> Title: symbiflow-arch-defs/ram_shifter.v at master · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com)

23:54 <hackerfoo> It's the compensation needed to line up the switches and LEDs.