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:
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 (56.81 KiB) Viewed 34674 times

- sweeper runs.jpg (68.59 KiB) Viewed 34674 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.