Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V9990 bug related to R#22 #90

Open
albs-br opened this issue Nov 30, 2022 · 3 comments
Open

V9990 bug related to R#22 #90

albs-br opened this issue Nov 30, 2022 · 3 comments

Comments

@albs-br
Copy link

albs-br commented Nov 30, 2022

Hi guys, me and some fellows on MSX.org found a bug on the V9990 implementation, more specifically on R#22:

The two higher bits (bits 7 and 6) are supposed to disable layers AND sprites, not just the tile patterns layer.

The documentation states:
SDA: Set to "1" to disable layer "A" and sprites.
SDB: Set to "1" to disable layer "B" and sprites.

The discussion can be viewd here (page 4 onwards).

If necessary I can provide a test ROM showing the bug.

@ppeccin
Copy link
Owner

ppeccin commented Nov 30, 2022 via email

@albs-br
Copy link
Author

albs-br commented Jan 23, 2024

I made some ROMs to help openMSX folks to fix this same issue on openMSX: openMSX/openMSX#1466

Maybe it helps here too.

@NullPointerReference
Copy link

@albs-br I think this is an easy fix in the WebMSX code; and I downloaded all 4 example ROMs, but it's not fully clear to me what the expected 'output' on screen is (don't have a real V9990 either); given the MSX.org thread linked from the OpenMSX ticket doesn't have any pictures (anymore ?) for me to check and it you also mentioned that variant D looks different on real hardware from what you described originally / above ? And it also isn't fixed in OpenMSX 20 either.

  1. So what is the exact 'correct' output for ROMS B and C and D ?
  2. As you can see currently in WebMSX, ROM variant D still shows some 'green tile/block/sprite';
    is that expect or a / another bug too ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants