On 29.09.2016 09: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 <g...@uclinux.org> > > This solution is as good as another, so: > > Reviewed-by: Laurent Vivier <laur...@vivier.eu>
Fine for me, too, so: Reviewed-by: Thomas Huth <h...@tuxfamily.org> By the way, we still do not have a m68k maintainer yet ... Laurent, maybe you could finally send a pull request for that patch here: https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg00805.html ? Meanwhile, Peter, could you please take this patch here from Greg directly? Or should it go via qemu-trivial instead? Thomas >> --- >> 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 >> >
