From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39883) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZMHGS-0005mB-9h for qemu-devel@nongnu.org; Mon, 03 Aug 2015 11:02:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZMHGO-0006yv-1d for qemu-devel@nongnu.org; Mon, 03 Aug 2015 11:02:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:41488) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZMHGN-0006yn-St for qemu-devel@nongnu.org; Mon, 03 Aug 2015 11:02:51 -0400 References: <1438609948-3744-1-git-send-email-lvivier@redhat.com> <55BF7A7E.1050303@redhat.com> <20150803144637.GQ29283@redhat.com> From: Paolo Bonzini Message-ID: <55BF8298.9080409@redhat.com> Date: Mon, 3 Aug 2015 17:02:48 +0200 MIME-Version: 1.0 In-Reply-To: <20150803144637.GQ29283@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] i6300esb: correctly convert watchdog clock ticks into nanoseconds List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Richard W.M. Jones" , Laurent Vivier Cc: qemu-devel@nongnu.org, David Gibson On 03/08/2015 16:46, Richard W.M. Jones wrote: >> > I'm wondering if a 33 Mhz clock is 33000000 Hz or 33333333 Hz ? > From the datasheet (chapter 16): > > https://www-ssl.intel.com/content/www/us/en/chipsets/6300esb-io-controller-hub-datasheet.html > > it says "33 MHz clock (30 ns clock ticks)" which is contradictory. I found that the spec allows for any speed up to 33333333 Hz (30 ns cycle), so both are okay. However, at least hw/net/rtl8139.c assumes it's 33000000 Hz, so it's nice to be consistent. Paolo