On 18 October 2013 12:45, <alex.ben...@linaro.org> wrote: > +typedef struct { > + SysBusDevice parent_obj; > + > + MemoryRegion iomem; > + > + uint32_t alpha; > + uint32_t leds; > + uint32_t switches;
These three fields are never used, or did I miss something? > +} IntegratorDebugState; Looks good otherwise. thanks -- PMM