1. Patchew
  2. linux
  3. [PATCH v2 0/2] gpu: nova: rename drivers and directory to use kebab-case convention
  4. View patch

[PATCH v2 2/2] gpu: nova-drm: rename nova/ directory to nova-drm/

Cheng-Yang Chou posted 2 patches 3 weeks ago
[PATCH v2 2/2] gpu: nova-drm: rename nova/ directory to nova-drm/
Posted by Cheng-Yang Chou 3 weeks ago 
Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
to nova_drm.rs (matching the object name nova_drm.o) to follow the
kebab-case naming convention already used by the auxiliary device name.

Update MAINTAINERS, Kconfig, and Makefile references accordingly.
Update the Kconfig help text to reflect the new module name nova-drm.

Link: https://github.com/Rust-for-Linux/linux/issues/1228
Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
---
 MAINTAINERS                                            | 6 +++---
 drivers/gpu/drm/Kconfig                                | 2 +-
 drivers/gpu/drm/Makefile                               | 2 +-
 drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
 drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
 drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
 drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
 drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
 drivers/gpu/drm/{nova/nova.rs => nova-drm/nova_drm.rs} | 0
 drivers/gpu/drm/nova/Makefile                          | 3 ---
 10 files changed, 9 insertions(+), 9 deletions(-)
 rename drivers/gpu/drm/{nova => nova-drm}/Kconfig (84%)
 create mode 100644 drivers/gpu/drm/nova-drm/Makefile
 rename drivers/gpu/drm/{nova => nova-drm}/driver.rs (100%)
 rename drivers/gpu/drm/{nova => nova-drm}/file.rs (100%)
 rename drivers/gpu/drm/{nova => nova-drm}/gem.rs (100%)
 rename drivers/gpu/drm/{nova/nova.rs => nova-drm/nova_drm.rs} (100%)
 delete mode 100644 drivers/gpu/drm/nova/Makefile

diff --git a/MAINTAINERS b/MAINTAINERS
index 7b277d5bf3d1..7ee486627420 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8201,7 +8201,7 @@ B:	https://gitlab.freedesktop.org/drm/nova/-/issues
 C:	irc://irc.oftc.net/nouveau
 T:	git https://gitlab.freedesktop.org/drm/rust/kernel.git drm-rust-next
 F:	Documentation/gpu/nova/
-F:	drivers/gpu/drm/nova/
+F:	drivers/gpu/drm/nova-drm/
 F:	include/uapi/drm/nova_drm.h
 
 DRM DRIVER FOR OLIMEX LCD-OLINUXINO PANELS
@@ -8484,7 +8484,7 @@ X:	drivers/gpu/drm/i915/
 X:	drivers/gpu/drm/kmb/
 X:	drivers/gpu/drm/mediatek/
 X:	drivers/gpu/drm/msm/
-X:	drivers/gpu/drm/nova/
+X:	drivers/gpu/drm/nova-drm/
 X:	drivers/gpu/drm/radeon/
 X:	drivers/gpu/drm/tegra/
 X:	drivers/gpu/drm/tyr/
@@ -8496,7 +8496,7 @@ M:	Alice Ryhl <aliceryhl@google.com>
 S:	Supported
 W:	https://drm.pages.freedesktop.org/maintainer-tools/drm-rust.html
 T:	git https://gitlab.freedesktop.org/drm/rust/kernel.git
-F:	drivers/gpu/drm/nova/
+F:	drivers/gpu/drm/nova-drm/
 F:	drivers/gpu/drm/tyr/
 F:	drivers/gpu/nova-core/
 F:	rust/kernel/drm/
diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
index d3d52310c9cc..958fa142b6cd 100644
--- a/drivers/gpu/drm/Kconfig
+++ b/drivers/gpu/drm/Kconfig
@@ -308,7 +308,7 @@ source "drivers/gpu/drm/mgag200/Kconfig"
 source "drivers/gpu/drm/msm/Kconfig"
 source "drivers/gpu/drm/mxsfb/Kconfig"
 source "drivers/gpu/drm/nouveau/Kconfig"
-source "drivers/gpu/drm/nova/Kconfig"
+source "drivers/gpu/drm/nova-drm/Kconfig"
 source "drivers/gpu/drm/omapdrm/Kconfig"
 source "drivers/gpu/drm/panel/Kconfig"
 source "drivers/gpu/drm/panfrost/Kconfig"
diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
index ec2c5ff82382..acdec9e30a27 100644
--- a/drivers/gpu/drm/Makefile
+++ b/drivers/gpu/drm/Makefile
@@ -186,7 +186,7 @@ obj-$(CONFIG_DRM_VMWGFX)+= vmwgfx/
 obj-$(CONFIG_DRM_VGEM)	+= vgem/
 obj-$(CONFIG_DRM_VKMS)	+= vkms/
 obj-$(CONFIG_DRM_NOUVEAU) +=nouveau/
-obj-$(CONFIG_DRM_NOVA) += nova/
+obj-$(CONFIG_DRM_NOVA) += nova-drm/
 obj-$(CONFIG_DRM_EXYNOS) +=exynos/
 obj-$(CONFIG_DRM_ROCKCHIP) +=rockchip/
 obj-$(CONFIG_DRM_GMA500) += gma500/
diff --git a/drivers/gpu/drm/nova/Kconfig b/drivers/gpu/drm/nova-drm/Kconfig
similarity index 84%
rename from drivers/gpu/drm/nova/Kconfig
rename to drivers/gpu/drm/nova-drm/Kconfig
index 3e637ad7b5ba..48a731bff9b1 100644
--- a/drivers/gpu/drm/nova/Kconfig
+++ b/drivers/gpu/drm/nova-drm/Kconfig
@@ -13,4 +13,4 @@ config DRM_NOVA
 
 	  This driver is work in progress and may not be functional.
 
-	  If M is selected, the module will be called nova.
+	  If M is selected, the module will be called nova-drm.
diff --git a/drivers/gpu/drm/nova-drm/Makefile b/drivers/gpu/drm/nova-drm/Makefile
new file mode 100644
index 000000000000..bd3a5e703742
--- /dev/null
+++ b/drivers/gpu/drm/nova-drm/Makefile
@@ -0,0 +1,3 @@
+# SPDX-License-Identifier: GPL-2.0
+
+obj-$(CONFIG_DRM_NOVA) += nova_drm.o
diff --git a/drivers/gpu/drm/nova/driver.rs b/drivers/gpu/drm/nova-drm/driver.rs
similarity index 100%
rename from drivers/gpu/drm/nova/driver.rs
rename to drivers/gpu/drm/nova-drm/driver.rs
diff --git a/drivers/gpu/drm/nova/file.rs b/drivers/gpu/drm/nova-drm/file.rs
similarity index 100%
rename from drivers/gpu/drm/nova/file.rs
rename to drivers/gpu/drm/nova-drm/file.rs
diff --git a/drivers/gpu/drm/nova/gem.rs b/drivers/gpu/drm/nova-drm/gem.rs
similarity index 100%
rename from drivers/gpu/drm/nova/gem.rs
rename to drivers/gpu/drm/nova-drm/gem.rs
diff --git a/drivers/gpu/drm/nova/nova.rs b/drivers/gpu/drm/nova-drm/nova_drm.rs
similarity index 100%
rename from drivers/gpu/drm/nova/nova.rs
rename to drivers/gpu/drm/nova-drm/nova_drm.rs
diff --git a/drivers/gpu/drm/nova/Makefile b/drivers/gpu/drm/nova/Makefile
deleted file mode 100644
index 42019bff3173..000000000000
--- a/drivers/gpu/drm/nova/Makefile
+++ /dev/null
@@ -1,3 +0,0 @@
-# SPDX-License-Identifier: GPL-2.0
-
-obj-$(CONFIG_DRM_NOVA) += nova.o
-- 
2.48.1
Re: [PATCH v2 2/2] gpu: nova-drm: rename nova/ directory to nova-drm/
Posted by John Hubbard 2 weeks, 6 days ago 
On 3/16/26 5:08 AM, Cheng-Yang Chou wrote:
> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
> to nova_drm.rs (matching the object name nova_drm.o) to follow the
> kebab-case naming convention already used by the auxiliary device name.
> 
> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
> Update the Kconfig help text to reflect the new module name nova-drm.
> 
> Link: https://github.com/Rust-for-Linux/linux/issues/1228
> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>

The subject line should be changed to:

   gpu: nova-drm: rename nova to nova-drm


Reviewed-by: John Hubbard <jhubbard@nvidia.com>

thanks,
-- 
John Hubbard

