00:42 <_whitenotifier> [YoWASP/yosys] whitequark pushed 1 commit to develop [+0/-0/±1] https://github.com/YoWASP/yosys/compare/daede40bbf86...67b4ef43da93

00:42 <_whitenotifier> [YoWASP/yosys] whitequark 67b4ef4 - Update dependencies.

01:22 <cr1901> whitequark[cis]: For the following Amaranth code passed into verilog.convert with ports=None 1/2

01:24 <cr1901> I get the following output where two ports do not appear at the module definition: http://gopher.wdj-consulting.com:70/paste/c58f33ec-3024-4b52-85ad-5e5c33df2bae.txt

01:24 <cr1901> These wires in module sequencer DO EXIST!! They're just not marked as input/output: wr__data, and ctl__cmd__ops

01:26 <cr1901> What's interesting about these two particular wires in my working code is that I had to use as_value() to convert them to Signals() before the ports= input arg to verilog.convert was happy (m.ctl.cmd.ops and m.wr.data are both unions)

01:26 <cr1901> https://github.com/cr1901/efbutils/blob/amseq/ufm_reader/gen/sequencer.py#L71-L75

01:26 <cr1901> Forgot to link the code testing #886: https://github.com/cr1901/efbutils/blob/vports/ufm_reader/gen/sequencer.py#L59-L71

03:33 <cr1901> Memo to self... sim.add_clock() takes a period. Not a frequency. So I was generating traces with periods of 12e6, and wondering why gtkwave was having a stroke loading them

05:02 <cr1901> I got nerdsniped and I wrote a working pipelined 8x8 multiplier. Uses amaranth.lib.data, but not wiring (b/c I couldn't figure out how to parameterize the signature): http://gopher.wdj-consulting.com:70/paste/79a90b88-5961-41c0-9dba-03b8b9532bbb.txt

05:02 <cr1901> Now that I got that out of my system... bedtime

09:14 <whitequark[cis]> cr1901: I don't actually understand what the issue is here

09:15 <whitequark[cis]> does it work with #886?

09:27 <galibert[m]> Yeah, interfacing that code looks simple

14:04 <cr1901> whitequark[cis]: No, it doesn't work with #886; Diamond dies with "Port ctl__cmd__ops does not exist"

14:05 <cr1901> which is correct; ctl__cmd__ops does not exist as a port, only as a wire.

14:08 <cr1901> But ctl__cmd__ops is part of the signature

14:08 <cr1901> Wrapper().ctl.cmd.ops* is part of Wrapper's signature

18:07 <whitequark[cis]> that's... interesting

18:07 <whitequark[cis]> please file an MCVE

19:11 <_whitenotifier> [amaranth] cr1901 commented on pull request #886: back.{verilog,rtlil}: in convert(), accept a Component without ports - https://github.com/amaranth-lang/amaranth/pull/886#issuecomment-1703915790

19:12 <cr1901> whitequark[cis]: Done

19:14 <whitequark[cis]> thanks!

19:47 <_whitenotifier> [amaranth-boards] whitequark commented on pull request #208: Initial support for Tang nano 9k board - https://github.com/amaranth-lang/amaranth-boards/pull/208#issuecomment-1703922332

21:28 <_whitenotifier> [amaranth] whitequark opened pull request #895: vendor.GowinPlatform: oscillator improvements - https://github.com/amaranth-lang/amaranth/pull/895

21:31 <_whitenotifier> [amaranth] codecov[bot] commented on pull request #895: vendor.GowinPlatform: oscillator improvements - https://github.com/amaranth-lang/amaranth/pull/895#issuecomment-1703945977

23:14 <cr1901> Huh, looks like I missed a whole bunch of messages due to scrollback being screwy.

23:15 <cr1901> > Uses amaranth.lib.data, but not wiring (b/c I couldn't figure out how to parameterize the signature) <-- well, uniterm has an example of parametric signatures (w/o using Component).

23:16 <cr1901> >hey, while I'm at it, are you using MachXO2 nextpnr? does it work? <-- Yes, it works last I checked a few months ago. But seeing as you already packaged it, that info prob doesn't matter :P.

23:18 <whitequark[cis]> yep, to make a parametric signature, I would suggest implementing a @property signature

23:19 <whitequark[cis]> and then returning an object with values you compute

23:57 <cr1901> Everything but the top-level, and EFB Verilog instantiation has been ported. Incredibly, I transcribed the Verilog of this last module to Amaranth correctly the first time.

23:57 <cr1901> I will take this Murphy's Law violation and run