From patchwork Fri Jul 14 10:46:04 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Simon Ser X-Patchwork-Id: 13313468 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D9AB1EB64DA for ; Fri, 14 Jul 2023 10:46:11 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C101D10E08E; Fri, 14 Jul 2023 10:46:10 +0000 (UTC) Received: from mail-4323.proton.ch (mail-4323.proton.ch [185.70.43.23]) by gabe.freedesktop.org (Postfix) with ESMTPS id 76C9D10E08E for ; Fri, 14 Jul 2023 10:46:09 +0000 (UTC) Date: Fri, 14 Jul 2023 10:46:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=emersion.fr; s=protonmail; t=1689331566; x=1689590766; bh=DY6x2Fh4B/JVNyCv47E1PVT/qIbnNcx2Fez2vh2dzN0=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: Feedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID: Message-ID:BIMI-Selector; b=dyBGoTrp7bR/f5YPyCKPsvuPs21/a/w7KSasRgnDTHwhDkZtcA63iXXyo3tOJtoPv BYnjicyHi2HjcAVIuZi3ue4q1u5mVUMAKhmJ9/NydvwtE8eEFcCJgnPFqCU4X2ppgx ArXnj+5Q85PBB8RFrR1T9cB6Y0jlsLk+AiEKSyrCmPPlCgolc8uOS2xImYZP5fprIJ UykByOLofJ5DHJoH0kWIyznBfOsEdnU7pD+27gtrOSzEMDWTxqzNg67ZEgs7jFhNS8 MqGqyEPaMwpDdHExMoPZb6zWyGm+SrYFHNphMZ1c0t+H0fqwd9wOY7EP9vjRTU1H1Q U18OOgZrg9HXQ== To: dri-devel@lists.freedesktop.org From: Simon Ser Subject: [PATCH 2/3] drm/file: use explicit values for enum drm_minor_type Message-ID: <20230714104557.518457-2-contact@emersion.fr> In-Reply-To: <20230714104557.518457-1-contact@emersion.fr> References: <20230714104557.518457-1-contact@emersion.fr> Feedback-ID: 1358184:user:proton MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Daniel Vetter , =?utf-8?b?TWFyZWsgT2zFocOhaw==?= , James Zhu , =?utf-8?q?Christian_K=C3=B6nig?= Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" This makes it clearer that the values cannot be changed because they are ABI. Signed-off-by: Simon Ser Cc: Christian König Cc: James Zhu Cc: Marek Olšák Cc: Daniel Vetter --- 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, };