[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <875xkphuqh.fsf@kernel.org>
Date: Tue, 04 Mar 2025 12:59:34 +0100
From: Andreas Hindborg <a.hindborg@...nel.org>
To: "Guilherme Giacomo Simoes" <trintaeoitogc@...il.com>
Cc: <alex.gaynor@...il.com>, <aliceryhl@...gle.com>, <apw@...onical.com>,
<arnd@...db.de>, <aswinunni01@...il.com>, <axboe@...nel.dk>,
<benno.lossin@...ton.me>, <bhelgaas@...gle.com>,
<bjorn3_gh@...tonmail.com>, <boqun.feng@...il.com>, <dakr@...nel.org>,
<dwaipayanray1@...il.com>, <ethan.twardy@...il.com>,
<fujita.tomonori@...il.com>, <gary@...yguo.net>,
<gregkh@...uxfoundation.org>, <joe@...ches.com>,
<lukas.bulwahn@...il.com>, <ojeda@...nel.org>, <pbonzini@...hat.com>,
<tmgross@...ch.edu>, <walmeida@...rosoft.com>,
<rust-for-linux@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V6 0/2] author field in module! macro should be a array
"Guilherme Giacomo Simoes" <trintaeoitogc@...il.com> writes:
> In the module! macro, the author field is currently of type String.
> Since modules can have multiple authors, this limitation prevents
> specifying more than one.
>
> - Change the author field to Option<Vec<String>> to allow creating
> modules with multiple authors.
>
> - rename the field from author to authors to make it explicit that it
> can refer to multiple authors.
>
> - In modules that use the author field, update its value to an array of
> strings, and also rename it from author to authors.
>
> - Change the checkpatch.pl to find poorly formatted arrays in the macro
> module!
>
> ---
> v6 changes
> - Use consistent style in warn messages
> - Improved the commit title for checkpatch changes
> ---
>
> Guilherme Giacomo Simoes (2):
> rust: module: change author to an array
> checkpatch: check format of Vec<String> in modules
>
> drivers/block/rnull.rs | 2 +-
> drivers/net/phy/ax88796b_rust.rs | 2 +-
> drivers/net/phy/qt2025.rs | 2 +-
> rust/kernel/net/phy.rs | 4 +--
> rust/kernel/pci.rs | 2 +-
> rust/kernel/platform.rs | 2 +-
> rust/macros/lib.rs | 6 ++--
> rust/macros/module.rs | 12 ++++----
> samples/rust/rust_driver_faux.rs | 2 +-
> samples/rust/rust_driver_pci.rs | 2 +-
> samples/rust/rust_driver_platform.rs | 2 +-
> samples/rust/rust_minimal.rs | 2 +-
> samples/rust/rust_misc_device.rs | 2 +-
> samples/rust/rust_print_main.rs | 2 +-
> scripts/checkpatch.pl | 43 ++++++++++++++++++++++++++++
> 15 files changed, 66 insertions(+), 21 deletions(-)
What is the base commit for this one?
Best regards,
Andreas Hindborg
Powered by blists - more mailing lists