Skip to content

Commit 7914ae5

Browse files
committed
rust: kernel: of: Fix !CONFIG_OF build
of_reserved_mem_region_to_resource_byname() has no empty implementation for !OF. Signed-off-by: Janne Grunau <[email protected]>
1 parent ab5645b commit 7914ae5

File tree

1 file changed

+20
-12
lines changed

1 file changed

+20
-12
lines changed

rust/kernel/of.rs

Lines changed: 20 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -214,20 +214,28 @@ impl Node {
214214
}
215215
}
216216

217+
#[allow(unused_variables)]
217218
/// Get a reserved memory region as a resource
218219
pub fn reserved_mem_region_to_resource_byname(&self, name: &CStr) -> Result<Resource> {
219-
let res = Resource::zeroed();
220-
// SAFETY: This function is safe to call as long as the arguments are valid pointers.
221-
let ret = unsafe {
222-
bindings::of_reserved_mem_region_to_resource_byname(
223-
self.raw_node,
224-
name.as_char_ptr(),
225-
res.as_raw(),
226-
)
227-
};
228-
to_result(ret)?;
229-
230-
Ok(res)
220+
#[cfg(not(CONFIG_OF))]
221+
{
222+
Err(ENOENT)
223+
}
224+
#[cfg(CONFIG_OF)]
225+
{
226+
let res = Resource::zeroed();
227+
// SAFETY: This function is safe to call as long as the arguments are valid pointers.
228+
let ret = unsafe {
229+
bindings::of_reserved_mem_region_to_resource_byname(
230+
self.raw_node,
231+
name.as_char_ptr(),
232+
res.as_raw(),
233+
)
234+
};
235+
to_result(ret)?;
236+
237+
Ok(res)
238+
}
231239
}
232240

233241
#[allow(unused_variables)]

0 commit comments

Comments
 (0)