superrob1500
/
mirror-linux
mirror of https://github.com/torvalds/linux
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

rust: regulator: use `to_result` for error handling 

Simplifies error handling by replacing the manual check
of the return value with the `to_result` helper.

Signed-off-by: Onur Özkan <work@onurozkan.dev>
Reviewed-by: Daniel Almeida <daniel.almeida@collabora.com>
Message-ID: <20250821090720.23939-1-work@onurozkan.dev>
Signed-off-by: Mark Brown <broonie@kernel.org>
pull/1354/merge
Onur Özkan 2025-08-21 12:07:20 +03:00 committed by Mark Brown
parent ec0be3cdf4
commit e2ab5f600b
No known key found for this signature in database
GPG Key ID: 24D68B725D5487D0
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
rust/kernel/regulator.rs
View File

@ -267,11 +267,8 @@ impl<T: RegulatorState> Regulator<T> {
pub fn get_voltage(&self) -> Result<Voltage> { pub fn get_voltage(&self) -> Result<Voltage> {
// SAFETY: Safe as per the type invariants of `Regulator`. // SAFETY: Safe as per the type invariants of `Regulator`.
let voltage = unsafe { bindings::regulator_get_voltage(self.inner.as_ptr()) }; let voltage = unsafe { bindings::regulator_get_voltage(self.inner.as_ptr()) };
if voltage < 0 {
Err(kernel::error::Error::from_errno(voltage)) to_result(voltage).map(|()| Voltage::from_microvolts(voltage))
} else {
Ok(Voltage::from_microvolts(voltage))
}
} }
fn get_internal(dev: &Device, name: &CStr) -> Result<Regulator<T>> { fn get_internal(dev: &Device, name: &CStr) -> Result<Regulator<T>> {