Skip to content

Add patched Optiarc AD-7203A#369

Merged
superg merged 1 commit intosuperg:mainfrom
seritools:patch-1
Apr 14, 2026
Merged

Add patched Optiarc AD-7203A#369
superg merged 1 commit intosuperg:mainfrom
seritools:patch-1

Conversation

@seritools
Copy link
Copy Markdown
Contributor

@seritools seritools commented Apr 12, 2026

drive::test:

redumper (build: b708)

arguments: drive::test
warning: drive not found in the database
warning: using generic drive

drive information
  path: /dev/sg1
  inquiry: Optiarc - DVD RW AD-7203A (revision level: 1.09, vendor specific: seri-01 BT-LIGGY)
  configuration: GENERIC (read offset: +48, C2 shift: 0, pre-gap start: +0, read method: BE, sector order: DATA_C2_SUB)
  profile: CD-ROM
  read speed: <optimal>

*** DRIVE::TEST (time check: 0s)

READ CD (BE) command (audio): yes
  DATA
  DATA_C2
  DATA_C2BEB
  DATA_C2BEB_SUB
  DATA_C2BEB_SUBQ
  DATA_C2BEB_SUBRW
  DATA_C2_SUB
  DATA_C2_SUBQ
  DATA_C2_SUBRW
  DATA_SUB
  DATA_SUBQ
  DATA_SUBRW

READ CD (BE) command (data): yes
  DATA
  DATA_C2
  DATA_C2BEB
  DATA_C2BEB_SUB
  DATA_C2BEB_SUBQ
  DATA_C2BEB_SUBRW
  DATA_C2_SUB
  DATA_C2_SUBQ
  DATA_C2_SUBRW
  DATA_SUB
  DATA_SUBQ
  DATA_SUBRW

READ CD (BE) command (scrambled): yes
  DATA
  DATA_C2
  DATA_C2BEB
  DATA_C2BEB_SUB
  DATA_C2BEB_SUBQ
  DATA_C2BEB_SUBRW
  DATA_C2_SUB
  DATA_C2_SUBQ
  DATA_C2_SUBRW
  DATA_SUB
  DATA_SUBQ
  DATA_SUBRW

READ CD MSF (B9) command (audio): yes
  DATA
  DATA_C2
  DATA_C2BEB
  DATA_C2BEB_SUB
  DATA_C2BEB_SUBQ
  DATA_C2BEB_SUBRW
  DATA_C2_SUB
  DATA_C2_SUBQ
  DATA_C2_SUBRW
  DATA_SUB
  DATA_SUBQ
  DATA_SUBRW

READ CD MSF (B9) command (data): yes
  DATA
  DATA_C2
  DATA_C2BEB
  DATA_C2BEB_SUB
  DATA_C2BEB_SUBQ
  DATA_C2BEB_SUBRW
  DATA_C2_SUB
  DATA_C2_SUBQ
  DATA_C2_SUBRW
  DATA_SUB
  DATA_SUBQ
  DATA_SUBRW

READ CD MSF (B9) command (scrambled): yes
  DATA
  DATA_C2
  DATA_C2BEB
  DATA_C2BEB_SUB
  DATA_C2BEB_SUBQ
  DATA_C2BEB_SUBRW
  DATA_C2_SUB
  DATA_C2_SUBQ
  DATA_C2_SUBRW
  DATA_SUB
  DATA_SUBQ
  DATA_SUBRW

READ CD MSF (D5) command (audio): no

READ CD MSF (D5) command (data): no

READ CD MSF (D5) command (scrambled): no

READ CDDA (D8) command (audio): no

READ CDDA (D8) command (data): no

PLEXTOR lead-in: no
lead-in/pre-gap: 135 sectors
lead-out: 100+ sectors
MEDIATEK cache read (F1): no

optidump repo is update and firmware is uploaded :)

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 12, 2026

📝 Walkthrough

Walkthrough

Added a new drive configuration entry for Optiarc "DVD RW AD-7203A" (revision 1.09) to the DRIVE_DATABASE table in drive.ixx, using standard read offset and sector ordering parameters consistent with existing similar drives.

Changes

Cohort / File(s) Summary
Drive Database Configuration
drive.ixx
Added single DriveConfig entry for Optiarc DVD RW AD-7203A with revision 1.09 and standard read/sector parameters.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A new drive joins the database fold,
Optiarc's legacy, bold and gold!
One line added, parameters true,
DVD RW flourishes anew! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add patched Optiarc AD-7203A' directly reflects the main change: adding a new drive configuration entry for the Optiarc AD-7203A to the DRIVE_DATABASE.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@superg superg merged commit 666f890 into superg:main Apr 14, 2026
11 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