From patchwork Fri Jun 9 13:44:27 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Geert Uytterhoeven X-Patchwork-Id: 105645 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp962022vqr; Fri, 9 Jun 2023 06:51:34 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7206Wq5StubvRD+10JeLCB8hlCqQqCNIRjMGT8Wvu56+R12zJMXiILEIH71Eoq9IE85Z2q X-Received: by 2002:a92:cbd2:0:b0:338:18f1:db72 with SMTP id s18-20020a92cbd2000000b0033818f1db72mr1588404ilq.29.1686318694180; Fri, 09 Jun 2023 06:51:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686318694; cv=none; d=google.com; s=arc-20160816; b=aSsQngQ7s/aHpaS0fQSG3OPWk3UYmLNU6Xh37sHTVJM59ep+MEAn91dmVEMtryNNwx XOvMzyTkyOSU5ePJnIEo+LAjr+Y8ZL3uxviO4AqMXrd5ml91prMRtYyQE9hnpOnUNNFm +OvesGw/DaTGwhSS01+yKeB+mWtv3zysVpmtuBpu531HLStYAHwWbJ/Qrj/Jm2NkT7Rw 9oFGO9CSCrMdU+dXhNLlE8GbsxACQ87RTwEiuKH78XTcoeNDd6Le+ZaeIvbrHkYRFIgv j6XIRUdiVoqDvqv5Ye4NFLSoleWsua14tSZ91O8gchtUg5CN5YxilnGW8VtTgrHk9BIX w7dg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=qLsSk7kh5zHgSeuoiARQKdV3jmuU4exT+q/UC8Qr2wU=; b=wPliYpo4UhpikaAK6IKgqiwBZQF/tayL4WAiMk4WGRs/EdSTGKc0LbFJFTIF1O1m8l IHNfhpi+PlfwsI6XYXcSHjnalMI/bxThWnFq1YbNQXsBdU5deyqt0FmrymaWMSG1i7+s xKES/NtfspYkLuTx5lHdPVqbQkMpk5LJrmMRwEB8DlGPkCI99PnJDzwtfr8O4tpAofNQ XYd0Y7CC6Q0x5DNjLx+W8SGX66zApUX4HqXQNblFBVX8Wid6jsKXpKQekVEuSN93dj8G hn0gK+hMvxB8qYYEGP/lPe1bTvZ8WsLCgIqsUGyEdPWu4yZOCEl1Ii2jJQPxubOTD1B+ Abmg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b5-20020a63eb45000000b0053b8c98d14bsi2622380pgk.859.2023.06.09.06.51.19; Fri, 09 Jun 2023 06:51:34 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241336AbjFINo6 (ORCPT + 99 others); Fri, 9 Jun 2023 09:44:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54772 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239848AbjFINoq (ORCPT ); Fri, 9 Jun 2023 09:44:46 -0400 Received: from michel.telenet-ops.be (michel.telenet-ops.be [IPv6:2a02:1800:110:4::f00:18]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C5BB735A9 for ; Fri, 9 Jun 2023 06:44:43 -0700 (PDT) Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed30:a1e2:1b31:cba3:390d]) by michel.telenet-ops.be with bizsmtp id 71kZ2A00512zQ4r061kZpM; Fri, 09 Jun 2023 15:44:42 +0200 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.95) (envelope-from ) id 1q7cPS-008La3-2T; Fri, 09 Jun 2023 15:44:33 +0200 Received: from geert by rox.of.borg with local (Exim 4.95) (envelope-from ) id 1q7cPp-00GCTQ-3L; Fri, 09 Jun 2023 15:44:33 +0200 From: Geert Uytterhoeven To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Jyri Sarha , Tomi Valkeinen , Laurent Pinchart , Bagas Sanjaya Cc: dri-devel@lists.freedesktop.org, linux-doc@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven , Javier Martinez Canillas Subject: [PATCH v2 1/4] drm/todo: Add atomic modesetting references Date: Fri, 9 Jun 2023 15:44:27 +0200 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1768233311217151627?= X-GMAIL-MSGID: =?utf-8?q?1768233311217151627?= The section about converting existing KMS drivers to atomic modesetting mentions the existence of a conversion guide, but does not reference it. While the guide is old and rusty, it still contains useful information, so add a link to it. Also link to the LWN.net articles that give an overview about the atomic mode setting design. While at it, remove the reference to unconverted virtual HW drivers, as they've been converted. Signed-off-by: Geert Uytterhoeven Reviewed-by: Javier Martinez Canillas Reviewed-by: Laurent Pinchart --- v2: - Add Reviewed-by, - Drop double space after full stop, - Use footnotes for references, - Remore reference to unconverted virtual HW drivers. --- Documentation/gpu/todo.rst | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 68bdafa0284f55f6..6c328613c049fc1d 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@ -49,14 +49,18 @@ converted over. Modern compositors like Wayland or Surfaceflinger on Android really want an atomic modeset interface, so this is all about the bright future. -There is a conversion guide for atomic and all you need is a GPU for a -non-converted driver (again virtual HW drivers for KVM are still all -suitable). +There is a conversion guide for atomic [1]_ and all you need is a GPU for a +non-converted driver. The "Atomic mode setting design overview" series [2]_ +[3]_ at LWN.net can also be helpful. As part of this drivers also need to convert to universal plane (which means exposing primary & cursor as proper plane objects). But that's much easier to do by directly using the new atomic helper driver callbacks. + .. [1] https://blog.ffwll.ch/2014/11/atomic-modeset-support-for-kms-drivers.html + .. [2] https://lwn.net/Articles/653071/ + .. [3] https://lwn.net/Articles/653466/ + Contact: Daniel Vetter, respective driver maintainers Level: Advanced From patchwork Fri Jun 9 13:44:28 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Geert Uytterhoeven X-Patchwork-Id: 105642 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp959257vqr; Fri, 9 Jun 2023 06:46:42 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4lNWlP4zMoFgrJI0YlA1eUd30wr+s275ivsfD5uYVfq5S3zx8c7EMJE8CgoZFQMXmpAm8G X-Received: by 2002:a17:902:6b8a:b0:1ae:1237:8754 with SMTP id p10-20020a1709026b8a00b001ae12378754mr801644plk.68.1686318401926; Fri, 09 Jun 2023 06:46:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686318401; cv=none; d=google.com; s=arc-20160816; b=WVlRi7owCxmZVtsEkeE7S5i73d2h/aTeRUD4pyqN0wN4kNOJS7iFqBiPPb18jZsHQt ExXZjVJ3WAn8MnzNQC/5Hdk2ho5QK5qAgCoXgfnQ7QjGpTSG7/WznU+0Iq4s50+gsKck 5aE6qMze+oWt5UJtkd4q2DCWViqTuGLonelSiL0MmDaHwTTYiXtNB+fLC2lJw6ZzgdMb C8g3I9ECvOHVKkvEBKlFqkvj8aV2Va23jAU96qXg/itnuF0algY6kSyYo9+IrZHe56Rc 7JClIOA/F4O4lVNlMCGeBeFq841WKucYBwMQSXOHil3OCVakM1D5HRwN3SU8wRh1jlPV 4iZw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=X8XdEe60OXABPX+3bfCN6o4laDj1Nl5HYoXN6NJTO9c=; b=DhaTm0sAl9LmfGnOE6N4heX/T9AOH+5EL4TFSXZGJMHMakyPnFGoZKaumNCY4olp4+ AkHyl3P+RCWFWG/czBrvteIPP/14Dtezc/z9EnDZkNLibUtcqMS5wxBFQFFYnheWyrNJ UKfZqMQVIe+m6h7upwpiatqYsCnripea5+xRPslfQx++bRbfmxVg/Aeph2I6dh2waarW qftrBvvCChJ2Gghtmn7QLePw4vJt0JcQbrY9eQiUAa/uWcdD4TnaZwXpQz4ZBtZU3aWM Is2qvyJUh/AW9abnR7BNJrLM4bSBhp20K3sqp3rsq8OTQZH3aje0XYNAwdNWexG0Lru2 xM3w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id p13-20020a170902ebcd00b001b041591a5fsi2770844plg.459.2023.06.09.06.46.25; Fri, 09 Jun 2023 06:46:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232169AbjFINos (ORCPT + 99 others); Fri, 9 Jun 2023 09:44:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54744 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232327AbjFINoo (ORCPT ); Fri, 9 Jun 2023 09:44:44 -0400 Received: from baptiste.telenet-ops.be (baptiste.telenet-ops.be [IPv6:2a02:1800:120:4::f00:13]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 98C6230FA for ; Fri, 9 Jun 2023 06:44:41 -0700 (PDT) Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed30:a1e2:1b31:cba3:390d]) by baptiste.telenet-ops.be with bizsmtp id 71kZ2A00512zQ4r011kZdr; Fri, 09 Jun 2023 15:44:39 +0200 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.95) (envelope-from ) id 1q7cPS-008La7-38; Fri, 09 Jun 2023 15:44:33 +0200 Received: from geert by rox.of.borg with local (Exim 4.95) (envelope-from ) id 1q7cPp-00GCTT-43; Fri, 09 Jun 2023 15:44:33 +0200 From: Geert Uytterhoeven To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Jyri Sarha , Tomi Valkeinen , Laurent Pinchart , Bagas Sanjaya Cc: dri-devel@lists.freedesktop.org, linux-doc@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH v2 2/4] drm/todo: Convert list of fbconv links to footnotes Date: Fri, 9 Jun 2023 15:44:28 +0200 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1768233004331710712?= X-GMAIL-MSGID: =?utf-8?q?1768233004331710712?= Convert the references to fbconv links to footnotes, so they can be navigated. Signed-off-by: Geert Uytterhoeven --- v2: - New. --- Documentation/gpu/todo.rst | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 6c328613c049fc1d..ce1d4e22c327063b 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@ -753,16 +753,16 @@ existing hardware. The new driver's call-back functions are filled from existing fbdev code. More complex fbdev drivers can be refactored step-by-step into a DRM -driver with the help of the DRM fbconv helpers. [1] These helpers provide +driver with the help of the DRM fbconv helpers [4]_. These helpers provide the transition layer between the DRM core infrastructure and the fbdev driver interface. Create a new DRM driver on top of the fbconv helpers, copy over the fbdev driver, and hook it up to the DRM code. Examples for -several fbdev drivers are available at [1] and a tutorial of this process -available at [2]. The result is a primitive DRM driver that can run X11 +several fbdev drivers are available at [4]_ and a tutorial of this process +available at [5]_. The result is a primitive DRM driver that can run X11 and Weston. - - [1] https://gitlab.freedesktop.org/tzimmermann/linux/tree/fbconv - - [2] https://gitlab.freedesktop.org/tzimmermann/linux/blob/fbconv/drivers/gpu/drm/drm_fbconv_helper.c + .. [4] https://gitlab.freedesktop.org/tzimmermann/linux/tree/fbconv + .. [5] https://gitlab.freedesktop.org/tzimmermann/linux/blob/fbconv/drivers/gpu/drm/drm_fbconv_helper.c Contact: Thomas Zimmermann From patchwork Fri Jun 9 13:44:29 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Geert Uytterhoeven X-Patchwork-Id: 105643 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp959292vqr; Fri, 9 Jun 2023 06:46:48 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5KNoM+eMIrH+N2ptm0Zc+sHubQG04Bz/kNsq9Yk3+2J30vAVFTFOshxGbJbtppOlN4TUaz X-Received: by 2002:a05:6a20:3d21:b0:10a:dd79:65bd with SMTP id y33-20020a056a203d2100b0010add7965bdmr6072720pzi.27.1686318407695; Fri, 09 Jun 2023 06:46:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686318407; cv=none; d=google.com; s=arc-20160816; b=pJ0Ig2xcXtGzp1gUyw0Z6xagf/I+ydzC5Pnm98ule9vLPEugyHbSrDSeE84UifXJIe 1HXxBmaZWBV4FBHK9Xd5aspSyeA5I4QlmLlu9CYrlhZ4bbcW6nR6tfxLFs3jqEpMcd7G yHSQXdc3GsJo+KWoeax9h71knFSiAZNr2F3PrfAQzhFUxbQ9HBXCa6LPRDZEgTd2Eu9F DJS3HktF0L3CB7Upo34ueHlrk+qRp6p1MHRuL1YwUFZmgDLMc98ZlaCiV45IVqzsDrpE fZsKQ8d3y0Qe3ZxBTa09/EaSKO8qijLLYEQq7dZj03l+b8MMEpSeuSjKLXdB9kyE1LTm blnA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=t4FJCVSWvbAP0xoh7a7jK5XL/VGvXzUuDoaqAhPx13M=; b=0b8r49HGSGN/LZoR8efaPXbcy6B/AzKE9WqdSvs+MKq7wF68OMB+QT2qpQp2LhIs/f CGF5UqcgVlt/JY/yvBVXWzLnSnGF/V2RA2snzpzlCrAHuSys89d8lGmT8kGdg66wja6z MyS0QFaUOU+52xHr8mz1ZZb6rW7c2dYV/PLdN8TeI9RBUWvZ4y9eSG6wf7gp6Fa1lRVL LBBywAWZf3E442YuOAxZzKPr1PoDA97i+cf5jELEUV3KQW+lw/bUXwwwqDqTvgq8tBTg t2MCa1MR+fesaoAHiap4sV5v17euYORp4rO00PzKeg/srwMFiykf2jqjsq1bSGaO/Cc3 ZcfA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id u197-20020a6379ce000000b00535bf852410si2602255pgc.313.2023.06.09.06.46.33; Fri, 09 Jun 2023 06:46:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241122AbjFINo4 (ORCPT + 99 others); Fri, 9 Jun 2023 09:44:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54778 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239443AbjFINop (ORCPT ); Fri, 9 Jun 2023 09:44:45 -0400 Received: from michel.telenet-ops.be (michel.telenet-ops.be [IPv6:2a02:1800:110:4::f00:18]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 258363588 for ; Fri, 9 Jun 2023 06:44:42 -0700 (PDT) Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed30:a1e2:1b31:cba3:390d]) by michel.telenet-ops.be with bizsmtp id 71kZ2A00612zQ4r061kZpN; Fri, 09 Jun 2023 15:44:41 +0200 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.95) (envelope-from ) id 1q7cPS-008LaC-3m; Fri, 09 Jun 2023 15:44:33 +0200 Received: from geert by rox.of.borg with local (Exim 4.95) (envelope-from ) id 1q7cPp-00GCTY-4r; Fri, 09 Jun 2023 15:44:33 +0200 From: Geert Uytterhoeven To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Jyri Sarha , Tomi Valkeinen , Laurent Pinchart , Bagas Sanjaya Cc: dri-devel@lists.freedesktop.org, linux-doc@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH v2 3/4] drm: Remove references to removed transitional helpers Date: Fri, 9 Jun 2023 15:44:29 +0200 Message-Id: <8f5e13f8f2d12daab9e7f06bba88dc547a4db695.1686318012.git.geert+renesas@glider.be> X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1768233010597990895?= X-GMAIL-MSGID: =?utf-8?q?1768233010597990895?= The transitional helpers were removed a long time ago, but some references stuck. Remove them. Fixes: 21ebe615c16994f3 ("drm: Remove transitional helpers") Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- v2: - Drop "first part" in drivers/gpu/drm/drm_plane_helper.c. --- drivers/gpu/drm/drm_plane_helper.c | 12 +----- include/drm/drm_crtc.h | 5 --- include/drm/drm_modeset_helper_vtables.h | 48 +++++++++++------------- 3 files changed, 23 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c index c91e454eba097942..5e95089676ff81ed 100644 --- a/drivers/gpu/drm/drm_plane_helper.c +++ b/drivers/gpu/drm/drm_plane_helper.c @@ -40,8 +40,8 @@ /** * DOC: overview * - * This helper library has two parts. The first part has support to implement - * primary plane support on top of the normal CRTC configuration interface. + * This helper library contains helpers to implement primary plane support on + * top of the normal CRTC configuration interface. * Since the legacy &drm_mode_config_funcs.set_config interface ties the primary * plane together with the CRTC state this does not allow userspace to disable * the primary plane itself. The default primary plane only expose XRBG8888 and @@ -51,14 +51,6 @@ * planes, and newly merged drivers must not rely upon these transitional * helpers. * - * The second part also implements transitional helpers which allow drivers to - * gradually switch to the atomic helper infrastructure for plane updates. Once - * that switch is complete drivers shouldn't use these any longer, instead using - * the proper legacy implementations for update and disable plane hooks provided - * by the atomic helpers. - * - * Again drivers are strongly urged to switch to the new interfaces. - * * The plane helpers share the function table structures with other helpers, * specifically also the atomic helpers. See &struct drm_plane_helper_funcs for * the details. diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 8e1cbc75143ef216..8b48a1974da3143c 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -77,11 +77,6 @@ struct drm_plane_helper_funcs; * intended to indicate whether a full modeset is needed, rather than strictly * describing what has changed in a commit. See also: * drm_atomic_crtc_needs_modeset() - * - * WARNING: Transitional helpers (like drm_helper_crtc_mode_set() or - * drm_helper_crtc_mode_set_base()) do not maintain many of the derived control - * state like @plane_mask so drivers not converted over to atomic helpers should - * not rely on these being accurate! */ struct drm_crtc_state { /** @crtc: backpointer to the CRTC */ diff --git a/include/drm/drm_modeset_helper_vtables.h b/include/drm/drm_modeset_helper_vtables.h index 965faf082a6d1acb..e3c3ac615909474b 100644 --- a/include/drm/drm_modeset_helper_vtables.h +++ b/include/drm/drm_modeset_helper_vtables.h @@ -59,8 +59,8 @@ enum mode_set_atomic { /** * struct drm_crtc_helper_funcs - helper operations for CRTCs * - * These hooks are used by the legacy CRTC helpers, the transitional plane - * helpers and the new atomic modesetting helpers. + * These hooks are used by the legacy CRTC helpers and the new atomic + * modesetting helpers. */ struct drm_crtc_helper_funcs { /** @@ -216,9 +216,7 @@ struct drm_crtc_helper_funcs { * * This callback is used to update the display mode of a CRTC without * changing anything of the primary plane configuration. This fits the - * requirement of atomic and hence is used by the atomic helpers. It is - * also used by the transitional plane helpers to implement a - * @mode_set hook in drm_helper_crtc_mode_set(). + * requirement of atomic and hence is used by the atomic helpers. * * Note that the display pipe is completely off when this function is * called. Atomic drivers which need hardware to be running before they @@ -333,8 +331,8 @@ struct drm_crtc_helper_funcs { * all updated. Again the recommendation is to just call check helpers * until a maximal configuration is reached. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. + * This callback is used by the atomic modeset helpers, but it is + * optional. * * NOTE: * @@ -373,8 +371,8 @@ struct drm_crtc_helper_funcs { * has picked. See drm_atomic_helper_commit_planes() for a discussion of * the tradeoffs and variants of plane commit helpers. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. + * This callback is used by the atomic modeset helpers, but it is + * optional. */ void (*atomic_begin)(struct drm_crtc *crtc, struct drm_atomic_state *state); @@ -397,8 +395,8 @@ struct drm_crtc_helper_funcs { * has picked. See drm_atomic_helper_commit_planes() for a discussion of * the tradeoffs and variants of plane commit helpers. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. + * This callback is used by the atomic modeset helpers, but it is + * optional. */ void (*atomic_flush)(struct drm_crtc *crtc, struct drm_atomic_state *state); @@ -507,8 +505,8 @@ static inline void drm_crtc_helper_add(struct drm_crtc *crtc, /** * struct drm_encoder_helper_funcs - helper operations for encoders * - * These hooks are used by the legacy CRTC helpers, the transitional plane - * helpers and the new atomic modesetting helpers. + * These hooks are used by the legacy CRTC helpers and the new atomic + * modesetting helpers. */ struct drm_encoder_helper_funcs { /** @@ -1185,8 +1183,7 @@ static inline void drm_connector_helper_add(struct drm_connector *connector, /** * struct drm_plane_helper_funcs - helper operations for planes * - * These functions are used by the atomic helpers and by the transitional plane - * helpers. + * These functions are used by the atomic helpers. */ struct drm_plane_helper_funcs { /** @@ -1221,9 +1218,8 @@ struct drm_plane_helper_funcs { * The helpers will call @cleanup_fb with matching arguments for every * successful call to this hook. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. See @begin_fb_access - * for preparing per-commit resources. + * This callback is used by the atomic modeset helpers, but it is + * optional. See @begin_fb_access for preparing per-commit resources. * * RETURNS: * @@ -1240,8 +1236,8 @@ struct drm_plane_helper_funcs { * This hook is called to clean up any resources allocated for the given * framebuffer and plane configuration in @prepare_fb. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. + * This callback is used by the atomic modeset helpers, but it is + * optional. */ void (*cleanup_fb)(struct drm_plane *plane, struct drm_plane_state *old_state); @@ -1295,8 +1291,8 @@ struct drm_plane_helper_funcs { * all updated. Again the recommendation is to just call check helpers * until a maximal configuration is reached. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. + * This callback is used by the atomic modeset helpers, but it is + * optional. * * NOTE: * @@ -1326,8 +1322,7 @@ struct drm_plane_helper_funcs { * has picked. See drm_atomic_helper_commit_planes() for a discussion of * the tradeoffs and variants of plane commit helpers. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. + * This callback is used by the atomic modeset helpers, but it is optional. */ void (*atomic_update)(struct drm_plane *plane, struct drm_atomic_state *state); @@ -1376,9 +1371,8 @@ struct drm_plane_helper_funcs { * has picked. See drm_atomic_helper_commit_planes() for a discussion of * the tradeoffs and variants of plane commit helpers. * - * This callback is used by the atomic modeset helpers and by the - * transitional plane helpers, but it is optional. It's intended to - * reverse the effects of @atomic_enable. + * This callback is used by the atomic modeset helpers, but it is + * optional. It's intended to reverse the effects of @atomic_enable. */ void (*atomic_disable)(struct drm_plane *plane, struct drm_atomic_state *state); From patchwork Fri Jun 9 13:44:30 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Geert Uytterhoeven X-Patchwork-Id: 105648 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp964833vqr; Fri, 9 Jun 2023 06:56:55 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ66ZfOYIcHkzuKd5V7Dl7Yu08MAhQIOu7iJVXBx3AVP0uQ6VrKRZwco66QfAPgUjF7QTp/G X-Received: by 2002:a05:6a20:3d81:b0:115:83f:fce1 with SMTP id s1-20020a056a203d8100b00115083ffce1mr1756235pzi.5.1686319014671; Fri, 09 Jun 2023 06:56:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686319014; cv=none; d=google.com; s=arc-20160816; b=UVjvKXnb8hSdQ5tdGTXn+jpDB4hzMJyDTen/rp9yX50tpv2EprGvlCDqG1wMIifzHG gXO8BA8z1DF5x1RnPV9OVR0Q0CKDADpD3vi3eboXNLv+l2xIkLHIj2kcwxltc9fwWCSN TTpZJ3fz5SK2wWIwkvW222a1AlItEjkw86Gb4z9ZDeOMg6JNlJ9s1CpIaJSmUlqPOcFN VK2f4IzQD4EUGhVgDLN7gnzt3BLNN/ybgUsDmf/8Qf5X1bkC/Zw97G9zij91swtKpKhX AuExWUAoIKBhHWq+WIOUgjc3wty/3ocm9+Kv7IxR6ojiOC957tW3F6FHKexqH9ti4Dig HomA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=O/sgh+hJuNWbSPnHJ4l4RX0tR2ONfUNesLjrQxzdJoU=; b=gghBkpfqvmka8oPczkiuCQzUx1v7+cIOKfiy6AyiOXpuCvPiUIcyGa/kU6CU7bZoDe zgnPCO6ToEJj5KGzRyYeEl4//7QYYdm1laNHeL+YCu2HXGDoql1KdDavHB2JI3O+2DYl IrWkE6+6ySLl1B1wSFAKzMU0slMrZJCHhJl7f4Izwe7E/h6hXe92c8Dx0XipRv6MKrhN 7Y2y8WFbPv+U58EqztRL7m0ltKcW65g0M+IgjqiVNRoWffL8sYfrnojABwFVufr2TH0k NrsZD3qZuGmet2Ykvatowyv8RQXwNNjlHv1Yhpyu7crTVdOxqvMx0DgWhQaBjvaMpjuU gsyQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b27-20020a63931b000000b00524b2358ffcsi2750493pge.574.2023.06.09.06.56.36; Fri, 09 Jun 2023 06:56:54 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240763AbjFINow (ORCPT + 99 others); Fri, 9 Jun 2023 09:44:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54760 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238562AbjFINoo (ORCPT ); Fri, 9 Jun 2023 09:44:44 -0400 Received: from xavier.telenet-ops.be (xavier.telenet-ops.be [IPv6:2a02:1800:120:4::f00:14]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 483AB3580 for ; Fri, 9 Jun 2023 06:44:42 -0700 (PDT) Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed30:a1e2:1b31:cba3:390d]) by xavier.telenet-ops.be with bizsmtp id 71kZ2A00E12zQ4r011kZNt; Fri, 09 Jun 2023 15:44:40 +0200 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.95) (envelope-from ) id 1q7cPS-008LaF-4Y; Fri, 09 Jun 2023 15:44:33 +0200 Received: from geert by rox.of.borg with local (Exim 4.95) (envelope-from ) id 1q7cPp-00GCTc-5T; Fri, 09 Jun 2023 15:44:33 +0200 From: Geert Uytterhoeven To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Jyri Sarha , Tomi Valkeinen , Laurent Pinchart , Bagas Sanjaya Cc: dri-devel@lists.freedesktop.org, linux-doc@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH v2 4/4] drm: Fix references to drm_plane_helper_check_state() Date: Fri, 9 Jun 2023 15:44:30 +0200 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1768233647208867121?= X-GMAIL-MSGID: =?utf-8?q?1768233647208867121?= As of commit a01cb8ba3f628293 ("drm: Move drm_plane_helper_check_state() into drm_atomic_helper.c"), drm_plane_helper_check_state() no longer exists, but is part of drm_atomic_helper_check_plane_state(). Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- v2: - Add Reviewed-by. --- drivers/gpu/drm/rcar-du/rcar_du_plane.c | 3 ++- drivers/gpu/drm/tidss/tidss_plane.c | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/rcar-du/rcar_du_plane.c index d759e019218181ce..e445fac8e0b46c21 100644 --- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c +++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c @@ -600,7 +600,8 @@ int __rcar_du_plane_atomic_check(struct drm_plane *plane, if (!state->crtc) { /* * The visible field is not reset by the DRM core but only - * updated by drm_plane_helper_check_state(), set it manually. + * updated by drm_atomic_helper_check_plane_state(), set it + * manually. */ state->visible = false; *format = NULL; diff --git a/drivers/gpu/drm/tidss/tidss_plane.c b/drivers/gpu/drm/tidss/tidss_plane.c index 6bdd6e4a955ab3cc..e1c0ef0c3894c855 100644 --- a/drivers/gpu/drm/tidss/tidss_plane.c +++ b/drivers/gpu/drm/tidss/tidss_plane.c @@ -38,7 +38,8 @@ static int tidss_plane_atomic_check(struct drm_plane *plane, if (!new_plane_state->crtc) { /* * The visible field is not reset by the DRM core but only - * updated by drm_plane_helper_check_state(), set it manually. + * updated by drm_atomic_helper_check_plane_state(), set it + * manually. */ new_plane_state->visible = false; return 0;