Re: [PATCH] gpu: nova: rename drivers to use kebab-case convention

John Hubbard
2026-03-11T01:34:56Z

On 3/9/26 11:24 PM, Cheng-Yang Chou wrote:
> Rename nova-core driver from "NovaCore" to "nova-core" and nova-drm
> driver from "Nova" to "nova-drm" to follow kernel naming conventions
> before these names become UAPI (e.g. exposed via driver_override).
>
> Link: https://github.com/Rust-for-Linux/linux/issues/1228
> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
> ---
> drivers/gpu/drm/nova/driver.rs | 4 ++--
> drivers/gpu/drm/nova/nova.rs | 2 +-
> drivers/gpu/nova-core/nova_core.rs | 2 +-
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/nova/driver.rs b/drivers/gpu/drm/nova/driver.rs
> index b1af0a099551..addf70c33cb2 100644
> --- a/drivers/gpu/drm/nova/driver.rs
> +++ b/drivers/gpu/drm/nova/driver.rs
> @@ -32,11 +32,11 @@ pub(crate) struct NovaData {
> major: 0,
> minor: 0,
> patchlevel: 0,
> - name: c"nova",
> + name: c"nova-drm",
> desc: c"Nvidia Graphics",

hmm, as long as we are fooling around with names here, I see that
this should be "NVIDIA Graphics and Compute", really.

Obviously that can be done separately, though.

> };
>
> -const NOVA_CORE_MODULE_NAME: &CStr = c"NovaCore";
> +const NOVA_CORE_MODULE_NAME: &CStr = c"nova-core";
> const AUXILIARY_NAME: &CStr = c"nova-drm";
>
> kernel::auxiliary_device_table!(
> diff --git a/drivers/gpu/drm/nova/nova.rs b/drivers/gpu/drm/nova/nova.rs
> index 8893e58ee0db..1fd454c7e0df 100644
> --- a/drivers/gpu/drm/nova/nova.rs
> +++ b/drivers/gpu/drm/nova/nova.rs
> @@ -10,7 +10,7 @@
>
> kernel::module_auxiliary_driver! {
> type: NovaDriver,
> - name: "Nova",
> + name: "nova-drm",

Well, but now it is out of sync with the file name: nova.rs.

You should change both, or neither, I'm thinking.

> authors: ["Danilo Krummrich"],
> description: "Nova GPU driver",
> license: "GPL v2",
> diff --git a/drivers/gpu/nova-core/nova_core.rs b/drivers/gpu/nova-core/nova_core.rs
> index c1121e7c64c5..90e98380687b 100644
> --- a/drivers/gpu/nova-core/nova_core.rs
> +++ b/drivers/gpu/nova-core/nova_core.rs
> @@ -22,7 +22,7 @@
>
> kernel::module_pci_driver! {
> type: driver::NovaCore,
> - name: "NovaCore",
> + name: "nova-core",

This bit looks good.

> authors: ["Danilo Krummrich"],
> description: "Nova Core GPU driver",
> license: "GPL v2",

thanks,
--
John Hubbard