Re: [PATCH v3 2/2] gpu: nova-drm: rename nova to nova-drm

[Alexandre Courbot]

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@xxxxxxxxx>
>>> Reviewed-by: John Hubbard <jhubbard@xxxxxxxxxx>
>>> ---
>>> 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".