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

PAL SMS border is incorrect over analog video when direct_video=0 #120

Closed
birdybro opened this issue Apr 4, 2022 · 3 comments
Closed

PAL SMS border is incorrect over analog video when direct_video=0 #120

birdybro opened this issue Apr 4, 2022 · 3 comments

Comments

@birdybro
Copy link
Member

birdybro commented Apr 4, 2022

https://misterfpga.org/viewtopic.php?p=48093#p48093

I confirmed this result as well. For some odd reason direct video being enabled fixes the otherwise broken border. the border for PAL is also incorrect over HDMI which leads to the screen being filled with an incorrect aspect ratio.

2022-04-04.21-27-52.mp4

Here's a video of me switching back and forth between two alternate ini's. Both the same, but one has direct_video=1 and shows the correct border. This is using the analog IO board with vga to YPbPr going to a retrotink 5x pro scaler.

@birdybro
Copy link
Member Author

birdybro commented Apr 5, 2022

I traced it down, it's a regression linked to this commit --> cbef92c

@birdybro
Copy link
Member Author

birdybro commented Jun 20, 2022

Following up on this so I don't lose track and don't forget, the issue is related to the regression I linked, but also paulb-nl mentioned that this is due to the 5x mode being forced. Kitrinx has the workings of a fix here, others and I were testing it for her, but she has yet to submit a PR, shes' probably wanting to finish it up some more. --> Kitrinx@41c4d1e

Next, after this is all fixed, then the exact resolutions should be validated for every mode. After that a math-derived aspect ratio for the scaler can be done similar to how the Sega Genesis core has a combinational block for that --> https://github.com/MiSTer-devel/Genesis_MiSTer/blob/ad60161e0d298b794b52a5ad84be355a2cbed88c/Genesis.sv#L194-L219

birdybro added a commit to birdybro/SMS_MiSTer that referenced this issue Jul 10, 2022
The blanking changes negatively impacted PAL + border to the scaler. They must have been done incorrectly. This was also leading to direct video output over PAL with border on looking incorrect.

Fixes --> MiSTer-devel#120
@birdybro
Copy link
Member Author

This is resolved and will be available in the next release, closing.

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

No branches or pull requests

1 participant