MEGA65 / mega65-tools

Tools and Utilities for the MEGA65 Retro Computers
GNU General Public License v3.0
28 stars 31 forks source link

m65 special SID file injection does not work #157

Closed lydon42 closed 1 year ago

lydon42 commented 1 year ago

If you give m65 a PRG that starts at $5350 or $5352 it will push a "player screen" to $400 (C64 screen addr) and then a short player routine of 56 bytes also to $400. And it will load the file to the original address.

But it will never start this player, which is assumed to be controlled by the PC as it seems.

Possible solutions:

lydon42 commented 1 year ago

There is a commented out part around m65.c:2910 (search for is_sid_tune), which is probably missing, but I don't really see why re-adding this should magically start the the player...

lydon42 commented 1 year ago

The commented out part is an artifact of the old PRG load which was replaced. And for a SID this did actually launch the SID tune. So all that is needed seems to be the right "go" monitor command at the end of the load.

lydon42 commented 1 year ago

I also think that not all SID files will work with this player, as there seems to be a confusion about start, init and play address... The question is: do we really want m65 be able to push a SID file and it plays perfectly? Or should we rather have some SID player for the MEGA65 as a normal application?

Sure, it's a nice gimmick to be able to just push a SID via m65, but then again the question is how much complexity do we want to put in here?

lydon42 commented 1 year ago

Gimmick wise this works again, so lets close for now.