> ---
>   MAINTAINERS                                            | 6 +++---
>   drivers/gpu/drm/Kconfig                                | 2 +-
>   drivers/gpu/drm/Makefile                               | 2 +-
>   drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>   drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>   drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>   drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>   drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
>   drivers/gpu/drm/{nova/nova.rs => nova-drm/nova_drm.rs} | 0
>   drivers/gpu/drm/nova/Makefile                          | 3 ---
>   10 files changed, 9 insertions(+), 9 deletions(-)
>   rename drivers/gpu/drm/{nova => nova-drm}/Kconfig (84%)
>   create mode 100644 drivers/gpu/drm/nova-drm/Makefile
>   rename drivers/gpu/drm/{nova => nova-drm}/driver.rs (100%)
>   rename drivers/gpu/drm/{nova => nova-drm}/file.rs (100%)
>   rename drivers/gpu/drm/{nova => nova-drm}/gem.rs (100%)
>   rename drivers/gpu/drm/{nova/nova.rs => nova-drm/nova_drm.rs} (100%)
>   delete mode 100644 drivers/gpu/drm/nova/Makefile
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7b277d5bf3d1..7ee486627420 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -8201,7 +8201,7 @@ B:	https://gitlab.freedesktop.org/drm/nova/-/issues
>   C:	irc://irc.oftc.net/nouveau
>   T:	git https://gitlab.freedesktop.org/drm/rust/kernel.git drm-rust-next
>   F:	Documentation/gpu/nova/
> -F:	drivers/gpu/drm/nova/
> +F:	drivers/gpu/drm/nova-drm/
>   F:	include/uapi/drm/nova_drm.h
>   
>   DRM DRIVER FOR OLIMEX LCD-OLINUXINO PANELS
> @@ -8484,7 +8484,7 @@ X:	drivers/gpu/drm/i915/
>   X:	drivers/gpu/drm/kmb/
>   X:	drivers/gpu/drm/mediatek/
>   X:	drivers/gpu/drm/msm/
> -X:	drivers/gpu/drm/nova/
> +X:	drivers/gpu/drm/nova-drm/
>   X:	drivers/gpu/drm/radeon/
>   X:	drivers/gpu/drm/tegra/
>   X:	drivers/gpu/drm/tyr/
> @@ -8496,7 +8496,7 @@ M:	Alice Ryhl <aliceryhl@google.com>
>   S:	Supported
>   W:	https://drm.pages.freedesktop.org/maintainer-tools/drm-rust.html
>   T:	git https://gitlab.freedesktop.org/drm/rust/kernel.git
> -F:	drivers/gpu/drm/nova/
> +F:	drivers/gpu/drm/nova-drm/
>   F:	drivers/gpu/drm/tyr/
>   F:	drivers/gpu/nova-core/
>   F:	rust/kernel/drm/
> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
> index d3d52310c9cc..958fa142b6cd 100644
> --- a/drivers/gpu/drm/Kconfig
> +++ b/drivers/gpu/drm/Kconfig
> @@ -308,7 +308,7 @@ source "drivers/gpu/drm/mgag200/Kconfig"
>   source "drivers/gpu/drm/msm/Kconfig"
>   source "drivers/gpu/drm/mxsfb/Kconfig"
>   source "drivers/gpu/drm/nouveau/Kconfig"
> -source "drivers/gpu/drm/nova/Kconfig"
> +source "drivers/gpu/drm/nova-drm/Kconfig"
>   source "drivers/gpu/drm/omapdrm/Kconfig"
>   source "drivers/gpu/drm/panel/Kconfig"
>   source "drivers/gpu/drm/panfrost/Kconfig"
> diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
> index ec2c5ff82382..acdec9e30a27 100644
> --- a/drivers/gpu/drm/Makefile
> +++ b/drivers/gpu/drm/Makefile
> @@ -186,7 +186,7 @@ obj-$(CONFIG_DRM_VMWGFX)+= vmwgfx/
>   obj-$(CONFIG_DRM_VGEM)	+= vgem/
>   obj-$(CONFIG_DRM_VKMS)	+= vkms/
>   obj-$(CONFIG_DRM_NOUVEAU) +=nouveau/
> -obj-$(CONFIG_DRM_NOVA) += nova/
> +obj-$(CONFIG_DRM_NOVA) += nova-drm/
>   obj-$(CONFIG_DRM_EXYNOS) +=exynos/
>   obj-$(CONFIG_DRM_ROCKCHIP) +=rockchip/
>   obj-$(CONFIG_DRM_GMA500) += gma500/
> diff --git a/drivers/gpu/drm/nova/Kconfig b/drivers/gpu/drm/nova-drm/Kconfig
> similarity index 84%
> rename from drivers/gpu/drm/nova/Kconfig
> rename to drivers/gpu/drm/nova-drm/Kconfig
> index 3e637ad7b5ba..48a731bff9b1 100644
> --- a/drivers/gpu/drm/nova/Kconfig
> +++ b/drivers/gpu/drm/nova-drm/Kconfig
> @@ -13,4 +13,4 @@ config DRM_NOVA
>   
>   	  This driver is work in progress and may not be functional.
>   
> -	  If M is selected, the module will be called nova.
> +	  If M is selected, the module will be called nova-drm.
> diff --git a/drivers/gpu/drm/nova-drm/Makefile b/drivers/gpu/drm/nova-drm/Makefile
> new file mode 100644
> index 000000000000..bd3a5e703742
> --- /dev/null
> +++ b/drivers/gpu/drm/nova-drm/Makefile
> @@ -0,0 +1,3 @@
> +# SPDX-License-Identifier: GPL-2.0
> +
> +obj-$(CONFIG_DRM_NOVA) += nova_drm.o
> diff --git a/drivers/gpu/drm/nova/driver.rs b/drivers/gpu/drm/nova-drm/driver.rs
> similarity index 100%
> rename from drivers/gpu/drm/nova/driver.rs
> rename to drivers/gpu/drm/nova-drm/driver.rs
> diff --git a/drivers/gpu/drm/nova/file.rs b/drivers/gpu/drm/nova-drm/file.rs
> similarity index 100%
> rename from drivers/gpu/drm/nova/file.rs
> rename to drivers/gpu/drm/nova-drm/file.rs
> diff --git a/drivers/gpu/drm/nova/gem.rs b/drivers/gpu/drm/nova-drm/gem.rs
> similarity index 100%
> rename from drivers/gpu/drm/nova/gem.rs
> rename to drivers/gpu/drm/nova-drm/gem.rs
> diff --git a/drivers/gpu/drm/nova/nova.rs b/drivers/gpu/drm/nova-drm/nova_drm.rs
> similarity index 100%
> rename from drivers/gpu/drm/nova/nova.rs
> rename to drivers/gpu/drm/nova-drm/nova_drm.rs
> diff --git a/drivers/gpu/drm/nova/Makefile b/drivers/gpu/drm/nova/Makefile
> deleted file mode 100644
> index 42019bff3173..000000000000
> --- a/drivers/gpu/drm/nova/Makefile
> +++ /dev/null
> @@ -1,3 +0,0 @@
> -# SPDX-License-Identifier: GPL-2.0
> -
> -obj-$(CONFIG_DRM_NOVA) += nova.o

