Skip to content

Disable DVRP workaround for compatible adapters - #915

Merged
uyjulian merged 1 commit into
ps2dev:masterfrom
uyjulian:fix_dvrp_workaround_to_0_if_gamestar
Aug 29, 2026
Merged

Disable DVRP workaround for compatible adapters#915
uyjulian merged 1 commit into
ps2dev:masterfrom
uyjulian:fix_dvrp_workaround_to_0_if_gamestar

Conversation

@uyjulian

Copy link
Copy Markdown
Member

Supersedes and fixes #914

@LAP87

LAP87 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Tested on real hardware: PAL SCPH-50004 with a clone adapter (Spartan-3 XC3S50AN + JM20330, board rev 080825V9) and a 4 TB GPT/exFAT drive.

Test build: current master (a13b5971) plus only this PR's diff, fully stock bdm/bdmfs_fatfs/ata_bd modules — none of the out-of-tree patches. Baseline for comparison: stock master fails mass0: mount on this adapter with -EIO, because the falsely asserted SPD_CAPS_DVR bit truncates capacity to 28-bit and GPT parsing rejects the disk.

Result with this PR:

PASS IOP + stock modules
PASS mount mass0:/
PASS root: entries=12 CD=1 DVD=1

The runtime detection fires correctly on this adapter (SPD_REG16(0x20) != 1), ata_dvrp_workaround is cleared, the full capacity is reported and the GPT/exFAT volume mounts and enumerates. This supersedes #914 nicely — closing that one.

Scope note for anyone reading along: data transfers beyond 128 GiB on these clones still hit two separate hardware faults (DMA address wrap and a multi-sector PIO burst limit) that this PR does not claim to address; characterization at https://github.com/LAP87/ps2-gamestar-pio.

🤖 Generated with Claude Code

@uyjulian
uyjulian merged commit 39a8992 into ps2dev:master Aug 29, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants