sidaddress initialized wrong
drmortalwombat opened this issue · 5 comments
drmortalwombat commented
in gt2stereo.c
unsigned sidaddress = 0xd400d420;
should be
unsigned sidaddress = 0xd420d400;
so that the first SID is at address 0xd400 and the second at 0xd420
see greloc.c:
int sidAddr1 = sidaddress & 0xffff;
int sidAddr2 = (sidaddress >> 16) & 0xffff;
aquamarine-axo commented
this could possibly be why 2SID tunes export incorrectly for me then
jpage8580 commented
These should be overwritten with the values you specify when exporting. So
the initial values shouldn’t matter.
…On Sun, 19 Jun 2022 at 16:13, nicco1690 ***@***.***> wrote:
this could possibly be why 2SID tunes export incorrectly for me then
—
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYGMDBXAU3PCSRXT74GLEMTVP42J5ANCNFSM5ZGQAQCQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
drmortalwombat commented
You can only overwrite the 2nd sid, not the first, which is stuck at 0xd420
jpage8580 commented
Will take a look. I’m currently rewriting this code so that it uses unique
3SID / 4SiD 6502 code, (obviously 4 SID will not play correctly in any SID
player due to format limitations )
…On Sun, 19 Jun 2022 at 16:21, drmortalwombat ***@***.***> wrote:
You can only overwrite the 2nd sid, not the first, which is stuck at 0xd420
—
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYGMDBXOMOGEWFBYDE6U66LVP43I7ANCNFSM5ZGQAQCQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
jpage8580 commented
Fixed my local version - First SID is now forced to $D400, Second SID is
loaded from cfg file (correctly!)
Have also creates a new player9.s file for 9 channel SIDs, so no longer
needing to force / hack 2 SID files to work together. Will do the same for
the altplayer.s file and then do the same for 12 channel SID.
Sorry about the problems.
On Sun, Jun 19, 2022 at 4:52 PM Jason Page ***@***.***>
wrote:
… Will take a look. I’m currently rewriting this code so that it uses unique
3SID / 4SiD 6502 code, (obviously 4 SID will not play correctly in any SID
player due to format limitations )
On Sun, 19 Jun 2022 at 16:21, drmortalwombat ***@***.***>
wrote:
> You can only overwrite the 2nd sid, not the first, which is stuck at
> 0xd420
>
> —
> Reply to this email directly, view it on GitHub
> <#27 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AYGMDBXOMOGEWFBYDE6U66LVP43I7ANCNFSM5ZGQAQCQ>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>