* [PATCH] drm/tyr: Use DRM device type alias across driver
@ 2026-03-02 20:23 Deborah Brouwer
2026-03-03 8:55 ` Alice Ryhl
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Deborah Brouwer @ 2026-03-02 20:23 UTC (permalink / raw)
To: dri-devel, rust-for-linux
Cc: daniel.almeida, aliceryhl, boris.brezillon, Deborah Brouwer
Currently Tyr defines a convenience type alias for its DRM device type,
`TyrDrmDevice` but it does not use the alias outside of `tyr/driver.rs`.
Replace `drm::Device<TyrDrmDriver>` with the alias `TyrDrmDevice` across
the driver.
This change will ease future upstream Tyr development by reducing the
diffs when multiple series are touching these files.
No functional changes are intended.
Signed-off-by: Deborah Brouwer <deborah.brouwer@collabora.com>
---
drivers/gpu/drm/tyr/file.rs | 7 +++++--
drivers/gpu/drm/tyr/gem.rs | 7 +++++--
2 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/tyr/file.rs b/drivers/gpu/drm/tyr/file.rs
index 450be5ab9aaf..31411da203c5 100644
--- a/drivers/gpu/drm/tyr/file.rs
+++ b/drivers/gpu/drm/tyr/file.rs
@@ -7,7 +7,10 @@
uapi, //
};
-use crate::driver::TyrDrmDriver;
+use crate::driver::{
+ TyrDrmDevice,
+ TyrDrmDriver, //
+};
#[pin_data]
pub(crate) struct TyrDrmFileData {}
@@ -25,7 +28,7 @@ fn open(_dev: &drm::Device<Self::Driver>) -> Result<Pin<KBox<Self>>> {
impl TyrDrmFileData {
pub(crate) fn dev_query(
- ddev: &drm::Device<TyrDrmDriver>,
+ ddev: &TyrDrmDevice,
devquery: &mut uapi::drm_panthor_dev_query,
_file: &TyrDrmFile,
) -> Result<u32> {
diff --git a/drivers/gpu/drm/tyr/gem.rs b/drivers/gpu/drm/tyr/gem.rs
index 514524ae07ef..5cd0cd9585e8 100644
--- a/drivers/gpu/drm/tyr/gem.rs
+++ b/drivers/gpu/drm/tyr/gem.rs
@@ -5,7 +5,10 @@
prelude::*, //
};
-use crate::driver::TyrDrmDriver;
+use crate::driver::{
+ TyrDrmDevice,
+ TyrDrmDriver, //
+};
/// GEM Object inner driver data
#[pin_data]
@@ -14,7 +17,7 @@ pub(crate) struct TyrObject {}
impl gem::DriverObject for TyrObject {
type Driver = TyrDrmDriver;
- fn new(_dev: &kernel::drm::Device<TyrDrmDriver>, _size: usize) -> impl PinInit<Self, Error> {
+ fn new(_dev: &TyrDrmDevice, _size: usize) -> impl PinInit<Self, Error> {
try_pin_init!(TyrObject {})
}
}
base-commit: 15da5bc9f3adab7242867db0251fe451ac3ddb72
--
2.52.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/tyr: Use DRM device type alias across driver
2026-03-02 20:23 [PATCH] drm/tyr: Use DRM device type alias across driver Deborah Brouwer
@ 2026-03-03 8:55 ` Alice Ryhl
2026-03-03 10:25 ` Boris Brezillon
2026-03-09 9:27 ` Alice Ryhl
2 siblings, 0 replies; 4+ messages in thread
From: Alice Ryhl @ 2026-03-03 8:55 UTC (permalink / raw)
To: Deborah Brouwer
Cc: dri-devel, rust-for-linux, daniel.almeida, boris.brezillon
On Mon, Mar 02, 2026 at 12:23:31PM -0800, Deborah Brouwer wrote:
> Currently Tyr defines a convenience type alias for its DRM device type,
> `TyrDrmDevice` but it does not use the alias outside of `tyr/driver.rs`.
>
> Replace `drm::Device<TyrDrmDriver>` with the alias `TyrDrmDevice` across
> the driver.
>
> This change will ease future upstream Tyr development by reducing the
> diffs when multiple series are touching these files.
>
> No functional changes are intended.
>
> Signed-off-by: Deborah Brouwer <deborah.brouwer@collabora.com>
Reviewed-by: Alice Ryhl <aliceryhl@google.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/tyr: Use DRM device type alias across driver
2026-03-02 20:23 [PATCH] drm/tyr: Use DRM device type alias across driver Deborah Brouwer
2026-03-03 8:55 ` Alice Ryhl
@ 2026-03-03 10:25 ` Boris Brezillon
2026-03-09 9:27 ` Alice Ryhl
2 siblings, 0 replies; 4+ messages in thread
From: Boris Brezillon @ 2026-03-03 10:25 UTC (permalink / raw)
To: Deborah Brouwer; +Cc: dri-devel, rust-for-linux, daniel.almeida, aliceryhl
On Mon, 2 Mar 2026 12:23:31 -0800
Deborah Brouwer <deborah.brouwer@collabora.com> wrote:
> Currently Tyr defines a convenience type alias for its DRM device type,
> `TyrDrmDevice` but it does not use the alias outside of `tyr/driver.rs`.
>
> Replace `drm::Device<TyrDrmDriver>` with the alias `TyrDrmDevice` across
> the driver.
>
> This change will ease future upstream Tyr development by reducing the
> diffs when multiple series are touching these files.
>
> No functional changes are intended.
>
> Signed-off-by: Deborah Brouwer <deborah.brouwer@collabora.com>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
> ---
> drivers/gpu/drm/tyr/file.rs | 7 +++++--
> drivers/gpu/drm/tyr/gem.rs | 7 +++++--
> 2 files changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/tyr/file.rs b/drivers/gpu/drm/tyr/file.rs
> index 450be5ab9aaf..31411da203c5 100644
> --- a/drivers/gpu/drm/tyr/file.rs
> +++ b/drivers/gpu/drm/tyr/file.rs
> @@ -7,7 +7,10 @@
> uapi, //
> };
>
> -use crate::driver::TyrDrmDriver;
> +use crate::driver::{
> + TyrDrmDevice,
> + TyrDrmDriver, //
> +};
>
> #[pin_data]
> pub(crate) struct TyrDrmFileData {}
> @@ -25,7 +28,7 @@ fn open(_dev: &drm::Device<Self::Driver>) -> Result<Pin<KBox<Self>>> {
>
> impl TyrDrmFileData {
> pub(crate) fn dev_query(
> - ddev: &drm::Device<TyrDrmDriver>,
> + ddev: &TyrDrmDevice,
> devquery: &mut uapi::drm_panthor_dev_query,
> _file: &TyrDrmFile,
> ) -> Result<u32> {
> diff --git a/drivers/gpu/drm/tyr/gem.rs b/drivers/gpu/drm/tyr/gem.rs
> index 514524ae07ef..5cd0cd9585e8 100644
> --- a/drivers/gpu/drm/tyr/gem.rs
> +++ b/drivers/gpu/drm/tyr/gem.rs
> @@ -5,7 +5,10 @@
> prelude::*, //
> };
>
> -use crate::driver::TyrDrmDriver;
> +use crate::driver::{
> + TyrDrmDevice,
> + TyrDrmDriver, //
> +};
>
> /// GEM Object inner driver data
> #[pin_data]
> @@ -14,7 +17,7 @@ pub(crate) struct TyrObject {}
> impl gem::DriverObject for TyrObject {
> type Driver = TyrDrmDriver;
>
> - fn new(_dev: &kernel::drm::Device<TyrDrmDriver>, _size: usize) -> impl PinInit<Self, Error> {
> + fn new(_dev: &TyrDrmDevice, _size: usize) -> impl PinInit<Self, Error> {
> try_pin_init!(TyrObject {})
> }
> }
>
> base-commit: 15da5bc9f3adab7242867db0251fe451ac3ddb72
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/tyr: Use DRM device type alias across driver
2026-03-02 20:23 [PATCH] drm/tyr: Use DRM device type alias across driver Deborah Brouwer
2026-03-03 8:55 ` Alice Ryhl
2026-03-03 10:25 ` Boris Brezillon
@ 2026-03-09 9:27 ` Alice Ryhl
2 siblings, 0 replies; 4+ messages in thread
From: Alice Ryhl @ 2026-03-09 9:27 UTC (permalink / raw)
To: Deborah Brouwer
Cc: dri-devel, rust-for-linux, daniel.almeida, boris.brezillon
On Mon, Mar 02, 2026 at 12:23:31PM -0800, Deborah Brouwer wrote:
> Currently Tyr defines a convenience type alias for its DRM device type,
> `TyrDrmDevice` but it does not use the alias outside of `tyr/driver.rs`.
>
> Replace `drm::Device<TyrDrmDriver>` with the alias `TyrDrmDevice` across
> the driver.
>
> This change will ease future upstream Tyr development by reducing the
> diffs when multiple series are touching these files.
>
> No functional changes are intended.
>
> Signed-off-by: Deborah Brouwer <deborah.brouwer@collabora.com>
Merged into drm-rust-next. Thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-03-09 9:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-02 20:23 [PATCH] drm/tyr: Use DRM device type alias across driver Deborah Brouwer
2026-03-03 8:55 ` Alice Ryhl
2026-03-03 10:25 ` Boris Brezillon
2026-03-09 9:27 ` Alice Ryhl
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox