From patchwork Fri May 3 21:32:41 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Doug Anderson X-Patchwork-Id: 13653498 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 A2756C25B5C for ; Fri, 3 May 2024 21:36:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=81bchGQ8nZ9XaN9dKs95FGRryKNeFYsy2rOrwjJbFWw=; b=AIHw8mo4p05z+j vACpoGUhFQsqcDKbysRlMNSJVol8US74evVXSzbBlwxNfKax16IlrgZbszKRbCOIYlOgLpjqe2v7k q9pJJgvTwtrMN9VVXmjidpQeXDjeJ9cRQKlA1MAeV5LhHJygMbk7dBM9o8ZZgU2CCwaxAp3KOQX1V 6yU6xq55+MLfcZKOCCQ9LNTzKCZ83392wgxm9KxTyNQFwxLWrwfPpLis0OKhGkcakegU8M8lE+U+J CxsKFBRofUkUQPpq9VbYwxwUPtJ1Bv6OBY7Awxb4QQNR6JW1kROKutVcTJNrN9xOjo7Y+rTKWi+8j VXYOF8KJz8HMUFMlzIag==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s30Zx-00000000RTA-0i47; Fri, 03 May 2024 21:36:29 +0000 Received: from mail-pl1-x62c.google.com ([2607:f8b0:4864:20::62c]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s30Zt-00000000RQA-2dXs for linux-arm-kernel@lists.infradead.org; Fri, 03 May 2024 21:36:27 +0000 Received: by mail-pl1-x62c.google.com with SMTP id d9443c01a7336-1ec69e3dbcfso1118705ad.0 for ; Fri, 03 May 2024 14:36:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1714772183; x=1715376983; darn=lists.infradead.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=O7IvcNsNHU8RMEEGHB36x94SUhZEWVNbyGS+d0gObkA=; b=Jdf7IlGsgDofBKyXGg8GLReR2GgNoDv8tZ2MWSaHP8tpM/t/tsGO73fXKXpmiTNw4P fwosJNKar6ymJ3PuJsZ8VgDQcS6Ete51OyzleByEyiG1cUNZAD6qMOTBlOY8IaEEk4cP tx24uPt2jCTzbhlp0KN3ledXvkjsclmjNy92k= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1714772183; x=1715376983; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=O7IvcNsNHU8RMEEGHB36x94SUhZEWVNbyGS+d0gObkA=; b=rp7AHBFsuD8Eab0+nW8YRjDH9KsCLCv8Jpxa399/5dqKrV3Qs0gosSCR9Ozzr9MyIz 4EIZOlAJV7DCphhskNYvnZRoMrwVuMFWBm5Xn3m9Ei5LZbywFrhUxJIdCGpjxH9aU5aL cxM9ORWRgEen7HOFEJWiil63ZjtkXxnP1gYkUsVHz7Jzx9xo2EYsIYaW4GuDOYr6sAGP CXI052MNBqn8zsYLd05YS5QC+yK/mXLrQZbz9VtEEN6QZCmDiGirPTu/AvHFPa6toErd wYMyJuC0F6W42y889FDRkEX1VJpC0WiWWdkipZZemvURsSA2iYIRGzjW2heCSMwhdITL 2JqQ== X-Forwarded-Encrypted: i=1; AJvYcCXsBYMW65Jo7VJkqK786htYH3r2htA2FbExudpsVfPggRi9yz9mPgULwxWjA1Fyz8g6Kqb5PkqP16h2Z0MMUMC2qtD50VLvY3lifzl6t6kHYzYBqEc= X-Gm-Message-State: AOJu0YxHUMT8yFOT5cmMoQU+PI8156S4ZVABhZhqnMZ1iiyVijLhAbGk m3N5OoC/MH8Nx/1x0MeDB/oyh0FZs0Je53EOWit/qmiG6wlB8Bv9lnu9C1mAHA== X-Google-Smtp-Source: AGHT+IH5eduOHdb1FCgLuhTvqIJg3l60grEr/JNT4BiC6IxKFTTgVq2iMUFgza4DOD4xTrjDsAYlqw== X-Received: by 2002:a17:902:e882:b0:1eb:4950:554f with SMTP id w2-20020a170902e88200b001eb4950554fmr4690309plg.22.1714772183009; Fri, 03 May 2024 14:36:23 -0700 (PDT) Received: from dianders.sjc.corp.google.com ([2620:15c:9d:2:fb6a:b54b:7580:64f3]) by smtp.gmail.com with ESMTPSA id j12-20020a170903024c00b001eb51a46f5bsm3729134plh.43.2024.05.03.14.36.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 May 2024 14:36:22 -0700 (PDT) From: Douglas Anderson To: dri-devel@lists.freedesktop.org, Maxime Ripard Cc: Linus Walleij , Chris Morgan , Yuran Pereira , Neil Armstrong , Douglas Anderson , AngeloGioacchino Del Regno , Daniel Vetter , David Airlie , =?utf-8?q?Guido_G=C3=BCnther?= , Jerry Han , Jessica Zhang , Jonathan Corbet , Maarten Lankhorst , Matthias Brugger , Ondrej Jirman , Purism Kernel Team , Robert Chiras , Sam Ravnborg , Stefan Mavrodiev , Sumit Semwal , Thomas Zimmermann , linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org Subject: [RFT PATCH v2 00/48] drm/panel: Remove most store/double-check of prepared/enabled state Date: Fri, 3 May 2024 14:32:41 -0700 Message-ID: <20240503213441.177109-1-dianders@chromium.org> X-Mailer: git-send-email 2.45.0.rc1.225.g2a3ae87e7f-goog MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240503_143625_756096_C308968D X-CRM114-Status: GOOD ( 22.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org As talked about in commit d2aacaf07395 ("drm/panel: Check for already prepared/enabled in drm_panel"), we want to remove needless code from panel drivers that was storing and double-checking the prepared/enabled state. Even if someone was relying on the double-check before, that double-check is now in the core and not needed in individual drivers. This series attempts to do just that. While the original grep, AKA: git grep 'if.*>prepared' -- drivers/gpu/drm/panel git grep 'if.*>enabled' -- drivers/gpu/drm/panel ...still produces a few hits after my series, they are _mostly_ all gone. The ones that are left are less trivial to fix. One of the main reasons that many panels probably needed to store and double-check their prepared/enabled appears to have been to handle shutdown and/or remove. Panels drivers often wanted to force the power off for panels in these cases and this was a good reason for the double-check. In response to my V1 series [1] we had much discussion of what to do. The conclusion was that as long as DRM modeset drivers properly called drm_atomic_helper_shutdown() that we should be able to remove the explicit shutdown/remove handling in the panel drivers. Most of the patches to improve DRM modeset drivers [2] [3] [4] have now landed. In contrast to my V1 series, I broke the V2 series up a lot more. Since a few of the panel drivers in V1 already landed, we had fewer total drivers and so we could devote a patch to each panel. Also, since we were now relying on DRM modeset drivers I felt like we should split the patches for each panel into two: one that's definitely safe and one that could be reverted if we found a problematic DRM modeset driver that we couldn't fix. Sorry for the large number of patches. I've set things to mostly just CC people on the cover letter and the patches that are relevant to them. I've tried to CC people on the whole series that have shown interest in this TODO item. As patches in this series are reviewed and/or tested they could be landed. There's really no ordering requirement for the series unless patches touch the same driver. NOTE: this touches _a lot_ of drivers, is repetitive, and is not really possible to generate automatically. That means it's entirely possible that my eyes glazed over and I did something wrong. Please double-check me and don't assume that I got everything perfect, though I did my best. I have at least confirmed that "allmodconfig" for arm64 doesn't fall on its face with this series. I haven't done a ton of other testing. [1] https://lore.kernel.org/r/20230804140605.RFC.4.I930069a32baab6faf46d6b234f89613b5cec0f14@changeid [2] https://lore.kernel.org/r/20230901234015.566018-1-dianders@chromium.org [3] https://lore.kernel.org/r/20230901234202.566951-1-dianders@chromium.org [4] https://lore.kernel.org/r/20230921192749.1542462-1-dianders@chromium.org Changes in v2: - ("drm/panel: raydium-rm692e5: Stop tracking prepared") new for v2. - Only handle 1 panel per patch. - Split removal of prepared/enabled from handling of remove/shutdown. - panel-edp and panel-simple just get a comment now. Douglas Anderson (48): drm/panel: raydium-rm692e5: Stop tracking prepared drm/panel: boe-himax8279d: Stop tracking prepared/enabled drm/panel: boe-himax8279d: Don't call unprepare+disable at shutdown/remove drm/panel: boe-tv101wum-nl6: Stop tracking prepared drm/panel: boe-tv101wum-nl6: Don't call unprepare+disable at shutdown/remove drm/panel: edp: Stop tracking prepared/enabled drm/panel: edp: Add a comment about unprepare+disable at shutdown/remove drm/panel: innolux-p079zca: Stop tracking prepared/enabled drm/panel: innolux-p079zca: Don't call unprepare+disable at shutdown/remove drm/panel: khadas-ts050: Stop tracking prepared/enabled drm/panel: khadas-ts050: Don't call unprepare+disable at shutdown/remove drm/panel: kingdisplay-kd097d04: Stop tracking prepared/enabled drm/panel: kingdisplay-kd097d04: Don't call unprepare+disable at shutdown/remove drm/panel: ltk050h3146w: Stop tracking prepared drm/panel: ltk050h3146w: Don't call unprepare+disable at shutdown/remove drm/panel: ltk500hd1829: Stop tracking prepared drm/panel: ltk500hd1829: Don't call unprepare+disable at shutdown/remove drm/panel: novatek-nt36672a: Stop tracking prepared drm/panel: novatek-nt36672a: Don't call unprepare+disable at shutdown/remove drm/panel: olimex-lcd-olinuxino: Stop tracking prepared/enabled drm/panel: olimex-lcd-olinuxino: Don't call unprepare+disable at remove drm/panel: osd-osd101t2587-53ts: Stop tracking prepared/enabled drm/panel: osd-osd101t2587-53ts: Don't call unprepare+disable at shutdown/remove drm/panel: samsung-atna33xc20: Stop tracking prepared/enabled drm/panel: samsung-atna33xc20: Don't call unprepare+disable at shutdown/remove drm/panel: simple: Stop tracking prepared/enabled drm/panel: simple: Add a comment about unprepare+disable at shutdown/remove drm/panel: tdo-tl070wsh30: Stop tracking prepared drm/panel: tdo-tl070wsh30: Don't call unprepare+disable at shutdown/remove drm/panel: xinpeng-xpp055c272: Stop tracking prepared drm/panel: xinpeng-xpp055c272: Don't call unprepare+disable at shutdown/remove drm/panel: jdi-lt070me05000: Stop tracking prepared/enabled drm/panel: jdi-lt070me05000: Don't call disable at shutdown/remove drm/panel: panasonic-vvx10f034n00: Stop tracking prepared/enabled drm/panel: panasonic-vvx10f034n00: Don't call disable at shutdown/remove drm/panel: seiko-43wvf1g: Stop tracking prepared/enabled drm/panel: seiko-43wvf1g: Don't call disable at shutdown/remove drm/panel: sharp-lq101r1sx01: Stop tracking prepared/enabled drm/panel: sharp-lq101r1sx01: Don't call disable at shutdown/remove drm/panel: sharp-ls043t1le01: Stop tracking prepared drm/panel: sharp-ls043t1le01: Don't call disable at shutdown/remove drm/panel: sitronix-st7703: Stop tracking prepared drm/panel: sitronix-st7703: Don't call disable at shutdown/remove drm/panel: raydium-rm67191: Stop tracking enabled drm/panel: raydium-rm67191: Don't call unprepare+disable at shutdown drm/panel: sony-acx565akm: Don't double-check enabled state in disable drm/panel: sony-acx565akm: Don't call disable at remove drm/panel: Update TODO list item for cleaning up prepared/enabled tracking Documentation/gpu/todo.rst | 47 +++++++------- drivers/gpu/drm/panel/panel-boe-himax8279d.c | 40 ------------ .../gpu/drm/panel/panel-boe-tv101wum-nl6.c | 23 ------- drivers/gpu/drm/panel/panel-edp.c | 60 +++++++----------- drivers/gpu/drm/panel/panel-innolux-p079zca.c | 55 ---------------- .../gpu/drm/panel/panel-jdi-lt070me05000.c | 35 ----------- drivers/gpu/drm/panel/panel-khadas-ts050.c | 39 ------------ .../drm/panel/panel-kingdisplay-kd097d04.c | 48 -------------- .../drm/panel/panel-leadtek-ltk050h3146w.c | 28 --------- .../drm/panel/panel-leadtek-ltk500hd1829.c | 28 --------- .../gpu/drm/panel/panel-novatek-nt36672a.c | 29 --------- .../drm/panel/panel-olimex-lcd-olinuxino.c | 44 ------------- .../drm/panel/panel-osd-osd101t2587-53ts.c | 41 +----------- .../drm/panel/panel-panasonic-vvx10f034n00.c | 47 +------------- drivers/gpu/drm/panel/panel-raydium-rm67191.c | 26 -------- drivers/gpu/drm/panel/panel-raydium-rm692e5.c | 10 --- .../gpu/drm/panel/panel-samsung-atna33xc20.c | 36 ----------- drivers/gpu/drm/panel/panel-seiko-43wvf1g.c | 49 --------------- .../gpu/drm/panel/panel-sharp-lq101r1sx01.c | 63 +------------------ .../gpu/drm/panel/panel-sharp-ls043t1le01.c | 24 ------- drivers/gpu/drm/panel/panel-simple.c | 60 +++++++----------- drivers/gpu/drm/panel/panel-sitronix-st7703.c | 35 +++-------- drivers/gpu/drm/panel/panel-sony-acx565akm.c | 6 -- drivers/gpu/drm/panel/panel-tdo-tl070wsh30.c | 23 ------- .../gpu/drm/panel/panel-xinpeng-xpp055c272.c | 28 --------- 25 files changed, 83 insertions(+), 841 deletions(-) Acked-by: Linus Walleij Acked-by: Maxime Ripard