On Sun, Jul 5, 2020 at 2:10 PM Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > > The 'card is readonly' and 'card inserted' IRQs are not wired. > Add a comment in case someone know where to wire them. > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
I'm not convinced adding fixmes or todos in the code is the right direction. It would be better to file bugs or use some other more official tracking mechanism. Alistair > --- > hw/lm32/milkymist.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/lm32/milkymist.c b/hw/lm32/milkymist.c > index 469e3c4322..117973c967 100644 > --- a/hw/lm32/milkymist.c > +++ b/hw/lm32/milkymist.c > @@ -87,6 +87,7 @@ static DeviceState *milkymist_memcard_create(hwaddr base) > dev = qdev_new("milkymist-memcard"); > sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal); > sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, base); > + /* FIXME wire 'card is readonly' and 'card inserted' IRQs? */ > > return dev; > } > -- > 2.21.3 > >