On 6/20/20 4:07 PM, Philippe Mathieu-Daudé wrote: > @@ -25,6 +25,7 @@ typedef struct { > > /*< public >*/ > MemoryRegion iomem; > + DeviceState *led[8];
LEDState? > + for (size_t i = 0; i < ARRAY_SIZE(s->led); i++) { > + led_set_state(LED(s->led[i]), !!extract32(value, i, 1)); > + } No need for !!. r~