Re: [PATCH 1/4] WIP: rust: Add basic KMS bindings

2024-04-25 Thread Benno Lossin
On 22.04.24 03:47, Lyude Paul wrote: > On Wed, 2024-03-27 at 20:50 +, Benno Lossin wrote: >> Hi, >> >> I just took a quick look and commented on the things that stuck >> out to me. Some general things: >> - several `unsafe` blocks have missing SAFETY comments, >> - missing documentation and exa

Re: [PATCH 1/4] WIP: rust: Add basic KMS bindings

2024-04-21 Thread Lyude Paul
On Wed, 2024-03-27 at 20:50 +, Benno Lossin wrote: > Hi, > > I just took a quick look and commented on the things that stuck > out to me. Some general things: > - several `unsafe` blocks have missing SAFETY comments, > - missing documentation and examples. This is really early on - so I had w

Re: [PATCH 1/4] WIP: rust: Add basic KMS bindings

2024-03-28 Thread Benno Lossin
Hi, I just took a quick look and commented on the things that stuck out to me. Some general things: - several `unsafe` blocks have missing SAFETY comments, - missing documentation and examples. On 22.03.24 23:03, Lyude Paul wrote: > Signed-off-by: Lyude Paul > --- > rust/bindings/bindings_helpe

[PATCH 1/4] WIP: rust: Add basic KMS bindings

2024-03-22 Thread Lyude Paul
Signed-off-by: Lyude Paul --- rust/bindings/bindings_helper.h | 4 + rust/helpers.c | 17 ++ rust/kernel/drm/device.rs| 2 + rust/kernel/drm/drv.rs | 115 +++-- rust/kernel/drm/kms.rs | 146 +++ rust/kernel/drm/kms/connector.rs | 404