drm/file: use explicit values for enum drm_minor_type
authorSimon Ser <contact@emersion.fr>
Fri, 14 Jul 2023 10:46:04 +0000 (10:46 +0000)
committerSimon Ser <contact@emersion.fr>
Mon, 17 Jul 2023 08:40:41 +0000 (10:40 +0200)
This makes it clearer that the values cannot be changed because
they are ABI.

Signed-off-by: Simon Ser <contact@emersion.fr>
Reviewed-by:James Zhu <James.Zhu@amd.com>
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Christian König <christian.koenig@amd.com>
Cc: Marek Olšák <marek.olsak@amd.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20230714104557.518457-2-contact@emersion.fr
include/drm/drm_file.h

index 9669120..0102393 100644 (file)
@@ -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,
 };