[PATCH 2/3] drm/file: use explicit values for enum drm_minor_type

Simon Ser contact at emersion.fr
Fri Jul 14 10:46:04 UTC 2023


This makes it clearer that the values cannot be changed because
they are ABI.

Signed-off-by: Simon Ser <contact at emersion.fr>
Cc: Christian König <christian.koenig at amd.com>
Cc: James Zhu <James.Zhu at amd.com>
Cc: Marek Olšák <marek.olsak at amd.com>
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
---
 include/drm/drm_file.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/drm/drm_file.h b/include/drm/drm_file.h
index 966912053cb0..010239392adf 100644
--- a/include/drm/drm_file.h
+++ b/include/drm/drm_file.h
@@ -50,16 +50,16 @@ struct file;
  * header include loops we need it here for now.
  */
 
-/* Note that the order of this enum is ABI (it determines
+/* Note that the values of this enum are ABI (it determines
  * /dev/dri/renderD* numbers).
  *
  * Setting DRM_MINOR_ACCEL to 32 gives enough space for more drm minors to
  * be implemented before we hit any future
  */
 enum drm_minor_type {
-	DRM_MINOR_PRIMARY,
-	DRM_MINOR_CONTROL,
-	DRM_MINOR_RENDER,
+	DRM_MINOR_PRIMARY = 0,
+	DRM_MINOR_CONTROL = 1,
+	DRM_MINOR_RENDER = 2,
 	DRM_MINOR_ACCEL = 32,
 };
 
-- 
2.41.0




More information about the dri-devel mailing list