Skip to content

Add routines based on MMC READ CD - #19

Open
skr4n wants to merge 4 commits into
mainfrom
read-cd
Open

Add routines based on MMC READ CD#19
skr4n wants to merge 4 commits into
mainfrom
read-cd

Conversation

@skr4n

@skr4n skr4n commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

A direct interface to the MMC command, with all options provided.
It allows for extracting data from various parts of the disc, including sub-channel data and error correction information.

@skr4n skr4n changed the title Implement READ CD Implement MMC READ CD Jul 27, 2026
@skr4n
skr4n force-pushed the read-cd branch 3 times, most recently from c16fd18 to a4e2363 Compare July 28, 2026 09:11
@skr4n skr4n changed the title Implement MMC READ CD Add routines based on MMC READ CD Jul 28, 2026
Comment on lines +182 to +195
/// Sub-Channel information to include.
#[repr(u8)]
#[derive(Clone, Copy, Debug, Default, Eq, Hash, Ord, PartialEq, PartialOrd)]
pub enum SubchannelOption {
/// RAW P-W sub-channel data (96 bytes)
RawPw = 0b001,

/// Formatted Q sub-channel data (16 bytes).
#[default]
Q = 0b010,

/// Corrected and de-interleaved R-W sub-channel data (96 bytes).
Rw = 0b100,
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If this corresponds to

Image

of https://www.13thmonkey.org/documentation/SCSI/x3_304_1997.pdf I am seeing other fields mentioned, like 000b (no sub-channel data - which is mandatory to support).

If these are the only options supported right now, we should indicate that there are others and where one can find information on the other unsupported options.

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