#milkymist IRC log for Friday, 2013-03-15

GitHub157[mibuild] sbourdeauducq pushed 5 new commits to master: http://git.io/JCgg4g10:43
GitHub157mibuild/master 2491017 Sebastien Bourdeauducq: CRG: use new Module API10:43
GitHub157mibuild/master 37d8029 Sebastien Bourdeauducq: CRG: support reset inversion10:43
GitHub157mibuild/master 71c8172 Sebastien Bourdeauducq: xilinx_ise/CRG_SE: reset inversion support10:43
larsclekernel: the issue is in the signal name. It iterates over a set of all signals, so the indices for signals with the same name end up different between different runs10:45
larscsorting the signals, fixes it10:47
lekernelcan you send a patch?10:47
larscI'm not sure where exactly they should be sorted, so I'll let you take care of it10:47
lekernelwhere's the iteration you are talking about?10:48
larscin build_namespace()10:49
larscI just inserted a signals = sorted(signals) in the first line10:49
lekerneland that fixes it?10:50
larscor I'm lucky10:50
larscI did do 10 runs, no difference10:50
lekernelSignal derives from HUID, so iteration order on sets should be deterministic10:50
lekernelthat's weird10:50
larscwithout the patch every second one is different10:50
larscWhen I build the milkymist-ng project the asmicon_cmd_chooser_stbX (or similar) signal with X either 1 or 0 tend to be swapped between runs10:52
lekernelyeh, I have that too10:53
GitHub34[mibuild] sbourdeauducq pushed 1 new commit to master: http://git.io/1sWw9A11:38
GitHub34mibuild/master 001bead Sebastien Bourdeauducq: altera_quartus, de0nano: add copyright notices11:38
lekernelGerman bureaucracy is truly incredible. I wanted to buy a lx9 microboard to play with fpgatools, and Avnet insist on a *German* VAT number without which you can't place your order17:24
lekernelI guess I'll just order from the US where they aren't that pesky and just pay extra in shipping...17:25
lekerneland don't tell me about German efficiency :) Farnell lost my company documents and after ages when they finally decided to ship my order, parts were (and still are) missing, then I can also talk about the EHSM room cancelled because of internal confusion 1 week before the event17:26
GitHub8[migen] sbourdeauducq pushed 4 new commits to master: http://git.io/1iDW_A18:17
GitHub8migen/master bd8bbd9 Sebastien Bourdeauducq: Make ClockDomains part of fragments18:17
GitHub8migen/master 208e039 Sebastien Bourdeauducq: Local clock domain example18:17
GitHub8migen/master 9b9bd77 Sebastien Bourdeauducq: sim: compatibility with new ClockDomain API18:17
GitHub18[mibuild] sbourdeauducq pushed 1 new commit to master: http://git.io/XYV9BA18:17
GitHub18mibuild/master 6feb6e6 Sebastien Bourdeauducq: New clock_domain API18:17
GitHub87[milkymist-ng] sbourdeauducq pushed 1 new commit to master: http://git.io/QZvswg18:18
GitHub87milkymist-ng/master b2173bb Sebastien Bourdeauducq: Use new ClockDomain API18:18
larscyeay, clockdomains!18:24
lekernelthey were there before, but you could not define module-local ones18:25
lekernelnow when you have a module that defines a clock domain itself, and have several instances of that module, it does the right thing18:25
lekernelalso you can write cd_myclock = ClockDomain() instead of cd_myclock = ClockDomain("myclock") - it also uses tracer.get_obj_var_name() now18:29
larscand which clock is used by default?18:29
larscmakes sense, I guess18:31
lekernelif you don't want the default behaviour (have sys_clk + sys_rst ports on the generated file) you need to have a ClockDomain named "sys" in some fragment, and drive its signals18:31
GitHub105[milkymist-ng] sbourdeauducq pushed 1 new commit to master: http://git.io/35twjQ18:52
GitHub105milkymist-ng/master 0168f83 Sebastien Bourdeauducq: MultiReg: remove idomain18:52
GitHub66[migen] sbourdeauducq pushed 4 new commits to master: http://git.io/Ar3BoA18:52
GitHub66migen/master 51bec34 Sebastien Bourdeauducq: sim: remove PureSimulable (superseded by Module)18:52
GitHub66migen/master 7b49fd9 Sebastien Bourdeauducq: fhdl/specials: fix rename_clock_domain declarations18:52
GitHub66migen/master e2d156e Sebastien Bourdeauducq: genlib/cdc/MultiReg: remove idomain18:52
GitHub135[mibuild] sbourdeauducq pushed 1 new commit to master: http://git.io/bhxELQ18:55
GitHub135mibuild/master 4bf3190 Sebastien Bourdeauducq: MultiReg: remove idomain18:55
--- Sat Mar 16 201300:00

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