fridtjof/qemu-cr16
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

rust/qdev: use addr_of! in QDevProp

Browse source 
We want a &raw pointer, so unsafe { &_ } is not needed.

Suggested-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Link: https://lore.kernel.org/r/20250920160520.3699591-6-zhao1.liu@intel.com
This commit is contained in:
Manos Pitsidianakis 2025-09-21 00:05:13 +08:00 • committed by Paolo Bonzini
parent 35d7735f76
commit 51d736cd71
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
rust/hw/core/src/qdev.rs
View file

@ -6,7 +6,7 @@
use std::{
ffi::{c_int, c_void, CStr, CString},
ptr::NonNull,
ptr::{addr_of, NonNull},
};
use chardev::Chardev;
@ -129,17 +129,17 @@ pub unsafe trait QDevProp {
/// Use [`bindings::qdev_prop_bool`] for `bool`.
unsafe impl QDevProp for bool {
const VALUE: *const bindings::PropertyInfo = unsafe { &bindings::qdev_prop_bool };
const VALUE: *const bindings::PropertyInfo = addr_of!(bindings::qdev_prop_bool);
}
/// Use [`bindings::qdev_prop_uint64`] for `u64`.
unsafe impl QDevProp for u64 {
const VALUE: *const bindings::PropertyInfo = unsafe { &bindings::qdev_prop_uint64 };
const VALUE: *const bindings::PropertyInfo = addr_of!(bindings::qdev_prop_uint64);
}
/// Use [`bindings::qdev_prop_chr`] for [`chardev::CharBackend`].
unsafe impl QDevProp for chardev::CharBackend {
const VALUE: *const bindings::PropertyInfo = unsafe { &bindings::qdev_prop_chr };
const VALUE: *const bindings::PropertyInfo = addr_of!(bindings::qdev_prop_chr);
}
/// Trait to define device properties.