Page 2 of 2

Re: New Game: Minesweeper

Posted: 07 May 2021 01:12
by under4mhz
Great. Thanks again for the feedback.

Changes:

- Added nop's between vdp address and write, as you suggested
- Increased the file size by 4 bytes to 20996

Re: New Game: Minesweeper

Posted: 07 May 2021 06:48
by Martin A
That's the MSX package ...

Re: New Game: Minesweeper

Posted: 07 May 2021 10:32
by under4mhz
Whoops. They both start with M and end with X.

Re: New Game: Minesweeper

Posted: 07 May 2021 23:12
by Martin A
I've given the new MTX version a try (alas no MSX kit here to try the other) the results unfortunately are the same, corruption on the loading screen and in the game display itself.

I've found another couple of code sequences that could cause timing issued on the MTX:

Code: Select all

A03D OUTI
A03F JP NZ,A03D
and the similar INI : JP NZ seqence a few bytes later.

For both OUTI and INI that's a 26 cycle loop on the MTX (as there's no wait states) and at 4mhz it's 6.5us. So those too might benefit from 2 NOPS to extend the access separation past 32 cycles/8us.

Re: New Game: Minesweeper

Posted: 08 May 2021 00:37
by under4mhz
Thanks again for your help. I didn't consider the CPU being at 4Mhz would change the vdu timings. I'd optimized the code for the 3.5Mhz MSX CPU.

I've added a few more wait states around the vdu decompression and copy routines.

Re: New Game: Minesweeper

Posted: 08 May 2021 12:07
by Bill B
If using the latest version of MEMU, try using the -diag-vid-time-check option to check the timings.

Re: New Game: Minesweeper

Posted: 08 May 2021 19:35
by Martin A
Success
Sweeper Loads.jpg
Sweeper Loads.jpg (56.81 KiB) Viewed 7740 times
sweeper runs.jpg
sweeper runs.jpg (68.59 KiB) Viewed 7740 times

Re: New Game: Minesweeper

Posted: 08 May 2021 20:48
by Dave
Looking good !

@under4mhz, I've taken the liberty of posting a copy on my website too - I hope you don't mind?

For the purposes of crediting you with the game, would you like your real name shown, or are you happy with the "under4mhz" handle?

http://primrosebank.net/computers/mtx/m ... dl.htm#new

regards
Dave

Re: New Game: Minesweeper

Posted: 09 May 2021 01:04
by under4mhz
Bill B wrote: 08 May 2021 12:07 If using the latest version of MEMU, try using the -diag-vid-time-check option to check the timings.
Thanks, that will be useful.

Re: New Game: Minesweeper

Posted: 09 May 2021 01:05
by under4mhz
Yes, I'm happy for you to add it to your website. The handle Under4Mhz is fine.

Thanks for your help.