Thursday, 2019-04-18

*** tpb has joined #tomu00:00
xobsThere's still some build weirdness with valentyusb.  It has about a 50% chance of not working once synthesized.  Tweaking the design fixes it.  Hmm...00:25
xobsThe interesting thing is that these builds *kinda* work, they just lock up after a few packets.  I should see if it's Wishbone locking up or something else.00:28
xobsThe good news is that the USB<->Wishbone bridge looks entirely doable, and yesterday I got "reading" working (i.e. I was able to transfer 4 bytes back to the host, but the "ADDRESS" field wasn't working right so unless I hardcoded the address they weren't the _correct_ 4 bytes.)00:47
*** ukembedded_ has joined #tomu01:02
*** ptotter[m] has quit IRC01:10
*** xobs has quit IRC01:10
*** ukembedded has quit IRC01:10
*** ukembedded_ is now known as ukembedded01:10
*** xobs has joined #tomu01:17
*** ptotter[m] has joined #tomu01:18
mithroxobs: Normally if you see that behaviour I would say that you are missing a timing constraint01:41
xobsmithro: what do you mean by timing constraint?  You mean that I have a clock network that's getting identified as the wrong frequency?02:10
mithroxobs: A clock network which is missing a timing constraint (in ISE it was common to be cause by the constraint not making it through a PLL or other similar primitive)02:11
mithroxobs: I would expose the current state of the import FSM as registers and check their values02:13
xobsWhat's the import FSM?02:19
xobsI was under the impression that it was the FSM, yeah.  I thought maybe it was getting into an "ERROR" state, but I patched it so that it immediately exits that state, but that didn't change things.02:19
*** emeb has left #tomu03:09
xobsmithro: according to the nextpnr output, the timing is all correct.03:26
mithros/import/important/03:26
xobsThough some of the async stuff has awfully short timings.03:26
*** rohitksingh has joined #tomu03:27
*** rohitksingh has quit IRC03:35
mithroxobs: My goal for latchup is to help you get the valentyusb hardware rock solid03:43
mithroxobs: I was also pondering seeing if we refactor valentyusb to look like the second diagram -> https://docs.google.com/drawings/d/1olpdWXglPGzJdW_R1DwWviBumlCetyuSZjl-luO_Xe8/edit03:44
tpbTitle: valentyusb - Parts - Google Drawings (at docs.google.com)03:44
mithroAnyway home time for me03:46
xobsmithro: that's my goal, too, since I'm planning on a trip to start production right after I get back.03:46
mithroxobs: btw you have a form2 3d printer thingy right?03:50
xobsmithro: yes, that's what I'm using to prototype the cases.03:53
mithroxobs: could you print up some of the little programmers that were recently posted to the Tomu list?04:19
mithroThat way I can give out a sheet to Fomu hacker boards with a programming jig04:20
mithroIt was designed to be printed with a form 204:20
mithroMuch more portable then the factory jig :-)04:22
xobsmithro: do you have any idea why "If((byte_counter == 1) or (byte_counter == 2) or (byte_counter == 3) or (byte_counter == 4)" gets turned into "if ((usb_byte_counter == 3'd4)) begin" in Verilog?04:26
xobsLooks like it's a bug in migen: https://github.com/m-labs/migen/issues/13704:36
*** futarisIRCcloud has joined #tomu04:44
xobsmithro: also, the FSM only gets into the "ERROR" state sometimes.  Not always.  I think there are two failures here.  One where the transfer.py FSM goes into "ERROR", and another that I still haven't figured out.05:01
* xobs uploaded an image: image.png (598KB) < https://matrix.org/_matrix/media/v1/download/matrix.org/LfHVINpgPXElltvNGzwlhbsj >05:03
xobsThe first one/two/three packets work successfully, but then things start to fail afte rthat.05:04
*** cdmatter has quit IRC05:48
*** cdmatter has joined #tomu05:54
tntxobs: (1) the fact the core doesn't use the IOB registers is often a recipe for random build results.  (2) the 'async fifo' fix I really meant as a quick hack to see if that was the issue, but depending on clock drift between host/fomu I'm pretty sure it can still go wrong some times. Ideally the architecture of the RX pipeline should be fixed.06:38
xobstnt: I'm actually starting to suspect it has to do with the phase difference between the 48 MHz and 12 MHz domains.  If I build using "--with-pll", the problem doesn't seem to happen.06:39
*** cdmatter has quit IRC07:05
*** cdmatter has joined #tomu07:13
*** rohitksingh has joined #tomu07:20
*** rohitksingh has quit IRC08:06
*** jas4711 has joined #tomu08:55
*** rohitksingh has joined #tomu08:56
*** futarisIRCcloud has quit IRC09:13
*** AmosSam has left #tomu09:22
*** AmosSam has joined #tomu09:23
xobsI'm reasonably certain the issue is caused by skew, since it goes away when I use the pll. Nextpnr doesn't support cross-domain timing constraints, so that makes sense.09:28
xobsAnyway. I have Wishbone reading working reasonably reliably now! Wishbone writing isn't quite there yet, but I just need to get the sequence right. And litex_server support is delightfully easy.09:29
*** AmosSam has left #tomu09:35
*** AmosSam has joined #tomu09:36
*** rohitksingh has quit IRC10:14
*** pollo has quit IRC10:18
*** pollo has joined #tomu10:19
*** AmosSam has left #tomu11:34
*** AmosSam has joined #tomu11:46
*** emeb has joined #tomu15:01
*** rohitksingh has joined #tomu15:50
*** rohitksingh has quit IRC16:14
*** rohitksingh has joined #tomu17:04
*** xkapastel has joined #tomu17:07
*** AmosSam has left #tomu17:38
*** AmosSam has joined #tomu17:42
*** AmosSam has left #tomu18:24
*** rohitksingh has quit IRC18:26
*** rohitksingh has joined #tomu18:27
*** AmosSam has joined #tomu18:32
*** rohitksingh has quit IRC19:13
*** jas4711 has quit IRC19:41
*** jas4711 has joined #tomu19:51
*** emeb has left #tomu21:14
*** futarisIRCcloud has joined #tomu23:06
futarisIRCcloudxobs: So the bridge is semi working now?  We can read from the fomu using the bridge?23:15
futarisIRCcloudhttps://github.com/potentialventures/cocotb - If anyone here hasn't already seen it.   I wonder if we should start writing some testbenches for foboot and the like...23:17
tpbTitle: GitHub - potentialventures/cocotb: Coroutine Co-simulation Test Bench (at github.com)23:17

Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!