[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Z9ZQV7EbzeOPsUUU@Mac.home>
Date: Sat, 15 Mar 2025 21:15:19 -0700
From: Boqun Feng <boqun.feng@...il.com>
To: Antonio Hickey <contact@...oniohickey.com>
Cc: a.hindborg@...nel.org, alex.gaynor@...il.com, aliceryhl@...gle.com,
benno.lossin@...ton.me, bhelgaas@...gle.com,
bjorn3_gh@...tonmail.com, brendan.higgins@...ux.dev,
dakr@...nel.org, davidgow@...gle.com, fujita.tomonori@...il.com,
gary@...yguo.net, gregkh@...uxfoundation.org,
kunit-dev@...glegroups.com, linux-block@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-kselftest@...r.kernel.org,
linux-pci@...r.kernel.org, netdev@...r.kernel.org, ojeda@...nel.org,
rafael@...nel.org, rmoar@...gle.com, rust-for-linux@...r.kernel.org,
tmgross@...ch.edu
Subject: Re: Re: [PATCH v3 2/3] rust: replace `addr_of[_mut]!` with `&raw
[mut]`
On Sun, Mar 16, 2025 at 02:57:15AM +0000, Antonio Hickey wrote:
> On Fri, Mar 14, 2025 at 05:18:17PM -0700, Boqun Feng wrote:
> > Besides, it'll be easy to review if you can split the changes into
> > multiple patches. Thanks!
>
>
> Hey Boqun,
>
> Thanks for the review, do you think splitting this patch
> into a seperate patch for each file would be best?
>
That'll be my suggestion. Although you could group a few changes
together for example, changes on rust/kernel/init.rs and
rust/kernel/init/macro.rs can be put into one patch. Thanks!
Regards,
Boqun
> Thanks,
> Antonio
>
Powered by blists - more mailing lists