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.

Sign up for GitHub

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

Jump to bottom

Fix sync issues #16

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fix sync issues #16

wants to merge 4 commits into from

Conversation

necroware
Copy link
Owner

Multiple fixes:

  • Replaced C2 capacitor with 4.7µF
  • Replaced R1/R2 with 33 Ohm resistors to raise the sync voltage to 3.3V
  • Simplified c-sync equation (this shouldn't have any effect)

@necroware
fix: replace C2 with 4,7uF
8ca875d
@necroware
issue-15: reduce resistor values
ec24745
@necroware
fix: simplify c-sync equation
0595d30 
The c-sync equation was slightly complicated and was trying to
incorporate the sync mode. The negative/positive v-sync level is
used for color mode detection and c-sync is also inverted dependent
on that. However, so far I didn't find hardware which would need that.
That could be even wrong, so for now I removed the mode from the c-sync
equation and it is again a simple XNOR, which hopefully will just work
with every mode, Hercules, CGA and HiRes-EGA.
This was linked to issues Feb 9, 2025
Auto mode detection filter doesn't work in CGA mode for C-Sync with C2=1uF #10
Open
TTL signal level for sync signals #15
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

TTL signal level for sync signals Auto mode detection filter doesn't work in CGA mode for C-Sync with C2=1uF
1 participant
@necroware