Re: [PATCH V6 0/2] author field in module! macro should be a array
From: Andreas Hindborg
Date: Tue Mar 04 2025 - 06:59:53 EST
"Guilherme Giacomo Simoes" <trintaeoitogc@xxxxxxxxx> 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