refactor(renesas): move renesis.rs into src/renesas/ and normalize spelling

Matches the ld/ platform-folder convention (MediaTek lives in src/ld/); the
module/struct spelling is normalized renesis->renesas to match the drives it
detects. Pure move + identifier rename, no behavior change.
This commit is contained in:
Matthew Jackson
2026-08-31 12:54:30 -07:00
parent 56688764ea
commit d8f1d2f3a7
3 changed files with 16 additions and 16 deletions
+4 -4
View File
@@ -21,10 +21,10 @@ mod css;
// [`all_unlockers`]. `aacs` and `css` carry no such public catalog, so they
// stay fully private.
pub mod ld;
// `renesis` is public for its `is_renesas` drive-probe (which reports a dead
// `renesas` is public for its `is_renesas` drive-probe (which reports a dead
// bus as `Err(UnlockError::Transport)` rather than "not a Renesas drive"); the unlocker impl
// (`Renesis`) is `pub(crate)` — reached only through [`all_unlockers`].
pub mod renesis;
// (`Renesas`) is `pub(crate)` — reached only through [`all_unlockers`].
pub mod renesas;
// `freemkv` carries no public catalog (the firmware self-identifies rather
// than matching a bundled profile) and no emulation wire format yet, so it
// stays fully private — the unlocker impl (`FreemkvUnlocker`) is reached only
@@ -196,7 +196,7 @@ pub fn all_unlockers() -> Vec<Box<dyn Unlocker>> {
vec![
Box::new(freemkv::FreemkvUnlocker::new()),
Box::new(ld::Mt1959Unlocker::new()),
Box::new(renesis::Renesis::new()),
Box::new(renesas::Renesas::new()),
Box::new(aacs::AacsCert::new()),
Box::new(css::DvdUnlocker::new()),
]
+11 -11
View File
@@ -1,4 +1,4 @@
//! renesis — Renesas-platform detection (Pioneer + HL-DT-ST Renesas drives).
//! renesas — Renesas-platform detection (Pioneer + HL-DT-ST Renesas drives).
//!
//! Optical drives split into two controller families: MediaTek (handled by
//! [`crate::ld`]) and Renesas. This module identifies the Renesas side via a
@@ -63,11 +63,11 @@ pub fn is_renesas(scsi: &mut dyn ScsiTransport) -> std::result::Result<bool, Unl
/// The Renesas-platform unlocker. `pub(crate)` — reached only through
/// [`crate::all_unlockers`].
pub(crate) struct Renesis;
pub(crate) struct Renesas;
impl Renesis {
impl Renesas {
pub(crate) fn new() -> Self {
Renesis
Renesas
}
/// The Pioneer/Renesas vendor "open" — MakeMKV's live sequence, replicated in
@@ -137,7 +137,7 @@ fn read_is_good(
}
}
impl Unlocker for Renesis {
impl Unlocker for Renesas {
fn name(&self) -> &'static str {
"Renesas"
}
@@ -281,12 +281,12 @@ mod tests {
let id = crate::DriveId::default();
let ctx = UnlockCtx::new(&id, DiscKind::Unknown, &[]);
// Recognized → Ok, but a feature-only unlock: bus NOT removed, no VID.
let u = Renesis::new()
let u = Renesas::new()
.unlock_features(&mut t, &ctx)
.expect("renesas → Ok");
assert!(
!u.drive_unlocked,
"renesis does not remove the bus (cert does)"
"renesas does not remove the bus (cert does)"
);
assert_eq!(u.vid, None);
assert_eq!(u.bus_key, None);
@@ -299,7 +299,7 @@ mod tests {
let mut t = RejectingTransport;
let id = crate::DriveId::default();
let ctx = UnlockCtx::new(&id, DiscKind::Unknown, &[]);
let err = Renesis::new().unlock_features(&mut t, &ctx).unwrap_err();
let err = Renesas::new().unlock_features(&mut t, &ctx).unwrap_err();
assert_eq!(err, UnlockError::NotApplicable);
}
@@ -327,7 +327,7 @@ mod tests {
let id = crate::DriveId::default();
let ctx = UnlockCtx::new(&id, DiscKind::Unknown, &[]);
assert_eq!(
Renesis::new().unlock_features(&mut t, &ctx).unwrap_err(),
Renesas::new().unlock_features(&mut t, &ctx).unwrap_err(),
UnlockError::Transport
);
}
@@ -375,7 +375,7 @@ mod tests {
let id = crate::DriveId::default();
let ctx = UnlockCtx::new(&id, DiscKind::Unknown, &[]);
assert_eq!(
Renesis::new().unlock_features(&mut t, &ctx).unwrap_err(),
Renesas::new().unlock_features(&mut t, &ctx).unwrap_err(),
UnlockError::NotApplicable
);
}
@@ -388,7 +388,7 @@ mod tests {
};
let id = crate::DriveId::default();
let ctx = UnlockCtx::new(&id, DiscKind::Aacs, &[]);
let err = Renesis::new().unlock_bus(&mut t, &ctx).unwrap_err();
let err = Renesas::new().unlock_bus(&mut t, &ctx).unwrap_err();
assert_eq!(err, UnlockError::NotApplicable);
}
}
+1 -1
View File
@@ -107,7 +107,7 @@ pub(crate) fn build_set_cd_speed(read_speed: u16) -> [u8; 12] {
// whole class of transport-fault misclassification bugs sat uncaught (a dead bus
// read as "this unlocker doesn't apply", a zero-filled buffer read as valid
// drive data). This fixture can express all of them, and is the red-before-green
// vehicle for the transport-contract fixes across ld / aacs / css / renesis.
// vehicle for the transport-contract fixes across ld / aacs / css / renesas.
#[cfg(test)]
#[allow(dead_code)] // a fixture: each helper is used by a subset of the modules
pub(crate) mod mock {