| *** tpb has joined #symbiflow | 00:00 | |
| hackerfoo | I get these critical warnings from Vivado when using fasm2v: | 01:58 |
|---|---|---|
| hackerfoo | [Constraints 18-5] Cannot loc instance 'CLBLL_R_X13Y110_SLICE_X19Y110_DLUT/LUT6' at site SLICE_X19Y110, for | 01:58 |
| hackerfoo | bel D5LUT Routing contention at pips element DOUTMUX | 01:58 |
| hackerfoo | [Constraints 18-5] Cannot loc instance 'CLBLM_R_X11Y110_SLICE_X14Y110_RAM32X1D_CD/DP' at site SLICE_X14Y110, for bel C6LUT Element SLICE_X14Y110.C5LUT can not be used as a route-through for net CLBLL_R_X13Y110_SLICE_X18Y110_BMUX taged to C5LUT_O5 because a RAM or shift register is placed there | 01:59 |
| hackerfoo | The resulting bitstream works on hardware, though, even through the bitstream from VPR doesn't. | 02:00 |
| hackerfoo | Resolution: When using BEL constraints, ensure the BEL constraints are defined before the LOC constraints to avoid conflicts at a given site. | 02:02 |
| hackerfoo | Using this approach: https://docs.google.com/document/d/11wJUvr2aVBkUiuYYsFN07jkoYr_ccWlLLLFH8YQw8uQ | 02:05 |
| hackerfoo | I don't know what it means if Step 8 works, and yet using the *_prog target doesn't work. I guess the next step is to use bit2fasm and diff that with the previous fasm file. | 02:07 |
| hackerfoo | Oh wait - I made some changes. I'm surprise it works. https://github.com/HackerFoo/symbiflow-arch-defs/commit/625452bf5ead85c4ac5d5082f69a4a62c0a3c730 | 02:09 |
| tpb | Title: WIP · HackerFoo/symbiflow-arch-defs@625452b · GitHub (at github.com) | 02:09 |
| hackerfoo | I'll have to revert them and see if it does the same thing. | 02:09 |
| *** citypw has joined #symbiflow | 03:07 | |
| *** Bertl_oO is now known as Bertl_zZ | 03:19 | |
| *** proteusguy has quit IRC | 04:33 | |
| *** proteusguy has joined #symbiflow | 04:46 | |
| *** jevinskie has joined #symbiflow | 05:05 | |
| *** proteusguy has quit IRC | 05:21 | |
| *** kraiskil has joined #symbiflow | 05:40 | |
| *** citypw has quit IRC | 06:11 | |
| *** citypw has joined #symbiflow | 06:23 | |
| *** kraiskil has quit IRC | 06:25 | |
| *** kraiskil has joined #symbiflow | 07:02 | |
| *** kraiskil has quit IRC | 07:33 | |
| *** jevinskie has quit IRC | 07:44 | |
| *** kraiskil has joined #symbiflow | 07:46 | |
| *** celadon has quit IRC | 09:06 | |
| *** celadon has joined #symbiflow | 09:07 | |
| *** kraiskil has quit IRC | 09:42 | |
| *** kraiskil has joined #symbiflow | 09:55 | |
| *** citypw has quit IRC | 10:13 | |
| *** kraiskil has quit IRC | 11:16 | |
| *** kraiskil has joined #symbiflow | 11:32 | |
| sf-slack2 | <acomodi> I have understood what is the problem with the following error | 11:40 |
| sf-slack2 | <acomodi> prjxray.fasm_assembler.FasmInconsistentBits: FASM line "CLBLM_R_X5Y117.SLICEL_X1.DOUTMUX.O5" wanted to clear bit (131743, 35, 21) but was set by FASM line "CLBLM_R_X5Y117.SLICEL_X1.DOUTMUX.D5Q" | 11:40 |
| sf-slack2 | <acomodi> basically I use the `placement_index` to retrieve the right fasm name of a SLICE. The problem is that I have noticed that `SLICEL1` has the same placement_index as `SLICEL2`. So the problem raises when a CLB has both SLICES utilized and in the FASM output there will be a bit collision as `genfasm` assigns the same SLICE name to the two different SLICEs. | 11:43 |
| *** OmniMancer has joined #symbiflow | 11:44 | |
| *** Bertl_zZ is now known as Bertl | 12:04 | |
| *** _florent_ has quit IRC | 12:09 | |
| *** sorear has quit IRC | 12:10 | |
| *** _florent_ has joined #symbiflow | 12:11 | |
| *** sorear has joined #symbiflow | 12:11 | |
| *** kraiskil has quit IRC | 12:15 | |
| *** kraiskil has joined #symbiflow | 12:16 | |
| *** proteusguy has joined #symbiflow | 12:21 | |
| sf-slack2 | <mkurc> I made a significant progress regarding CLB tile split. Everything related to routing stored in `channels.db` seems to be remapped correctly. Now I am focusing on adapting the architecture definition to the split tiles. | 12:24 |
| sf-slack2 | <mkurc> I managed to generate initial `arch.xml` with correct grid layout, tile types and FASM prefixes. Working on SLICE tile pb_type xmls | 12:50 |
| sf-slack2 | <acomodi> scalable_proc with N=1 is working with equivalent tiles. I have added a temporary workaround to get the correct FASM slice name | 12:50 |
| sf-slack2 | <acomodi> @mkurc great, I guess that we are converging to a nice tile split solution. As soon as it is done I think I will be able to remove the workarounds from genfasm | 12:53 |
| sf-slack2 | <mkurc> Yeah, the VPR will operate only on SLICEL and SLICEM tiles. | 12:54 |
| sf-slack2 | <mkurc> @acomodi But there is a catch with the genfasm: Right now multiple instances of eg. SLICEL have the same FASM prefix if they come from the same CLB | 12:55 |
| sf-slack2 | <mkurc> its a matter of metadata | 12:55 |
| sf-slack2 | <acomodi> Yes, the problem is that the metadata for a SLICE in CLBLL is different from the one in a CLBLM. The problem is that I couldn't find any clean solution to retrieve the correct metadata | 12:57 |
| sf-slack2 | <acomodi> Actually, I am thinking that it would be better to find a solution to have a cleaner solution in genfasm anyways as it should be independent from the architecture | 12:58 |
| sf-slack2 | <mkurc> @acomodi Maybe the metadata should travel further in the VPR flow ? I'm not familiar with how it is implemented there. | 13:00 |
| sf-slack2 | <acomodi> @mkurc Here is a visualization of the situation: https://imgur.com/a/MWwoesf. If a CLBLM is used instead of the CLBLM it will use different FASM features. The problem is that genfasm is working at the `net` level, and the `net` level is unaware of the fact that the tile have been swapped | 13:06 |
| sf-slack2 | <mkurc> So maybe the net level should be aware of that. | 13:08 |
| sf-slack2 | <mkurc> @acomodi BTW What is the difference between SLICEL0, SLICEL1 and SLICEL2? Why are they specialized? | 13:09 |
| sf-slack2 | <mkurc> I must get rid of that specialization to make generic SLICE tiles | 13:09 |
| sf-slack2 | <acomodi> @mkurc I think the reason is that in the CLB pb_type right now there are two different SLICES with two different names (even if, for instance, CLBLL has two identical SLICELs). They need to have different names otherwise VPR will throw an error | 13:15 |
| sf-slack2 | <acomodi> But apart from that there should be no difference at all between them | 13:15 |
| sf-slack2 | <mkurc> @acomodi You are probably right. I see that there is `ntemplate.slicelN.model.xml` file which is a template. And the only thing being changed is the `{N}` section which is present for some wire names. | 13:17 |
| sf-slack2 | <acomodi> exactly | 13:17 |
| *** _florent_ has quit IRC | 13:22 | |
| *** _florent__ has joined #symbiflow | 13:22 | |
| *** OmniMancer has quit IRC | 13:47 | |
| *** futarisIRCcloud has quit IRC | 14:09 | |
| *** citypw has joined #symbiflow | 14:20 | |
| *** citypw has quit IRC | 14:27 | |
| *** citypw has joined #symbiflow | 14:33 | |
| *** acomodi has quit IRC | 15:24 | |
| mithro | Morning | 15:32 |
| sf-slack2 | <acomodi> Morning | 15:33 |
| elms | Good time of day yall | 15:36 |
| sf-slack2 | <acomodi> I have just created a PR to add the `tiles` tag during prjxray import in symbiflow-arch-defs. https://github.com/SymbiFlow/symbiflow-arch-defs/pull/653 | 15:53 |
| tpb | Title: WIP: Modified prjxray import to add tiles tags by acomodi · Pull Request #653 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 15:53 |
| *** kraiskil has quit IRC | 16:36 | |
| *** citypw has quit IRC | 16:46 | |
| *** proteusguy has quit IRC | 16:49 | |
| *** citypw has joined #symbiflow | 16:55 | |
| mithro | elms: Did you see https://github.com/mithro/symbiflow-arch-defs/commit/c2579a2e867519f94d722bc6975d14f7fd95a8ee ? | 17:17 |
| tpb | Title: ice40: Adding real world timing values. · mithro/symbiflow-arch-defs@c2579a2 · GitHub (at github.com) | 17:17 |
| elms | mithro: yeah, but then I immediately forgot about it. I'm not sure I understand the full context. Did you just mean for reference? | 17:20 |
| mithro | elms: Looks like I had already started importing the real timing values directly into the XML... | 17:21 |
| elms | mithro: ok | 17:21 |
| mithro | elms: Would be good to compare to either your manual or generated version? | 17:22 |
| elms | yep | 17:22 |
| elms | Will do | 17:22 |
| *** proteusguy has joined #symbiflow | 17:54 | |
| mithro | elms: Happy if I merge https://github.com/SymbiFlow/symbiflow-arch-defs/pull/584 | 19:19 |
| tpb | Title: rr_graph tests and docstrings by elmsfu · Pull Request #584 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 19:19 |
| elms | mithro: I will | 19:29 |
| *** kmehall has quit IRC | 21:44 | |
| *** kmehall has joined #symbiflow | 21:46 | |
| *** zkms has quit IRC | 22:20 | |
| *** zkms has joined #symbiflow | 22:26 | |
| *** futarisIRCcloud has joined #symbiflow | 22:45 | |
| *** OmniMancer has joined #symbiflow | 23:34 | |
| *** zkms has quit IRC | 23:52 | |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!