libvgm icon indicating copy to clipboard operation
libvgm copied to clipboard

Published 20 hours ago • verified publisher icon ValleyBell

Not supported Sega CD PCM sounds?

Open vitamin-caig opened this issue 2 years ago • 7 comments

Hello!

Found out that Sega CD tracks sound improperly. Please check out https://vgmrips.net/packs/pack/silpheed-mega-cd

Checked also at the most recent version 8dcffa434ade3998bfbbece14

vitamin-caig avatar May 23 '22 14:05 vitamin-caig

What tracks sound incorrect and in which way? (specific songs would be nice and timestamps would be even nicer)

I listenened to a few of them using the current libvgm player and also vgmplay-libvgm and didn't notice anything that is obviously wrong.

ValleyBell avatar May 23 '22 17:05 ValleyBell

Right first one "01 - IPL ~ Opening Demo.vgz" produces no sound. Maybe, I've missed some devices definitions?

vitamin-caig avatar May 23 '22 20:05 vitamin-caig

You need to enable "RF5C68: Gens core" for MegaCD PCM. Due to historic reasons, RF5C68 MAME is used for the "RF5C68" from the VGM spec. and RF5C68 Gens is used for the "RF5C164".

I should probably relax that requirement.

ValleyBell avatar May 23 '22 21:05 ValleyBell

Both RF5C68 cores were used. I've disabled SNDDEV_SELECT (adding all necessary files to linkage) and tracks are working.

Looks like it's required to support minimal essential cores set to avoid linking redundant code.

vitamin-caig avatar May 23 '22 21:05 vitamin-caig

Closing as invalid:)

vitamin-caig avatar May 24 '22 10:05 vitamin-caig

Well... I think it's not entirely invalid. Requireing both cores for everything to work is effectively undocumented behaviour.

I'll keep it open as long as this requirement is in place.

ValleyBell avatar May 24 '22 16:05 ValleyBell

This helped me fix a similar issue in Chip Player JS. Thanks @vitamin-caig

mmontag avatar Dec 13 '23 05:12 mmontag