thanks,
-- 
John Hubbard
[PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Cheng-Yang Chou 2 weeks, 6 days ago 
Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
to nova_drm.rs (matching the object name nova_drm.o) to follow the
kebab-case naming convention already used by the auxiliary device name.

Update MAINTAINERS, Kconfig, and Makefile references accordingly.
Update the Kconfig help text to reflect the new module name nova-drm.

Link: https://github.com/Rust-for-Linux/linux/issues/1228
Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
Reviewed-by: John Hubbard <jhubbard@nvidia.com>
---
Changes in v3:
- Rename subject line (John Hubbard)

 MAINTAINERS                                            | 6 +++---
 drivers/gpu/drm/Kconfig                                | 2 +-
 drivers/gpu/drm/Makefile                               | 2 +-
 drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
 drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
 drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
 drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
 drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
 drivers/gpu/drm/{nova/nova.rs => nova-drm/nova_drm.rs} | 0
 drivers/gpu/drm/nova/Makefile                          | 3 ---
 10 files changed, 9 insertions(+), 9 deletions(-)
 rename drivers/gpu/drm/{nova => nova-drm}/Kconfig (84%)
 create mode 100644 drivers/gpu/drm/nova-drm/Makefile
 rename drivers/gpu/drm/{nova => nova-drm}/driver.rs (100%)
 rename drivers/gpu/drm/{nova => nova-drm}/file.rs (100%)
 rename drivers/gpu/drm/{nova => nova-drm}/gem.rs (100%)
 rename drivers/gpu/drm/{nova/nova.rs => nova-drm/nova_drm.rs} (100%)
 delete mode 100644 drivers/gpu/drm/nova/Makefile

diff --git a/MAINTAINERS b/MAINTAINERS
index 7b277d5bf3d1..7ee486627420 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8201,7 +8201,7 @@ B:	https://gitlab.freedesktop.org/drm/nova/-/issues
 C:	irc://irc.oftc.net/nouveau
 T:	git https://gitlab.freedesktop.org/drm/rust/kernel.git drm-rust-next
 F:	Documentation/gpu/nova/
-F:	drivers/gpu/drm/nova/
+F:	drivers/gpu/drm/nova-drm/
 F:	include/uapi/drm/nova_drm.h
 
 DRM DRIVER FOR OLIMEX LCD-OLINUXINO PANELS
@@ -8484,7 +8484,7 @@ X:	drivers/gpu/drm/i915/
 X:	drivers/gpu/drm/kmb/
 X:	drivers/gpu/drm/mediatek/
 X:	drivers/gpu/drm/msm/
-X:	drivers/gpu/drm/nova/
+X:	drivers/gpu/drm/nova-drm/
 X:	drivers/gpu/drm/radeon/
 X:	drivers/gpu/drm/tegra/
 X:	drivers/gpu/drm/tyr/
@@ -8496,7 +8496,7 @@ M:	Alice Ryhl <aliceryhl@google.com>
 S:	Supported
 W:	https://drm.pages.freedesktop.org/maintainer-tools/drm-rust.html
 T:	git https://gitlab.freedesktop.org/drm/rust/kernel.git
-F:	drivers/gpu/drm/nova/
+F:	drivers/gpu/drm/nova-drm/
 F:	drivers/gpu/drm/tyr/
 F:	drivers/gpu/nova-core/
 F:	rust/kernel/drm/
diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
index d3d52310c9cc..958fa142b6cd 100644
--- a/drivers/gpu/drm/Kconfig
+++ b/drivers/gpu/drm/Kconfig
@@ -308,7 +308,7 @@ source "drivers/gpu/drm/mgag200/Kconfig"
 source "drivers/gpu/drm/msm/Kconfig"
 source "drivers/gpu/drm/mxsfb/Kconfig"
 source "drivers/gpu/drm/nouveau/Kconfig"
-source "drivers/gpu/drm/nova/Kconfig"
+source "drivers/gpu/drm/nova-drm/Kconfig"
 source "drivers/gpu/drm/omapdrm/Kconfig"
 source "drivers/gpu/drm/panel/Kconfig"
 source "drivers/gpu/drm/panfrost/Kconfig"
diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
index ec2c5ff82382..acdec9e30a27 100644
--- a/drivers/gpu/drm/Makefile
+++ b/drivers/gpu/drm/Makefile
@@ -186,7 +186,7 @@ obj-$(CONFIG_DRM_VMWGFX)+= vmwgfx/
 obj-$(CONFIG_DRM_VGEM)	+= vgem/
 obj-$(CONFIG_DRM_VKMS)	+= vkms/
 obj-$(CONFIG_DRM_NOUVEAU) +=nouveau/
-obj-$(CONFIG_DRM_NOVA) += nova/
+obj-$(CONFIG_DRM_NOVA) += nova-drm/
 obj-$(CONFIG_DRM_EXYNOS) +=exynos/
 obj-$(CONFIG_DRM_ROCKCHIP) +=rockchip/
 obj-$(CONFIG_DRM_GMA500) += gma500/
diff --git a/drivers/gpu/drm/nova/Kconfig b/drivers/gpu/drm/nova-drm/Kconfig
similarity index 84%
rename from drivers/gpu/drm/nova/Kconfig
rename to drivers/gpu/drm/nova-drm/Kconfig
index 3e637ad7b5ba..48a731bff9b1 100644
--- a/drivers/gpu/drm/nova/Kconfig
+++ b/drivers/gpu/drm/nova-drm/Kconfig
@@ -13,4 +13,4 @@ config DRM_NOVA
 
 	  This driver is work in progress and may not be functional.
 
-	  If M is selected, the module will be called nova.
+	  If M is selected, the module will be called nova-drm.
diff --git a/drivers/gpu/drm/nova-drm/Makefile b/drivers/gpu/drm/nova-drm/Makefile
new file mode 100644
index 000000000000..bd3a5e703742
--- /dev/null
+++ b/drivers/gpu/drm/nova-drm/Makefile
@@ -0,0 +1,3 @@
+# SPDX-License-Identifier: GPL-2.0
+
+obj-$(CONFIG_DRM_NOVA) += nova_drm.o
diff --git a/drivers/gpu/drm/nova/driver.rs b/drivers/gpu/drm/nova-drm/driver.rs
similarity index 100%
rename from drivers/gpu/drm/nova/driver.rs
rename to drivers/gpu/drm/nova-drm/driver.rs
diff --git a/drivers/gpu/drm/nova/file.rs b/drivers/gpu/drm/nova-drm/file.rs
similarity index 100%
rename from drivers/gpu/drm/nova/file.rs
rename to drivers/gpu/drm/nova-drm/file.rs
diff --git a/drivers/gpu/drm/nova/gem.rs b/drivers/gpu/drm/nova-drm/gem.rs
similarity index 100%
rename from drivers/gpu/drm/nova/gem.rs
rename to drivers/gpu/drm/nova-drm/gem.rs
diff --git a/drivers/gpu/drm/nova/nova.rs b/drivers/gpu/drm/nova-drm/nova_drm.rs
similarity index 100%
rename from drivers/gpu/drm/nova/nova.rs
rename to drivers/gpu/drm/nova-drm/nova_drm.rs
diff --git a/drivers/gpu/drm/nova/Makefile b/drivers/gpu/drm/nova/Makefile
deleted file mode 100644
index 42019bff3173..000000000000
--- a/drivers/gpu/drm/nova/Makefile
+++ /dev/null
@@ -1,3 +0,0 @@
-# SPDX-License-Identifier: GPL-2.0
-
-obj-$(CONFIG_DRM_NOVA) += nova.o
-- 
2.48.1
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Alexandre Courbot 2 weeks, 6 days ago 
On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
> to nova_drm.rs (matching the object name nova_drm.o) to follow the
> kebab-case naming convention already used by the auxiliary device name.
>
> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
> Update the Kconfig help text to reflect the new module name nova-drm.
>
> Link: https://github.com/Rust-for-Linux/linux/issues/1228
> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
> Reviewed-by: John Hubbard <jhubbard@nvidia.com>
> ---
> Changes in v3:
> - Rename subject line (John Hubbard)
>
>  MAINTAINERS                                            | 6 +++---
>  drivers/gpu/drm/Kconfig                                | 2 +-
>  drivers/gpu/drm/Makefile                               | 2 +-
>  drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>  drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>  drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>  drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>  drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0

That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
Even `tegra-drm` is in `drm/tegra`.
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Danilo Krummrich 2 weeks, 6 days ago 
On Tue Mar 17, 2026 at 2:20 PM CET, Alexandre Courbot wrote:
> On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
>> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
>> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
>> to nova_drm.rs (matching the object name nova_drm.o) to follow the
>> kebab-case naming convention already used by the auxiliary device name.
>>
>> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
>> Update the Kconfig help text to reflect the new module name nova-drm.
>>
>> Link: https://github.com/Rust-for-Linux/linux/issues/1228
>> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
>> Reviewed-by: John Hubbard <jhubbard@nvidia.com>
>> ---
>> Changes in v3:
>> - Rename subject line (John Hubbard)
>>
>>  MAINTAINERS                                            | 6 +++---
>>  drivers/gpu/drm/Kconfig                                | 2 +-
>>  drivers/gpu/drm/Makefile                               | 2 +-
>>  drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>>  drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>>  drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>>  drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>>  drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
>
> That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
> Even `tegra-drm` is in `drm/tegra`.

Well, that's why I named it "nova" in the first place, but I don't want
KBUILD_MODNAME to diverge from the directory name and I also don't want the
driver name to diverge from KBUILD_MODNAME.

So, if we rename the driver name to nova-drm, then the other parts should be
renamed as well.
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Alexandre Courbot 2 weeks, 6 days ago 
On Tue Mar 17, 2026 at 10:53 PM JST, Danilo Krummrich wrote:
> On Tue Mar 17, 2026 at 2:20 PM CET, Alexandre Courbot wrote:
>> On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
>>> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
>>> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
>>> to nova_drm.rs (matching the object name nova_drm.o) to follow the
>>> kebab-case naming convention already used by the auxiliary device name.
>>>
>>> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
>>> Update the Kconfig help text to reflect the new module name nova-drm.
>>>
>>> Link: https://github.com/Rust-for-Linux/linux/issues/1228
>>> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
>>> Reviewed-by: John Hubbard <jhubbard@nvidia.com>
>>> ---
>>> Changes in v3:
>>> - Rename subject line (John Hubbard)
>>>
>>>  MAINTAINERS                                            | 6 +++---
>>>  drivers/gpu/drm/Kconfig                                | 2 +-
>>>  drivers/gpu/drm/Makefile                               | 2 +-
>>>  drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>>>  drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>>>  drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>>>  drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>>>  drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
>>
>> That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
>> Even `tegra-drm` is in `drm/tegra`.
>
> Well, that's why I named it "nova" in the first place, but I don't want
> KBUILD_MODNAME to diverge from the directory name and I also don't want the
> driver name to diverge from KBUILD_MODNAME.

What is the rationale for that? I like consistency too, but also think
it is not particularly difficult to figure out that "nova-drm" resides
in "drm/nova".
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Danilo Krummrich 2 weeks, 6 days ago 
On Tue Mar 17, 2026 at 4:22 PM CET, Alexandre Courbot wrote:
> On Tue Mar 17, 2026 at 10:53 PM JST, Danilo Krummrich wrote:
>> On Tue Mar 17, 2026 at 2:20 PM CET, Alexandre Courbot wrote:
>>> On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
>>>> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
>>>> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
>>>> to nova_drm.rs (matching the object name nova_drm.o) to follow the
>>>> kebab-case naming convention already used by the auxiliary device name.
>>>>
>>>> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
>>>> Update the Kconfig help text to reflect the new module name nova-drm.
>>>>
>>>> Link: https://github.com/Rust-for-Linux/linux/issues/1228
>>>> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
>>>> Reviewed-by: John Hubbard <jhubbard@nvidia.com>
>>>> ---
>>>> Changes in v3:
>>>> - Rename subject line (John Hubbard)
>>>>
>>>>  MAINTAINERS                                            | 6 +++---
>>>>  drivers/gpu/drm/Kconfig                                | 2 +-
>>>>  drivers/gpu/drm/Makefile                               | 2 +-
>>>>  drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>>>>  drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>>>>  drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>>>>  drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>>>>  drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
>>>
>>> That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
>>> Even `tegra-drm` is in `drm/tegra`.
>>
>> Well, that's why I named it "nova" in the first place, but I don't want
>> KBUILD_MODNAME to diverge from the directory name and I also don't want the
>> driver name to diverge from KBUILD_MODNAME.
>
> What is the rationale for that? I like consistency too, but also think
> it is not particularly difficult to figure out that "nova-drm" resides
> in "drm/nova".

It is consistency. But why are we discussing this, what is the concern being
consistent here?
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Alexandre Courbot 2 weeks, 6 days ago 
On Wed Mar 18, 2026 at 12:34 AM JST, Danilo Krummrich wrote:
> On Tue Mar 17, 2026 at 4:22 PM CET, Alexandre Courbot wrote:
>> On Tue Mar 17, 2026 at 10:53 PM JST, Danilo Krummrich wrote:
>>> On Tue Mar 17, 2026 at 2:20 PM CET, Alexandre Courbot wrote:
>>>> On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
>>>>> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
>>>>> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
>>>>> to nova_drm.rs (matching the object name nova_drm.o) to follow the
>>>>> kebab-case naming convention already used by the auxiliary device name.
>>>>>
>>>>> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
>>>>> Update the Kconfig help text to reflect the new module name nova-drm.
>>>>>
>>>>> Link: https://github.com/Rust-for-Linux/linux/issues/1228
>>>>> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
>>>>> Reviewed-by: John Hubbard <jhubbard@nvidia.com>
>>>>> ---
>>>>> Changes in v3:
>>>>> - Rename subject line (John Hubbard)
>>>>>
>>>>>  MAINTAINERS                                            | 6 +++---
>>>>>  drivers/gpu/drm/Kconfig                                | 2 +-
>>>>>  drivers/gpu/drm/Makefile                               | 2 +-
>>>>>  drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>>>>>  drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>>>>>  drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>>>>>  drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>>>>>  drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
>>>>
>>>> That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
>>>> Even `tegra-drm` is in `drm/tegra`.
>>>
>>> Well, that's why I named it "nova" in the first place, but I don't want
>>> KBUILD_MODNAME to diverge from the directory name and I also don't want the
>>> driver name to diverge from KBUILD_MODNAME.
>>
>> What is the rationale for that? I like consistency too, but also think
>> it is not particularly difficult to figure out that "nova-drm" resides
>> in "drm/nova".
>
> It is consistency. But why are we discussing this, what is the concern being
> consistent here?

I just find the repetition of "drm" unnecessary. That's not a big deal
though and I can definitely live with it.
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by John Hubbard 2 weeks, 5 days ago 
On 3/17/26 5:56 PM, Alexandre Courbot wrote:
> On Wed Mar 18, 2026 at 12:34 AM JST, Danilo Krummrich wrote:
>> On Tue Mar 17, 2026 at 4:22 PM CET, Alexandre Courbot wrote:
>>> On Tue Mar 17, 2026 at 10:53 PM JST, Danilo Krummrich wrote:
>>>> On Tue Mar 17, 2026 at 2:20 PM CET, Alexandre Courbot wrote:
>>>>> On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
....
>>>>> That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
>>>>> Even `tegra-drm` is in `drm/tegra`.
>>>>
>>>> Well, that's why I named it "nova" in the first place, but I don't want
>>>> KBUILD_MODNAME to diverge from the directory name and I also don't want the
>>>> driver name to diverge from KBUILD_MODNAME.
>>>
>>> What is the rationale for that? I like consistency too, but also think
>>> it is not particularly difficult to figure out that "nova-drm" resides
>>> in "drm/nova".
>>
>> It is consistency. But why are we discussing this, what is the concern being
>> consistent here?
> 
> I just find the repetition of "drm" unnecessary. That's not a big deal
> though and I can definitely live with it.

As I mentioned in an earlier version of this patchset, things such as
lsmod(1) do not show the source code directory of one's kernel module.

 $ lsmod|grep nova
nova                   16384  0
nova_core             253952  0

And of course, lsmod(1) is not the only case where this comes up: few
people know the source directory paths! Nor should they. But this
means that in many situations, we are left with an initial report
that only includes the module's basename.

We would like some hint about what "nova" means, especially
given the presence of nova_core.

So it turns out that a little redundancy, and even saying something
more than once, is helpful in this particular case. :)


