Nano SwinSID upgrades (paddles, etc...)

Ahh, yes I've figured out ---- the code checks if the voice gates are equal, then if it was shut for a short time AND -- this is the latest fix -- check, that the gate was not shut in the first place before the ADSR update route has been taken. --- Now it eliminates the stuck voices in 'Fanta in space' too
(actually, the interrupt handler can cope with the few additional cycles by the sbrc and mov, no need additional hardware)
Will upload some proof video later this Sunday.

Please share, what other game/music has hard time with the swinsid and I will look into it. (actually still no clue how the music being calculated, but the control-lines are kind of clear now)

And now I can go back and fix up the ADCs
For reading, the additional components cost is only a few cents so I don't know what's the problem with it...

I know it's not the best solution, but i assure you, it's working. No one can make perfect SID emulation on just 32Mhz single core cpu.. not a chance! It would require that the total calculation of every ADSR, OSC, filter should be done in 32 cycle....
But making a fair enough emulation, it is still a viable alternative. So why NOT check more than absolutely necessary is essential to keep low any overhead. If the int handler looks for transition it had to jump to a less tight space, look-up the old value, compare, branch and store - and for every single ctrl register. My code works because most of the time the main loop can do at least one loop before the same registers updated. However in 'Lazy Jones' there is too little time between gate close and reopen, so for the main loop it often looks like nothing has changed.
Yes, sure, other access will overwrite the address, if the data ends with 0, but will it really change the output? NOT! If it's not gate, then the code for voice just ignores it. You can write code were you bang a lot of data to the SID in rapid succession, but that won't affect that much either, because a) it doesn't audible - gate-on-off multiple times on the same channel - only the last gate_on matters and the fact that there were an off too b) the main loop goes through one more time - updating a channel requires quite some register updates and the gate is the last one so there is time to catch it in the regular way c) "banging" multiple voice gates in rapid succession - it can "hurt", but i yet to find a player to do this. (why the heck would someone restart all voices one-by-one when it's easier to close them all and turn on all after -- and by doing so, the main loop may catch up with them) d) REU DMA may "hurt" too in some cases -- I can't test....

What problem I still have is in 'Master of the Lamp' there is some hissing/screeching sound (sames as in frodo emulator) where in real SID (and in VICE) there is a decreasing frequency sweep (beginning in the flying stage 1 / from SID voice 2) I have no idea what or how to fix it tough....

Here is the proof of my my fix:

And also there is the firmware, compatible with nanoSwinSID -- please report back, if it's now fixes other games/musics and/or if it's breaks anothers

Zitat von Nobody

Btw, do I need to change anything at the hardware or only to flash the firmware?

Just update the flash, no need to change even the fuse bits (use the same as you used before) and you're done

Looked into '180' and that is exactly the code to break my "lazy fix" -- sort gate close-and-open and update of other registers, which I haven't thought about. It can then overwrite the temp address, so the note restart doesn't happens.
Still has more notes than before... just not all

Well, I have already thought about another solution before this one, but maybe it has it's downfalls too: instead of using some temp, just set the ctrl registers to an "invalid" state (0xff) after processed, and check if it still 0xff or some more useful value (copy the old values to processing if still 0xff). So instead of hunting gate-close events, now the gate-open ones are enough (as this is the more interesting phase of the ADSR). Sadly, it still makes a mushy result in '180' - now the notes are there, but doesn't cuts off at the end, making them blend together
Maybe it needs a bit cleanup in the code as this invalid state is kinda dirty hack into the current check....
It also breaks something in 'Fanta in space' - the digi plays fine, but the normal notes are way off - volume and sound too.

Zitat von CodeKiller

but maybe it has it's downfalls too

Oh yes, it retriggers the notes even if no gate close has received before.

Alternatively, if the int-handler stores the bytes ends with 0 to a second set of registers (addr+0x20 - with the internal registers moved further) may help----- however in PCINT, I can't go below 13cycle - most likely the turning point of the good-enough and the not-so-good (previous code with 12 cycle worked without a hick-up). Maybe a faster osc can help...
OOOOH FFS!!!! It was the ORI f#cking up the SREG during the interrupt, that caused all the glitchy sound. After the SREG saving and restoring, now '180' sounds as intended too.

So it combines all the previous fixes with some new one:
- saving if the data ends with bit0 =0 (int handler)
- save to a new set of registers too (the old registers also updated, but if the data is even, it stores to a new set too) (int handler)
- check if the new ctrl_regs has valid values (main loop)
- if valid and the current ctrl not set to gate-close, then the voice can be retriggered / and the new_ctrl set to invalid (0xFF) again (main loop)

I think all the problematic SIDs Swinkels posted now plays fine.

Now i have to squeeze it into the old INT0 handler if I want to support the old hardware --- if I can.....

So, any more problematic music out there?

(Master of the Lamp issue is not related to control flow, so I'm not sure how to fix it... also would be better to have a real RND generator instead of the loop, as is quite noticeable..)

Sorry, no support for old hardware--- the interrupt handler is too long there, unless external hardware added (an inverter gate for the CLK, and a 3-input OR gate for CS/W-R/and the inverted CLK)---not yet tested....
What is tested is a longer, jumpy routine which is so long that it does misses out updates.

On the new HW tough it still needs externals - now there is an OR gate and a JK Flip-flop for edge detection, because the PCINT works a little different.
But at least it doesn't need to fiddle with the data's and address' bits.

I think ´Chase that feeling´ from Julian_Jaymz sounds sh!t even with good, original SID
(confirmed with 8580 and 6581r? )