Jean-François Nguyen

"TODO" isn't a very convincing pitch.

- RFC PR: https://github.com/amaranth-lang/rfcs/pull/16 - Amaranth SoC PR: https://github.com/amaranth-lang/amaranth-soc/pull/40 - [x] Implementation: amaranth-lang/amaranth-soc#40 - [x] Fix amaranth-lang/amaranth-soc#69 - [ ] Reference documentation - [ ] Guide-level documentation

Currently, `csr.Builder` works around this by casting array indices to strings calling `MemoryMap.add_resource()`. Repro: ```python3 from amaranth import * from amaranth_soc import csr class FooRegister(csr.Register, access="r"): a: csr.Field(csr.action.R, unsigned(8)) regs...

Add annotations for memory maps, wishbone and CSR primitives.

1

Depends on PR amaranth-lang/amaranth#978. `MemoryMap` resources (e.g. `csr.Element`s) are now assumed to have a `.signature` property, which lets them hold annotations. I am not sure whether we should enforce this...

Peripheral API design: exposing bus interfaces

7

Peripherals are a currently missing building block from nmigen-soc. They would provide wrappers to cores by means of a CSR interface (also interrupts, but handling these could be the subject...

Make optional properties return None while uninitialized.

1

The previous behaviour was to raise a NotImplementedError. It is now the responsibility of the caller to check if the property was initialized or not. Affected properties: * wishbone.Interface.memory_map *...

csr.periph: add Peripheral base class.

5

This PR aims to add support for CSR-capable peripherals. Related issue: #10 A new base class `csr.Peripheral` can be subclassed to provide nmigen-soc peripherals with helpers for managing CSR registers...

Repro: ```python3 from nmigen import * from nmigen.back import rtlil from nmigen_soc import wishbone class Top(Elaboratable): def elaborate(self, platform): m = Module() m.submodules.dec = dec = wishbone.Decoder(addr_width=5, data_width=8, granularity=8) bus...

wishbone.bus: add Arbiter.

2

This arbiter should be compatible with all the optional features of `wishbone.Interface`. The round-robin implementation could perhaps be moved to `nmigen.lib` and imported back here, but that would make nmigen-soc...

`AttributeError` caused by `UnusedElaboratable` after an elaboratable has failed its initialization

1

Repro: ```python3 from amaranth import * from amaranth.lib import wiring from amaranth.hdl import Fragment class Foo(wiring.Component): def __init__(self, arg): if not isinstance(arg, int): raise TypeError self._arg = arg super().__init__({}) def...