thanks,
-- 
John Hubbard
Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm
Posted by Martin Roukala 2 weeks, 6 days ago 
On 3/17/26 15:20, Alexandre Courbot wrote:
> On Tue Mar 17, 2026 at 6:23 PM JST, Cheng-Yang Chou wrote:
>> Rename the nova-drm driver directory from drivers/gpu/drm/nova/ to
>> drivers/gpu/drm/nova-drm/ and the top-level source file from nova.rs
>> to nova_drm.rs (matching the object name nova_drm.o) to follow the
>> kebab-case naming convention already used by the auxiliary device name.
>>
>> Update MAINTAINERS, Kconfig, and Makefile references accordingly.
>> Update the Kconfig help text to reflect the new module name nova-drm.
>>
>> Link: https://github.com/Rust-for-Linux/linux/issues/1228
>> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
>> Reviewed-by: John Hubbard <jhubbard@nvidia.com>
>> ---
>> Changes in v3:
>> - Rename subject line (John Hubbard)
>>
>>   MAINTAINERS                                            | 6 +++---
>>   drivers/gpu/drm/Kconfig                                | 2 +-
>>   drivers/gpu/drm/Makefile                               | 2 +-
>>   drivers/gpu/drm/{nova => nova-drm}/Kconfig             | 2 +-
>>   drivers/gpu/drm/nova-drm/Makefile                      | 3 +++
>>   drivers/gpu/drm/{nova => nova-drm}/driver.rs           | 0
>>   drivers/gpu/drm/{nova => nova-drm}/file.rs             | 0
>>   drivers/gpu/drm/{nova => nova-drm}/gem.rs              | 0
> 
> That `drm/nova-drm` reads a bit awkwardly to me and feels unnecessary.
> Even `tegra-drm` is in `drm/tegra`.

Same opinion here.

Cheers,
Martin (an old nouveau dev)

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.