Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-11 Thread Mark Cave-Ayland
On 11/03/2021 09:44, Laurent Vivier wrote: Yes indeed, depending upon the documentation it is referred to as either the 60Hz or the 60.15Hz timer. Certainly that's enough information for anyone familiar with Mac internals to understand exactly what you are referring to. There are also plenty o

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-11 Thread Laurent Vivier
Le 11/03/2021 à 10:04, Mark Cave-Ayland a écrit : > On 11/03/2021 00:15, BALATON Zoltan wrote: > >> On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: >>> On 10/03/2021 13:24, Laurent Vivier wrote: Le 10/03/2021 à 14:10, Laurent Vivier a écrit : > Le 10/03/2021 à 13:56, Laurent Vivier a écrit :

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-11 Thread Mark Cave-Ayland
On 11/03/2021 00:15, BALATON Zoltan wrote: On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: On 10/03/2021 13:24, Laurent Vivier wrote: Le 10/03/2021 à 14:10, Laurent Vivier a écrit : Le 10/03/2021 à 13:56, Laurent Vivier a écrit : Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : On Wed, 10 Mar 2

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread BALATON Zoltan
On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: On 10/03/2021 13:24, Laurent Vivier wrote: Le 10/03/2021 à 14:10, Laurent Vivier a écrit : Le 10/03/2021 à 13:56, Laurent Vivier a écrit : Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: The 60Hz timer i

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Mark Cave-Ayland
On 10/03/2021 13:24, Laurent Vivier wrote: Le 10/03/2021 à 14:10, Laurent Vivier a écrit : Le 10/03/2021 à 13:56, Laurent Vivier a écrit : Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: The 60Hz timer is initialised using timer_new_ns() meaning t

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Philippe Mathieu-Daudé
On 3/10/21 2:10 PM, Laurent Vivier wrote: > Le 10/03/2021 à 13:56, Laurent Vivier a écrit : >> Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : >>> On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: The 60Hz timer is initialised using timer_new_ns() meaning that the timer interval should be meas

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Laurent Vivier
Le 10/03/2021 à 14:10, Laurent Vivier a écrit : > Le 10/03/2021 à 13:56, Laurent Vivier a écrit : >> Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : >>> On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: The 60Hz timer is initialised using timer_new_ns() meaning that the timer interval should b

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Laurent Vivier
Le 10/03/2021 à 13:56, Laurent Vivier a écrit : > Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : >> On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: >>> 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 thousa

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Laurent Vivier
Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : > On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: >> 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

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread BALATON Zoltan
On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: 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 --- hw/misc/mac_via.c | 4 ++-- 1 file changed, 2 inse

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Laurent Vivier
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 > --- > hw/misc/mac_via.c | 4 ++-- > 1