whitequark[cis] changed the topic of #amaranth-lang to: Amaranth hardware definition language · weekly meetings: Amaranth each Mon 1700 UTC, Amaranth SoC each Fri 1700 UTC · play https://amaranth-lang.org/play/ · code https://github.com/amaranth-lang · logs https://libera.irclog.whitequark.org/amaranth-lang · Matrix #amaranth-lang:matrix.org
<_whitenotifier-5> [rfcs] wanda-phi commented on pull request #55: RFC 55: New `lib.io` components. - https://github.com/amaranth-lang/rfcs/pull/55#issuecomment-2000812367
lf has joined #amaranth-lang
lf_ has quit [Ping timeout: 272 seconds]
Degi_ has joined #amaranth-lang
Degi has quit [Ping timeout: 252 seconds]
Degi_ is now known as Degi
<tpw_rules> is m.submodules[""] = ... defined?
<tpw_rules> looking the code it's different to m.submodules += ... but i'm not sure what exactly the effect is
<tpw_rules> yeah it explodes in the rtlil something or other, but sim is okay
<Wanda[cis]> that sounds like a bug, can you show me the code?
<tpw_rules> yes, making an example rn
<Wanda[cis]> m.submodules["abc"] = ... is defined, and it creates a named submodule, same as m.submodules.abc = ...
<Wanda[cis]> while m.submodules += ... will make an anonymous one
<_whitenotifier-5> [amaranth] tpwrules opened issue #1209: Zero-length submodule name breaks things - https://github.com/amaranth-lang/amaranth/issues/1209
<Wanda[cis]> ohh, you mean an actual empty name
<Wanda[cis]> uhhh.
<tpw_rules> yeah, sorry. it should be simple to fix, just not sure which way
<Wanda[cis]> yeah
<Wanda[cis]> we do need general rules on acceptable names actually
<Wanda[cis]> there's another outstanding bug that happens when you put a space in the name
<tpw_rules> i actually wrote a sim test cause i thought this might be a problem but it wasn't caught
<_whitenotifier-5> [amaranth-soc] tpwrules opened pull request #79: Implement RFC 57: Single-Field Register Definition and Usage Shortcut - https://github.com/amaranth-lang/amaranth-soc/pull/79
* cr1901 is going to bed for now
<Wanda[cis]> const = None is not necessarily a good plan
<Wanda[cis]> remember that with RFC 36, we're planning to use from_bits + const to implement sim get / set
<Wanda[cis]> const stops being just for simulation
<Wanda[cis]> soo... working around the non-contig field limitation
<Wanda[cis]> I actually feel that _Imm is completely unnecessary, and you could just add accessor methods on the Insn that assemble the proper fields
<Wanda[cis]> like Insn.imm_J property that reaturns the proper Cat
jjsuperpower_ has joined #amaranth-lang
jjsuperpower__ has quit [Ping timeout: 240 seconds]
<_whitenotifier-6> [amaranth] github-merge-queue[bot] created branch gh-readonly-queue/main/pr-1208-83701d74cfca7d684f1137a3eb5d277d1ef572ce - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth] wanda-phi 23f1b63 - lib.memory: Add `Signature.create` implementations.
<_whitenotifier-6> [amaranth-lang/amaranth] github-merge-queue[bot] pushed 1 commit to main [+0/-0/±2] https://github.com/amaranth-lang/amaranth/compare/83701d74cfca...23f1b6342561
<_whitenotifier-6> [amaranth] whitequark closed pull request #1208: lib.memory: Add `Signature.create` implementations. - https://github.com/amaranth-lang/amaranth/pull/1208
<_whitenotifier-6> [amaranth] github-merge-queue[bot] deleted branch gh-readonly-queue/main/pr-1208-83701d74cfca7d684f1137a3eb5d277d1ef572ce - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth-lang.github.io] whitequark pushed 1 commit to main [+0/-0/±35] https://github.com/amaranth-lang/amaranth-lang.github.io/compare/603480b03106...da6fc37607ec
<_whitenotifier-6> [amaranth-lang/amaranth-lang.github.io] github-merge-queue[bot] da6fc37 - Deploying to main from @ amaranth-lang/amaranth@23f1b63425613b13cb99c3d1881b8adaccc9983e 🚀
FFY00_ has quit [Read error: Connection reset by peer]
frgo_ has quit [Ping timeout: 252 seconds]
notgull has quit [Ping timeout: 260 seconds]
notgull has joined #amaranth-lang
notgull has quit [Ping timeout: 268 seconds]
FFY00 has joined #amaranth-lang
<_whitenotifier-5> [amaranth] github-merge-queue[bot] created branch gh-readonly-queue/main/pr-1207-23f1b63425613b13cb99c3d1881b8adaccc9983e - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth] github-merge-queue[bot] pushed 1 commit to main [+0/-0/±3] https://github.com/amaranth-lang/amaranth/compare/23f1b6342561...18b54ded0ab9
<_whitenotifier-6> [amaranth-lang/amaranth] wanda-phi 18b54de - hdl._ir: Fix fallout from #1190, add more tests.
<_whitenotifier-5> [amaranth] whitequark closed pull request #1207: hdl._ir: Fix fallout from #1190, add more tests. - https://github.com/amaranth-lang/amaranth/pull/1207
<_whitenotifier-5> [amaranth] github-merge-queue[bot] deleted branch gh-readonly-queue/main/pr-1207-23f1b63425613b13cb99c3d1881b8adaccc9983e - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth-lang.github.io] github-merge-queue[bot] 010c9da - Deploying to main from @ amaranth-lang/amaranth@18b54ded0ab9d5dc1ca7955137e8306c90b75b24 🚀
<_whitenotifier-6> [amaranth-lang/amaranth-lang.github.io] whitequark pushed 1 commit to main [+0/-0/±35] https://github.com/amaranth-lang/amaranth-lang.github.io/compare/da6fc37607ec...010c9da384f1
frgo has joined #amaranth-lang
<cr1901> Wanda[cis]: Your suggestion was similar to what I initially did. I think I got carried away w/ wanting to make my own psuedo-layouts lol. Gonna apply your suggestion.
mindw0rk has quit [Read error: Connection reset by peer]
mindw0rk has joined #amaranth-lang
notgull has joined #amaranth-lang
asjackson has left #amaranth-lang [#amaranth-lang]
GenTooMan has quit [Ping timeout: 272 seconds]
notgull has quit [Ping timeout: 264 seconds]
GenTooMan has joined #amaranth-lang
Tomz_plug[m] has joined #amaranth-lang
<Tomz_plug[m]> Hello everyone I'm a Grower and supplier of cannabis and psychedelics products such as;... (full message at <https://catircservices.org/_matrix/media/v3/download/catircservices.org/LDADUrvpKgpWLrityQFWJKCb>)
<whitequark[cis]> <cr1901> "http://gopher.wdj-consulting.com..." <- so this kind of stuff is exactly what FlexibleLayout was designed for
<zyp[m]> FlexibleLayout still doesn't let you make fields from discontinuous bits, does it?
<whitequark[cis]> nope
Tomz_plug[m] has left #amaranth-lang [#amaranth-lang]
<cr1901> I should've been specific. I meant abusing the call syntax to force a specific View subclass to be returned when an aggregate layout is used in a Signature
<cr1901> anyways, I decided against the code you linked, so it's moot
<cr1901> (But also good to know that you think it's a legitimate use of FlexibleLayout)
<whitequark[cis]> oh
<cr1901> >still doesn't let you make fields from discontinuous bits <-- I would like this, but don't think it can be done within a Layout subclass (which is expected to return Fields as part of its contract, which are contiguous)
<cr1901> It may be niche enough that properties on the view are the "least bad" thing