Le 10/03/2021 à 09:09, Mark Cave-Ayland a écrit : > The 60Hz timer is initialised using timer_new_ns() meaning that the timer > interval should be measured in ns, and therefore its period is a thousand > times too short. > > Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> > --- > hw/misc/mac_via.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/misc/mac_via.c b/hw/misc/mac_via.c > index f994fefa7c..c6e1552a59 100644 > --- a/hw/misc/mac_via.c > +++ b/hw/misc/mac_via.c > @@ -302,8 +302,8 @@ static void via1_sixty_hz_update(MOS6522Q800VIA1State > *v1s) > MOS6522State *s = MOS6522(v1s); > > /* 60 Hz irq */ > - v1s->next_sixty_hz = (qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + 16630) / > - 16630 * 16630; > + v1s->next_sixty_hz = (qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + 16630000) / > + 16630000 * 16630000; > > if (s->ier & VIA1_IRQ_60HZ) { > timer_mod(v1s->sixty_hz_timer, v1s->next_sixty_hz); >
Reviewed-by: Laurent Vivier <laur...@vivier.eu>