Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CDVD: Fix OSDSYS Audio CD regressions. #12032

Merged
merged 5 commits into from
Nov 25, 2024
Merged

CDVD: Fix OSDSYS Audio CD regressions. #12032

merged 5 commits into from
Nov 25, 2024

Conversation

F0bes
Copy link
Member

@F0bes F0bes commented Nov 24, 2024

Description of Changes

Fixes a regression introduced back in 2022, preventing OSDSYS from playing CD audio longer than one second.
Fixes another regression regarding SUBQ and an off-by-one in the getTOC implementation

Rationale behind Changes

Accuracy is good. Fixes the OSDSYS CD player for those who like watching that little cube spin around.
Fixes #4178

Suggested Testing Steps

Use an audio CD in OSDSYS and see if it plays music or stops after 1 second.
See if dance factory still works. EDIT: It does.

There is still an existing issue with the tracks.
@github-actions github-actions bot added the SPU2 label Nov 24, 2024
@github-actions github-actions bot added the CDVD label Nov 24, 2024
@F0bes F0bes changed the title CDVD: Fix OSDSYS Audio CD regression. CDVD: Fix OSDSYS Audio CD regressions. Nov 24, 2024
@knight-ryu12
Copy link
Contributor

Mounting via actual CDVD works well; recognized CD perfectly and playback in OSDSYS works.

But there is one issue that I can't mount "split BIN+CUE" file.
seems that check of discs for game precedes before detecting as Audio discs.

@Mrlinkwii
Copy link
Contributor

But there is one issue that I can't mount "split BIN+CUE" file. seems that check of discs for game precedes before detecting as Audio discs.

this pr dose not fix this you need to mount the bin cue on a virtually mounted drive , on linux use Gcdemu can do this or windows automatically

Copy link
Contributor

@Mrlinkwii Mrlinkwii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested with virtually mounted bin cue , works really well , i think most of the issues are fixed re: OSDSYS Audio CD regressions , i think this may also fix #4178

Unknown.Game_20241125121634.mp4

@StillPlay2
Copy link

But there is one issue that I can't mount "split BIN+CUE" file. seems that check of discs for game precedes before detecting as Audio discs.

this pr dose not fix this you need to mount the bin cue on a virtually mounted drive , on linux use Gcdemu can do this or windows automatically

It needs to be able to mount, and should be able to. There is some game that REQUIRES CD-Audio to be mounted (DJBox) And PS1 CD-DA games, and PS2 Multi tracked games. Which ISO won't and shouldn't cover.

PCSX2 can mount BIN+CUE, but fails to recognise files being audio, which may need to fix in preceding PR or this PR (which, I think it should qualify as a regression)

If you mean playing the .bin file directly from the .cue file then cue support hasn't been implemented yet.
In the case of Dance Factory & games with Red Book Audio, you can mount the .cue with Daemon Tools, PowerISO etc & it will read the proceeding files associated.

@knight-ryu12
Copy link
Contributor

Yes I figured that out. Sorry 😔

@F0bes F0bes merged commit c9642b7 into master Nov 25, 2024
22 checks passed
@F0bes F0bes deleted the osdsys-cd branch November 25, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG - BIOS] Audio CDs doesn't work anymore.
4 participants