Re: [PATCH 1/2] hw/m68k/q800: Don't connect two qemu_irqs directly to the same input

2020-11-10 Thread Mark Cave-Ayland
On 07/11/2020 14:52, Philippe Mathieu-Daudé wrote: Cc'ing SPARC maintainers ... On 11/7/20 12:51 AM, Peter Maydell wrote: The q800 board code connects both of the IRQ outputs of the ESCC to the same pic[3] qemu_irq. Connecting two qemu_irqs outputs directly to the same input is not valid as

Re: [PATCH 1/2] hw/m68k/q800: Don't connect two qemu_irqs directly to the same input

2020-11-07 Thread Laurent Vivier
Le 07/11/2020 à 00:51, Peter Maydell a écrit : > The q800 board code connects both of the IRQ outputs of the ESCC > to the same pic[3] qemu_irq. Connecting two qemu_irqs outputs directly > to the same input is not valid as it produces subtly wrong behaviour > (for instance if both the IRQ lines

Re: [PATCH 1/2] hw/m68k/q800: Don't connect two qemu_irqs directly to the same input

2020-11-07 Thread Philippe Mathieu-Daudé
> On 11/7/20 12:51 AM, Peter Maydell wrote: >> The q800 board code connects both of the IRQ outputs of the ESCC >> to the same pic[3] qemu_irq. Connecting two qemu_irqs outputs directly >> to the same input is not valid as it produces subtly wrong behaviour >> (for instance if both the IRQ lines

Re: [PATCH 1/2] hw/m68k/q800: Don't connect two qemu_irqs directly to the same input

2020-11-07 Thread Philippe Mathieu-Daudé
Cc'ing SPARC maintainers ... On 11/7/20 12:51 AM, Peter Maydell wrote: > The q800 board code connects both of the IRQ outputs of the ESCC > to the same pic[3] qemu_irq. Connecting two qemu_irqs outputs directly > to the same input is not valid as it produces subtly wrong behaviour > (for instance

[PATCH 1/2] hw/m68k/q800: Don't connect two qemu_irqs directly to the same input

2020-11-06 Thread Peter Maydell
The q800 board code connects both of the IRQ outputs of the ESCC to the same pic[3] qemu_irq. Connecting two qemu_irqs outputs directly to the same input is not valid as it produces subtly wrong behaviour (for instance if both the IRQ lines are high, and then one goes low, the PIC input will see