From patchwork Sun Mar 31 20:28:57 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13612280 Received: from mail-lf1-f51.google.com (mail-lf1-f51.google.com [209.85.167.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8431E1474AF for ; Sun, 31 Mar 2024 20:29:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711916950; cv=none; b=PaNG/TQVhyhSTEV1Vtk72GwT4bzqBgEbtwt9w1bUCvMCbFhYMoyazWHKph/N15splTLWNSOxGjKbVjoLagxuh/6Hi6U4r6DQQoOthSDcvPQzOjFQxx7xa55ORE5gE4EinyT+GKbbxADwiuwrBSb5XS9EsKVVRsfRJ5nTvg7uQIQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711916950; c=relaxed/simple; bh=zVnbAbPF8sxPI5DAV4JMslU2Y9lcTwAMMXk+k4+/EQA=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=ZOaa6gLS9SO6pKsYHgy3tUHeATfjOMCVgPA/2DDNyJmCOXCJMXy27iLXkmngkq0jWAbDa/z5vT8yI+/dxa35sbdS0GWFMZyIewHrugXTbWf0aNpVT+9sF32ERyCphf92c3L/myrOL9lVY0L5wgkoqQAuPRT2u3tVhBUkdkAi+gw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=u5dlbuw8; arc=none smtp.client-ip=209.85.167.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="u5dlbuw8" Received: by mail-lf1-f51.google.com with SMTP id 2adb3069b0e04-515a81928faso4226907e87.1 for ; Sun, 31 Mar 2024 13:29:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1711916946; x=1712521746; darn=lists.linux.dev; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=Ke8yGtCVv/5QoZ8dU5U7lxdg3fQhjUiZTY8VG7siut0=; b=u5dlbuw8ebtChsNrqQYJPTmDve4jeT4s035fN3C/AEywi3EnvWImUxGrWutTTqUVLk zPUoek6KZOOIl87+C/12468ByXW/hdRcGrBHAADJg74aqhG2qV0MaYCYDIZpC9NiojPr 1XO/ipgHE/tp1U6ZaHi3aYC3nKlJyFhe7ey6hOmOoDMk86n3tVYGsFUnAZZKotmc+HbM Jk8/Dk5iJFetB5Ius5cSxi4+7suuEetaC1CZpQ7iYAmYCj33RSnqRAbgWo8APU3lutTh CZYHLktyZROuJtDVUwqutKj9FQdU4yQUT+H6mD+IR7eYR4oyZBD5tP14w43quIHl2nCw 55Pw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711916946; x=1712521746; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Ke8yGtCVv/5QoZ8dU5U7lxdg3fQhjUiZTY8VG7siut0=; b=vMV9oK3XD668yz4rnAFjdQX/dSS48dMshcqZ82BmSROsb50I5SfygHf2JBTT39YVl+ lGNth4FWJ2B/8rwGFr6PZxJIg9uq+htpGUbqkh4Fe6mRsfYe1srIo27k4PVqMALk5IQ/ LlhyGzhp4i2XL7uFMFaaFvJSZ9MBVavpGq3SwU5suq2978qsy/Tq+jO/mWc3DIgSqYe6 E5lX0rC8C6+rrFMFyiyWlGz7O3ffRr1zneS+xC3yjEMrFg9WIDe5LGBcV6wjY3WqdLJY pdroJNrwpGUZ1dmi5MkxIky6P+ituLv4yGTqPVE5YhsnkThawxVYry5gez6BMwWicMoi 3icQ== X-Forwarded-Encrypted: i=1; AJvYcCWAsleQRQ6/c6WZh4F9DeNp6EEp5AKkbHjL2VaPynNZsQOiN/UrEEfiT7H4pSpV0TqTnZC+N4r7T/GAKT7/6JbXKt6f X-Gm-Message-State: AOJu0YzhZ5WV/g+2BOJBIcxpk9IWvmdLuZ9eS9pZpq0ntTBo11ZaRK1U 0Yk1fqAqV4s6lP+9KIdlHgATbcXpXO0fSIs+pv7h2cyxHQp19b1Xq6W6NHTLLls= X-Google-Smtp-Source: AGHT+IHLQL1U5tLEA/uaNC1Wpt0XJL27jBfG3hvdugM1uDaF7fxhj5M2+UlDtzseElbw9HxjBQ7osg== X-Received: by 2002:ac2:5224:0:b0:513:5eea:2942 with SMTP id i4-20020ac25224000000b005135eea2942mr4878774lfl.37.1711916945549; Sun, 31 Mar 2024 13:29:05 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id y25-20020a197519000000b00513cfc2a7aesm1237276lfe.71.2024.03.31.13.29.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 31 Mar 2024 13:29:05 -0700 (PDT) From: Dmitry Baryshkov Subject: [PATCH v2 00/12] drm/imx/ipuv3: switch LDB and parallel-display driver to use drm_bridge_connector Date: Sun, 31 Mar 2024 23:28:57 +0300 Message-Id: <20240331-drm-imx-cleanup-v2-0-d81c1d1c1026@linaro.org> Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAInHCWYC/2WNQQ6CMBBFr0Jm7ZhOBYmuvIdhUWGASaAlUyUYw t2tJK5cvpf891eIrMIRrtkKyrNECT6BPWRQ9853jNIkBmtsbk5ksNERZVywHtj514RkyvzsLo+ S2gLSalJuZdmL9ypxL/EZ9L0fzPS1vxb9tWZCg0wmb01RO1fQbRDvNByDdlBt2/YBoEvwOq8AA AA= To: Philipp Zabel , David Airlie , Daniel Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam Cc: Chris Healy , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, Dmitry Baryshkov X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=2868; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=zVnbAbPF8sxPI5DAV4JMslU2Y9lcTwAMMXk+k4+/EQA=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBmCceOK2zSH8bT5rAhCDbo6xqbRSw4VoyfN7T5Y JX4uWZfri6JATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZgnHjgAKCRCLPIo+Aiko 1ZDUB/4jbmbPWrHNwE9pT9AGhCIMSIBHR72AqX2B7V9sYkYp0RP+BUdO3z1dYHaCioJXBw5XLkY nn8BV/5o2wiXZfxlRwFLe/6f4RLBGYi1Pd+1YIk/cv6BOWQuquu6ivk22dJXdeh5WGrKGkmp0TL xOH7pS66fmxI/cLmNzUJNqOyPRLXG2D+9L/gllrrAA3ZL0V6AJISK/pFL1uXe99NgBWlK8utAxK Xn6PlF9WZOdeoyl6wvVVb8DnTovfpY4cVo/po7d0JKuMhE4YC2sVWZkrgrqeghJbpDjSjS+wG8w CqaBQmEvoIqtcoWzH+QtrbxF5wnL9CHF7SASz9g3cyRulpAM X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A The IPUv3 DRM i.MX driver contains several codepaths for different usescases: both LDB and paralllel-display drivers handle next-bridge, panel and the legacy display-timings DT node on their own. Drop unused ddc-i2c-bus and edid handling (none of the DT files merged upstream ever used these features), switch to panel-bridge driver, removing the need to handle drm_panel codepaths separately and finally switch to drm_bridge_connector, removing requirement for the downstream bridges to create drm_connector on their own. This has been tested on the iMX53 with the DPI panel attached to LDB via LVDS decoder, using all possible usecases (lvds-codec + panel, panel linked directly to LDB node and the display-timings node). Signed-off-by: Dmitry Baryshkov --- Changes in v2: - Fixed drm_bridge_attach flags in imx/parallel-display driver. - Moved the legacy bridge to drivers/gpu/drm/bridge - Added missing EXPORT_SYMBOL_GPL to the iMX legacy bridge - Link to v1: https://lore.kernel.org/r/20240311-drm-imx-cleanup-v1-0-e104f05caa51@linaro.org --- Dmitry Baryshkov (12): dt-bindings: display: fsl-imx-drm: drop edid property support dt-bindings: display: imx/ldb: drop ddc-i2c-bus property drm/imx: cleanup the imx-drm header drm/imx: parallel-display: drop edid override support drm/imx: ldb: drop custom EDID support drm/imx: ldb: drop custom DDC bus support drm/imx: ldb: switch to drm_panel_bridge drm/imx: parallel-display: switch to drm_panel_bridge drm/imx: add internal bridge handling display-timings DT node drm/imx: ldb: switch to imx_legacy_bridge / drm_bridge_connector drm/imx: parallel-display: switch to imx_legacy_bridge / drm_bridge_connector drm/imx: move imx_drm_connector_destroy to imx-tve .../bindings/display/imx/fsl-imx-drm.txt | 2 - .../devicetree/bindings/display/imx/ldb.txt | 1 - drivers/gpu/drm/bridge/imx/Kconfig | 10 + drivers/gpu/drm/bridge/imx/Makefile | 1 + drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c | 85 +++++++++ drivers/gpu/drm/imx/ipuv3/Kconfig | 5 + drivers/gpu/drm/imx/ipuv3/imx-drm-core.c | 7 - drivers/gpu/drm/imx/ipuv3/imx-drm.h | 14 -- drivers/gpu/drm/imx/ipuv3/imx-ldb.c | 202 +++++---------------- drivers/gpu/drm/imx/ipuv3/imx-tve.c | 8 +- drivers/gpu/drm/imx/ipuv3/parallel-display.c | 137 +++----------- include/drm/bridge/imx.h | 13 ++ 12 files changed, 184 insertions(+), 301 deletions(-) --- base-commit: 13ee4a7161b6fd938aef6688ff43b163f6d83e37 change-id: 20240310-drm-imx-cleanup-10746a9b71f5 Best regards,