Re: [PATCH v14 1/2] drm: add kms driver for loongson display controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 2023/5/21 20:21, WANG Xuerui wrote:
--- /dev/null
+++ b/drivers/gpu/drm/loongson/Kconfig
@@ -0,0 +1,17 @@
+# SPDX-License-Identifier: GPL-2.0
+
+config DRM_LOONGSON
+    tristate "DRM support for Loongson Graphics"
+    depends on DRM && PCI && MMU
+    select DRM_KMS_HELPER
+    select DRM_TTM
+    select I2C
+    select I2C_ALGOBIT
+    help
+      This is a DRM driver for Loongson Graphics, it may including

Drop "it may"; "including" should be enough.

'it may' is more *precise* here, because currently we don't ship with the support for loongson 2K series SoC.

I'm try to be precise as far as I can, we avoid made this driver too large by ignore loongson 2K series SoC temporary.

+      LS7A2000, LS7A1000, LS2K2000 and LS2K1000 etc. Loongson LS7A
+      series are bridge chipset, while Loongson LS2K series are SoC.
+
+      If "M" is selected, the module will be called loongson.

Just "loongson"?

Yes,  when compile this driver as module,  loongson.ko will be generated.

 drm radeon is also doing so, See drm/radeon/Kconfig.

I know it's like this for ages (at least dating back to the MIPS days) but you really don't want to imply Loongson is mainly a GPU company. Something like "loongson_drm" or "lsdc" or "gsgpu" could be better.

No, these name may have backward compatibility problems.

Downstream driver already taken those name.

userspace driver need to differentiate them who is who.





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux