FM Sound option gone

[Threepwood]

Since the latest framework update the FM-Sound option is gone? I can't find it anymore.

[12characters]

Since the latest framework update the FM-Sound option is gone? I can't find it anymore.

I cannot find it either. If one has set the option before, it seems to still apply though.

[Sorgelig]

Let's spam everywhere!

[Threepwood]

I cannot find it either. If one has set the option before, it seems to still apply though.

I reported it on github and it is fixed already: https://github.com/MiSTer-devel/SMS_MiSTer/issues/71

https://github.com/MiSTer-devel/SMS_MiSTer/pull/72

Will surely be part of the next SMS release.

EDIT:

Let's spam everywhere!

After I posted here I remembered github and thought that it is probably better to report it as an issue there. Sorry for the inconvenience, Sorg.

[rhester72]

Was this ever actually fixed? I saw where a fix put in, but I can't get it to appear with any ROM for any region.

[12characters]

Was this ever actually fixed? I saw where a fix put in, but I can't get it to appear with any ROM for any region.

There has not been any release after the fix.

[rhester72]

Thank you - this was the push I needed to get the Quartus environment up and running.

Compiled and confirmed the fix works - thanks again!

[Threepwood]

Compiled and confirmed the fix works

Would you be willing to share the compiled core here?

[rhester72]

So far, the question (to the project lead) of being allowed to share custom-compiled cores has been raised but not answered...I'd rather not do so without his blessing, especially since the software required to compile your own is readily available and free.

[Threepwood]

I doubt that it is an issue to attach a core compiled from unmodified sources. Besides that it is a forum attachment and nothing that will be pushed to people via updater script.

In case others want to use the core I compiled:

SMS_20200714.zip

(1.09 MiB) Downloaded 336 times

[Moondandy]

So is FM sound still not added back into the core?

[Threepwood]

So is FM sound still not added back into the core?

It has been fixed in the sources five weeks ago, there just never was a fixed release.

In the compiled core that I attached "SMS FM Sound" is in "Audio & Video".

[Moondandy]

Huh, so how come nobody has released the fixed version? Seems odd the fix hasn't made it back to Main.

[wark91]

Maybe other fixes since the last release required more tests

[rhester72]

Not in this case - it was an obvious one-line fix with no check-ins since. This one confuses me too.

[Moondandy]

Is it possible Sorg has forgotten to release this? It has been sitting marked as fixed for a long time now...

As an aside, the MDFourier gang have acquired a Japanese MS and FM module so that hopefully will pave the way for more accurate FM sound on the core.

[eightbit]

I doubt that it is an issue to attach a core compiled from unmodified sources. Besides that it is a forum attachment and nothing that will be pushed to people via updater script.

In case others want to use the core I compiled: SMS_20200714.zip

Thanks a lot for this. In my case the FM sound was by default enabled...with no option to turn it off. In the USA we never had the FM sound so it just sounds weird and unnatural to us here

[Moondandy]

Has this fix still not come through to Main yet?

[rhester72]

Still not, oddly enough.

[Moondandy]

Re: MiSTer updates and changelog
Unread post by Sorgelig » Wed Aug 19, 2020 4:02 pm

SMS:
- Fix FM OSD option.
- Update the framework.

Thanks Sorg!