mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-27 22:13:55 -04:00
0b76335e8f
The common practice in C drivers is to store pointers into `driver_data` field of device IDs. The Rust code is however currently storing indices into the fields and then carry a side table that maps the index to pointers. It is much simpler to just have `DeviceId` carry the pointer like C code does. However, just doing so naively would cause a "pointers cannot be cast to integers during const eval" error, as kernel_ulong_t does not have provenance while pointers do, and Rust forbids `expose_provenance` during consteval. Work around this limitation by wrapping raw IDs in `MaybeUninit`. `MaybeUninit` is allowed to host arbitrary bytes with or without provenance, so we can just then use `unsafe` to store a pointer with provenance there. This has the same effect as changing the C-side definition to use `void*` instead of `kernel_ulong_t`, but without actually changing the C side. Signed-off-by: Gary Guo <gary@garyguo.net> Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Link: https://patch.msgid.link/20260629-id_info-v2-8-56fccbe9c5ef@garyguo.net Signed-off-by: Danilo Krummrich <dakr@kernel.org>
56 lines
1.7 KiB
Rust
56 lines
1.7 KiB
Rust
// SPDX-License-Identifier: GPL-2.0
|
|
|
|
//! Device Tree / Open Firmware abstractions.
|
|
|
|
use crate::{
|
|
bindings,
|
|
device_id::{RawDeviceId, RawDeviceIdIndex},
|
|
prelude::*,
|
|
};
|
|
|
|
/// IdTable type for OF drivers.
|
|
pub type IdTable<T> = &'static dyn kernel::device_id::IdTable<DeviceId, T>;
|
|
|
|
/// An open firmware device id.
|
|
#[repr(transparent)]
|
|
#[derive(Clone, Copy)]
|
|
pub struct DeviceId(bindings::of_device_id);
|
|
|
|
// SAFETY: `DeviceId` is a `#[repr(transparent)]` wrapper of `struct of_device_id` and
|
|
// does not add additional invariants, so it's safe to transmute to `RawType`.
|
|
unsafe impl RawDeviceId for DeviceId {
|
|
type RawType = bindings::of_device_id;
|
|
}
|
|
|
|
// SAFETY: `DRIVER_DATA_OFFSET` is the offset to the `data` field.
|
|
unsafe impl RawDeviceIdIndex for DeviceId {
|
|
const DRIVER_DATA_OFFSET: usize = core::mem::offset_of!(bindings::of_device_id, data);
|
|
}
|
|
|
|
impl DeviceId {
|
|
/// Create a new device id from an OF 'compatible' string.
|
|
pub const fn new(compatible: &'static CStr) -> Self {
|
|
let src = compatible.to_bytes_with_nul();
|
|
// Replace with `bindings::of_device_id::default()` once stabilized for `const`.
|
|
// SAFETY: FFI type is valid to be zero-initialized.
|
|
let mut of: bindings::of_device_id = unsafe { core::mem::zeroed() };
|
|
|
|
// TODO: Use `copy_from_slice` once stabilized for `const`.
|
|
let mut i = 0;
|
|
while i < src.len() {
|
|
of.compatible[i] = src[i];
|
|
i += 1;
|
|
}
|
|
|
|
Self(of)
|
|
}
|
|
}
|
|
|
|
/// Create an OF `IdTable` with an "alias" for modpost.
|
|
#[macro_export]
|
|
macro_rules! of_device_table {
|
|
($($tt:tt)*) => {
|
|
$crate::module_device_table!("of", $crate::of::DeviceId, $($tt)*);
|
|
};
|
|
}
|