From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47443) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpaRB-0003EQ-8T for qemu-devel@nongnu.org; Thu, 29 Sep 2016 08:27:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bpaR6-0001y4-59 for qemu-devel@nongnu.org; Thu, 29 Sep 2016 08:27:40 -0400 Received: from icp-osb-irony-out1.external.iinet.net.au ([203.59.1.210]:9515) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpaR5-0001wN-PX for qemu-devel@nongnu.org; Thu, 29 Sep 2016 08:27:36 -0400 References: <1475021199-28674-1-git-send-email-gerg@uclinux.org> From: Greg Ungerer Message-ID: Date: Thu, 29 Sep 2016 22:27:25 +1000 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCHv2] m68k: change default system clock for m5208evb List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Laurent Vivier , qemu-devel@nongnu.org Hi Laurent, On 29/09/16 17:50, Laurent Vivier wrote: > Le 28/09/2016 à 02:06, Greg Ungerer a écrit : >> The shipping default setting for the Freescale M5208EVB board is to run >> the CPU at 166.67MHz. The current qemu emulation code for this board is >> defaulting to 66MHz. This results in time appearing to run way to slowly. >> So a "sleep 5" in a standard ColdFire Linux build takes almost 15 >> seconds in real time to actually complete. >> >> Change the hard coded default to match the default hardware setting. >> >> Signed-off-by: Greg Ungerer > > This solution is as good as another, so: > > Reviewed-by: Laurent Vivier Thanks for the review and comments, much appreciated. Regards Greg >> --- >> hw/m68k/mcf5208.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> v2: more accurately set frequency (166666666 instead of 166000000) >> >> diff --git a/hw/m68k/mcf5208.c b/hw/m68k/mcf5208.c >> index 9240ebf..3438314 100644 >> --- a/hw/m68k/mcf5208.c >> +++ b/hw/m68k/mcf5208.c >> @@ -21,7 +21,7 @@ >> #include "elf.h" >> #include "exec/address-spaces.h" >> >> -#define SYS_FREQ 66000000 >> +#define SYS_FREQ 166666666 >> >> #define PCSR_EN 0x0001 >> #define PCSR_RLD 0x0002 >> >