On 17 September 2011 02:08, andrzej zaborowski <balr...@gmail.com> wrote: >> --- a/hw/omap_gpmc.c >> +++ b/hw/omap_gpmc.c >> @@ -284,7 +284,7 @@ static void omap_gpmc_write(void *opaque, >> target_phys_addr_t addr, >> break; >> >> case 0x018: /* GPMC_IRQSTATUS */ >> - s->irqen = ~value; >> + s->irqen &= ~value; > > Should we be clearing s->irqst here instead of irqen?
Oops, you're right... (IIRC this change was a spotted-while-reading-code one, not a response to a behavioural issue with the model.) -- PMM
