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

Edtv passthrough #182

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

goutsune
Copy link

No description provided.

@gingerbeardman
Copy link
Contributor

Please consider merging this @ramapcsx2

@ramapcsx2
Copy link
Owner

Ah, I looked at this and there were problems. A lot of talk required here ><

@goutsune
Copy link
Author

goutsune commented Mar 9, 2021

Ah, I looked at this and there were problems. A lot of talk required here ><

I'm all ears, I have been using gbs-control like this for about an year with PS2 and Wii, but perhaps I forgot to change some pre/post preset flags?

@ramapcsx2
Copy link
Owner

Just found this old PR ><
Okay, I think my worry here was that I have too many obfuscated program flow paths. I'm not sure modifying the conditionals will work in all other cases (other than EDTV passthrough).
This is a bad code problem, and I can't even test it all now.
So basically, I'm worried that this will break something unintentionally, and be forever hard to debug.

@goutsune
Copy link
Author

goutsune commented Jun 8, 2022

I understand that code is messy and mine no better, so I think it's ok to let people test it and decide :D
I have been using GBS like this for a few years (+ I think I reduced some of input signal test delays to make CRT go to standby faster that in 5 or so minutes).

On a side note, I have been pondering around possibilities of true dynamic preset genetaion based on analyzed input signal — I noticed there are quite some test registers on GBS and I think it is possible to exploit clockgen remaining channels to try and catch pixel clock, kinda how LCD monitors do that when auto-matching screen.

@ramapcsx2
Copy link
Owner

I think I've probed the registers thoroughly, but it remains truly black box reversing. If you find anything new that can be used, great, but I also already tried hard :)

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

Successfully merging this pull request may close these issues.

3 participants