On Sun, 27 Feb 2022 at 18:25, Richard Henderson <richard.hender...@linaro.org> wrote: > > Drop irq_pending boolean.
It wasn't a boolean, it was a uint32_t. > Drop helper_check_interrupts. > Move checks for irq disabled into nios2_cpu_exec_interrupt. > End the TB on writes to ienable, just like to status. > > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> The code changes look good to me, but I think it would be useful to have the commit message include a brief sketch of how the new interrupt-handling design works and mention that this means nios2 now works like the other targets. At the moment the commit message is all "what" (which you can get by looking at the patch) and no "why". Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM
