From nobody Thu Sep 18 10:20:47 2025 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A9027C63705 for ; Wed, 7 Dec 2022 13:25:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230075AbiLGNZL (ORCPT ); Wed, 7 Dec 2022 08:25:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56902 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229593AbiLGNYd (ORCPT ); Wed, 7 Dec 2022 08:24:33 -0500 Received: from fanzine2.igalia.com (fanzine2.igalia.com [213.97.179.56]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A36253056A for ; Wed, 7 Dec 2022 05:24:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=Content-Transfer-Encoding:Content-Type:MIME-Version:References: In-Reply-To:Message-Id:Date:Subject:Cc:To:From:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=YOKduRxqOQ+ggRn/s0wpQfzORzZWr0CzU1FGkX5EVD0=; b=DPxXbUbrYZV8/6yujrWCdEem8+ /eFYPbecOcm3T5FzjKxYsA3M6J/6KNF7h2pqzC4+Gq/xOT28YXb+UCledR5MWWeMBfg5EqrZvqkZ6 k8JLwbPn7ctypP0xVJfvyp6vyKbxPEe1XLyRZac4Ueqf4iEc4Re35pFPjI1F0j8Ilz6S/NlpEeEr4 2t7oBJyXuZFJ4WKukPxsbsWELaIFHAh8fDFwmJcK+t6etuz8ITvKl6cu056oZwFXXVI7H+zbkIo4q q6VGIwdmqT7yVxf/xQuoFXLr/SpyddbxVxPPl4hdyXIkr+7gh0P2GwvhoOxeb/XTRr+cItlDCDFT3 X6929ZAw==; Received: from [177.34.169.227] (helo=bowie..) by fanzine2.igalia.com with esmtpsa (Cipher TLS1.3:ECDHE_X25519__RSA_PSS_RSAE_SHA256__AES_256_GCM:256) (Exim) id 1p2uPO-00Gaon-Md; Wed, 07 Dec 2022 14:24:23 +0100 From: =?UTF-8?q?Ma=C3=ADra=20Canal?= To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Oded Gabbay , Jani Nikula Cc: Melissa Wen , =?UTF-8?q?Andr=C3=A9=20Almeida?= , Emma Anholt , Rodrigo Siqueira , Wambui Karuga , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, =?UTF-8?q?Ma=C3=ADra=20Canal?= Subject: [PATCH v3 6/6] drm/todo: update the debugfs clean up task Date: Wed, 7 Dec 2022 10:23:25 -0300 Message-Id: <20221207132325.140393-7-mcanal@igalia.com> X-Mailer: git-send-email 2.38.1 In-Reply-To: <20221207132325.140393-1-mcanal@igalia.com> References: <20221207132325.140393-1-mcanal@igalia.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The structs drm_debugfs_info and drm_debugfs_entry introduced a new debugfs structure to DRM, centered on drm_device instead of drm_minor. Therefore, remove the tasks related to create a new device-centered debugfs structure and add a new task to replace the use of drm_debugfs_create_files() for the use of drm_debugfs_add_file() and drm_debugfs_add_files(). Signed-off-by: Ma=C3=ADra Canal --- Documentation/gpu/todo.rst | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index b2c6aaf1edf2..f64abf69f341 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@ -508,17 +508,14 @@ Clean up the debugfs support =20 There's a bunch of issues with it: =20 -- The drm_info_list ->show() function doesn't even bother to cast to the d= rm - structure for you. This is lazy. +- Convert drivers to support the drm_debugfs_add_files() function instead = of + the drm_debugfs_create_files() function. =20 - We probably want to have some support for debugfs files on crtc/connecto= rs and maybe other kms objects directly in core. There's even drm_print support= in the funcs for these objects to dump kms state, so it's all there. And th= en the ->show() functions should obviously give you a pointer to the right obje= ct. =20 -- The drm_info_list stuff is centered on drm_minor instead of drm_device. = For - anything we want to print drm_device (or maybe drm_file) is the right th= ing. - - The drm_driver->debugfs_init hooks we have is just an artifact of the old midlayered load sequence. DRM debugfs should work more like sysfs, where= you can create properties/files for an object anytime you want, and the core @@ -527,8 +524,6 @@ There's a bunch of issues with it: this (together with the drm_minor->drm_device move) would allow us to re= move debugfs_init. =20 -Previous RFC that hasn't landed yet: https://lore.kernel.org/dri-devel/202= 00513114130.28641-2-wambui.karugax@gmail.com/ - Contact: Daniel Vetter =20 Level: Intermediate --=20 2.38.1