sb0_ changed the topic of #m-labs to: https://m-labs.hk :: Logs http://irclog.whitequark.org/m-labs
proteusguy has quit [Remote host closed the connection]
<whitequark> sb0: the migen bits_sign calculation for * is broken
<whitequark> it goes like this:
<whitequark> # one operand signed, the other unsigned (add sign bit)
<whitequark> return a_bits + b_bits + 1 - 1, True
<whitequark> but, if you are multiplying C(4,(3,True)) by C(1,(1,False)) you need an extra bit
<whitequark> amusingly, you check that case in test_signed, but i think there's a matching bug in simulator.
<whitequark> er, wrong case
<whitequark> C(4,(3,True)) by C(-1,(1,True)), is the right case
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 2 commits to master [+9/-0/±4] https://git.io/fhK8i
<_whitenotifier-c> [m-labs/nmigen] whitequark f71e0ff - hdl.ast: fix shape calculation for *.
<_whitenotifier-c> [m-labs/nmigen] whitequark 4922a73 - test.compat: import tests from Migen as appropriate.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484616219?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Success. 86.06% (+3.05%) compared to 07e9cfa - https://codecov.io/gh/m-labs/nmigen/commit/4922a73c5d2773eec1b55ac262dcde80e840d994
<_whitenotifier-c> [nmigen] Success. Coverage not affected when comparing 07e9cfa...4922a73 - https://codecov.io/gh/m-labs/nmigen/commit/4922a73c5d2773eec1b55ac262dcde80e840d994
_whitelogger has joined #m-labs
_whitelogger has joined #m-labs
Gurty has quit [Read error: Connection timed out]
Gurty has joined #m-labs
Gurty has quit [Changing host]
Gurty has joined #m-labs
X-Scale has quit [Ping timeout: 240 seconds]
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±28] https://git.io/fhKRw
<_whitenotifier-c> [m-labs/nmigen] whitequark 4948162 - hdl.ir: rename .get_fragment() to .elaborate().
<_whitenotifier-c> [nmigen] whitequark closed issue #9: Bikeshed: design for .get_fragment() - https://git.io/fhKRr
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484632513?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Failure. 85.91% (-0.16%) compared to 4922a73 - https://codecov.io/gh/m-labs/nmigen/commit/4948162f33d2b083acdbeb72e528438f5d27a7c1
<_whitenotifier-c> [nmigen] Failure. 84.09% of diff hit (target 86.06%) - https://codecov.io/gh/m-labs/nmigen/commit/4948162f33d2b083acdbeb72e528438f5d27a7c1
mumptai_ has joined #m-labs
mumptai has quit [Ping timeout: 244 seconds]
X-Scale has joined #m-labs
jevinskie has joined #m-labs
jevinski_ has quit [Ping timeout: 268 seconds]
Rednaxela has quit [Quit: Page closed]
rohitksingh has joined #m-labs
rohitksingh has quit [Ping timeout: 272 seconds]
rohitksingh has joined #m-labs
rohitksingh has quit [Ping timeout: 246 seconds]
rohitksingh has joined #m-labs
rohitksingh has quit [Ping timeout: 244 seconds]
proteusguy has joined #m-labs
<key2_> whitequark: in this case you need m or m.lower
<key2_> ?
<whitequark> key2_: thanks for reminding, that is another thing i need to fix
mumptai_ has quit [Quit: Verlassend]
mumptai has joined #m-labs
<sb0> of course, with ultrascale, adding the SAWG cranks up the jitter in the DDMTD core quite nicely
<whitequark> what affects jitter in an FPGA?
<sb0> it's not using the IOB register, so it's probably picking up noise from the fabric routing
<sb0> recompiling ...
<sb0> good thing I added all sorts of diagnostics to the new sync firmware, so this sort of shenanigan gets flagged immediately
X-Scale has quit [Read error: Connection reset by peer]
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 3 commits to master [+0/-0/±6] https://git.io/fhKFP
<_whitenotifier-c> [m-labs/nmigen] whitequark 4887771 - compat.sim: fix deprecated stdlib import.
<_whitenotifier-c> [m-labs/nmigen] whitequark 7890c0a - test.compat: reenable tests converting to Verilog.
<_whitenotifier-c> [m-labs/nmigen] whitequark 4bf80a6 - compat: suppress deprecation warnings that are internal or during test.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484771833?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Success. Absolute coverage decreased by -0.33% but relative coverage increased by +14.08% compared to 4948162 - https://codecov.io/gh/m-labs/nmigen/commit/4bf80a6e3356c4d6ce420f068aa1b3b4fc95dd2e
<_whitenotifier-c> [nmigen] Success. 100% of diff hit (target 85.91%) - https://codecov.io/gh/m-labs/nmigen/commit/4bf80a6e3356c4d6ce420f068aa1b3b4fc95dd2e
<sb0> still jittery with IOB FF... pfff
<sb0> okay, averaging over several measurements gets rid of it
<sb0> rjo: here?
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 2 commits to master [+0/-0/±6] https://git.io/fhKNz
<_whitenotifier-c> [m-labs/nmigen] whitequark b133eb7 - back.rtlil: accept any elaboratable, not just fragments.
<_whitenotifier-c> [m-labs/nmigen] whitequark 7acea8f - examples: update for newer API.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484782324?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Success. 85.58% remains the same compared to 4bf80a6 - https://codecov.io/gh/m-labs/nmigen/commit/7acea8f3ce84f1f1e8efea1bbb1580e84e17f577
<_whitenotifier-c> [nmigen] Success. Coverage not affected when comparing 4bf80a6...7acea8f - https://codecov.io/gh/m-labs/nmigen/commit/7acea8f3ce84f1f1e8efea1bbb1580e84e17f577
<sb0> rjo: I'm trying to do the equivalent of tracking phase mode on SAWG
<sb0> sawg.frequency0.set_mu(sawg_ftw)
<sb0> sawg.phase0.set_mu(sawg_ftw*now_mu()*4 >> 16)
<sb0> tried this but does not work
<sb0> splines 1/2 are at their reset values
<sb0> aah, got it, now_mu is in 8x rtio
<sb0> so it's /2 and not *4
<sb0> oooh, it works! I have 2.4GHz Sayma DAC sync'd with Urukul
<sb0> over DRTIO
<sb0> amazing
<sb0> so, with some hacks, it seems ultrascale can do 2.4GHz timing
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 3 commits to master [+0/-0/±4] https://git.io/fhKhL
<_whitenotifier-c> [m-labs/nmigen] whitequark 8686e9a - back.pysim: give better names to unnamed fragments and their signals.
<_whitenotifier-c> [m-labs/nmigen] whitequark e74dbc3 - back.pysim: support async reset.
<_whitenotifier-c> [m-labs/nmigen] whitequark f44ca29 - lib.cdc: add ResetSynchronizer.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484810052?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Success. 85.6% (+0.02%) compared to 7acea8f - https://codecov.io/gh/m-labs/nmigen/commit/f44ca291c12958640515ad0fadd2498ca69a8157
<_whitenotifier-c> [nmigen] Success. 86.66% of diff hit (target 85.58%) - https://codecov.io/gh/m-labs/nmigen/commit/f44ca291c12958640515ad0fadd2498ca69a8157
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 2 commits to master [+1/-0/±2] https://git.io/fhKhi
<_whitenotifier-c> [m-labs/nmigen] whitequark 2fb85a6 - compat.fifo: fix _FIFOInterface deprecation wrapper.
<_whitenotifier-c> [m-labs/nmigen] whitequark 6cd9f7d - compat.genlib.resetsync: add shim for AsyncResetSynchronizer.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484814662?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Failure. 85.6% (-0.01%) compared to f44ca29 - https://codecov.io/gh/m-labs/nmigen/commit/6cd9f7db1937cb34b946368ebfaa3c0021713da9
<_whitenotifier-c> [nmigen] Success. Coverage not affected when comparing f44ca29...6cd9f7d - https://codecov.io/gh/m-labs/nmigen/commit/6cd9f7db1937cb34b946368ebfaa3c0021713da9
<_whitenotifier-c> [nmigen] q3k opened issue #27: Simulator: adding two clocks to a single domain causes DeadlineError in simulation - https://git.io/fh6f6
early has quit [Quit: Leaving]
early has joined #m-labs
<_whitenotifier-c> [nmigen] whitequark opened issue #28: Multiclock simulation broken - https://git.io/fh6tI
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 2 commits to master [+0/-0/±2] https://git.io/fh6m4
<_whitenotifier-c> [m-labs/nmigen] whitequark ce7ba70 - compat.fhdl.specials: fix __all__ list.
<_whitenotifier-c> [m-labs/nmigen] whitequark e844b0e - compat.fhdl.module: fix typo.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484881284?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Success. 85.6% remains the same compared to 6cd9f7d - https://codecov.io/gh/m-labs/nmigen/commit/e844b0e0958de57faf6e436746f38a7e38c65a58
<_whitenotifier-c> [nmigen] Failure. 0% of diff hit (target 85.6%) - https://codecov.io/gh/m-labs/nmigen/commit/e844b0e0958de57faf6e436746f38a7e38c65a58
<_whitenotifier-c> [m-labs/nmigen] whitequark pushed 2 commits to master [+0/-0/±2] https://git.io/fh6YG
<_whitenotifier-c> [m-labs/nmigen] whitequark bc5a127 - hdl.ast: fix ValueKey for Cat.
<_whitenotifier-c> [m-labs/nmigen] whitequark 43e4833 - back.rtlil: accept ast.Const as cell parameter.
<_whitenotifier-c> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/484885699?utm_source=github_status&utm_medium=notification
<_whitenotifier-c> [nmigen] Failure. 85.51% (-0.09%) compared to e844b0e - https://codecov.io/gh/m-labs/nmigen/commit/43e4833ddb5f36d8343ae830288eca221cec5fad
<_whitenotifier-c> [nmigen] Failure. 0% of diff hit (target 85.6%) - https://codecov.io/gh/m-labs/nmigen/commit/43e4833ddb5f36d8343ae830288eca221cec5fad