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

DEV9: Add support for external HDD ID #10420

Merged
merged 1 commit into from
Dec 28, 2023
Merged

DEV9: Add support for external HDD ID #10420

merged 1 commit into from
Dec 28, 2023

Conversation

AKuHAK
Copy link
Contributor

@AKuHAK AKuHAK commented Dec 16, 2023

This commit introduces the capability to read the content of a *.hddid file located in the same directory as the HDD image (same name, different extension). The retrieved content is then used to fill the SCE_IDENTIFY_DRIVE response.

Description of Changes

Some PS2 apps requires that SCE_IDENTIFY_DRIVE not only return true, but also needs proper content of its response. The response is unique for each HDD.

Rationale behind Changes

It improves previous #9263 Next step to the PlayOnline Viewer games and BB Navigator games support.

PlayOnline Viewer games:

BB Navigator games:

Misc games:

Suggested Testing Steps

This requires specially prepared HDD images, that are not currently publicly available.

@github-actions github-actions bot added the DEV9 label Dec 16, 2023
@AKuHAK AKuHAK changed the title DEV9: Add support for external HDD ID. DEV9: Add support for external HDD ID Dec 16, 2023
@TheLastRar
Copy link
Contributor

This attempt does fix the issue I noted in the prior pr #9263 (comment), so nice work

@AKuHAK AKuHAK marked this pull request as draft December 22, 2023 13:15
This commit introduces the capability to read the content of a *.hddid file located in the same directory as the HDD image (same name, different extension). The retrieved content is then used to fill the SCE_IDENTIFY_DRIVE response.
@AKuHAK AKuHAK marked this pull request as ready for review December 22, 2023 17:22
@stenzek stenzek merged commit 8172b2e into PCSX2:master Dec 28, 2023
12 checks passed
@AKuHAK AKuHAK deleted the patch-9 branch December 28, 2023 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants