From patchwork Mon May 6 13:34:29 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michael Walle X-Patchwork-Id: 13655437 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 C42DAC25B4F for ; Mon, 6 May 2024 13:35:11 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id D013610EB5B; Mon, 6 May 2024 13:35:10 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="XtKnAA/X"; dkim-atps=neutral Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by gabe.freedesktop.org (Postfix) with ESMTPS id 25E2610EB5B for ; Mon, 6 May 2024 13:35:09 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 2B2BECE0E76; Mon, 6 May 2024 13:35:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8C0E4C116B1; Mon, 6 May 2024 13:34:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715002503; bh=7KQKdwRJtBFZkLdfMhs2OLN3qUwlGvTF8MBUDqOjd3s=; h=From:Subject:Date:To:Cc:From; b=XtKnAA/XWOfsPqfFPNMzDVsGyICjYHcSlf8yN7QfZtjlSZqSiy5hAzHxSY8rsgFJw 3/hyujLhOEw1+5r+M6JvWl7pIFUa759fB+oMeFWlMjrnpyy/jbd6Wcgfw5Y7L8e4ew FOZG46X4xaoX09TdyFSn6ZQf1dSKkjYei64HiVGsq42hEqOtxuBnw7BwPEPZRpVhhy KTQDbLBkXcbirv4Ug2rw8AvhFbgFpQcT2myKjRx2vLHBf83Szm0wich1/Jp9Zq51Ge s0S261RfU+xxKJ0DjL64p/1LO0WzWkNKtRwEyPkWRG/lQJshAOVx0XKzGhpo1TToeS 9xAidLnwcpCgg== From: Michael Walle Subject: [PATCH 00/20] drm/bridge: tc358775: proper bridge bringup and code cleanup Date: Mon, 06 May 2024 15:34:29 +0200 Message-Id: <20240506-tc358775-fix-powerup-v1-0-545dcf00b8dd@kernel.org> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAGXcOGYC/x2Myw5AMBAAf0X2bJOlreJXxEFY7IWm9Uoa/65xn ExmIgT2wgHaLILnS4LsW4Iiz2Bch21hlCkxlFRqMlThMSpTW2twlgfdfrM/HRo164ZIq8I2kFL nOel/2/Xv+wEve/DHZgAAAA== To: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Chun-Kuang Hu , Philipp Zabel , Matthias Brugger , AngeloGioacchino Del Regno , Sam Ravnborg , Vinay Simha BN , Tony Lindgren Cc: Daniel Semkowicz , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, Michael Walle X-Mailer: b4 0.12.4 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: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" This patchset fixes the bridge initialization according to the datasheet. Not sure how that even worked before. Maybe because the initialization was done prior to linux (?). The bridge has some peculiarities: (1) The reset has to be deasserted in LP-11 mode (2) For I2C access the bridge needs the DSI clock (3) The bridge has to be configured while the video stream is disabled. (4) The bridge has limitations on the display timings. In particular, the horizontal pulse width has to be at least 8 pixels wide and both the horizontal pulse width as well as the back porch has to be even. According to the datasheet the horizontal front porch as well but in line sync mode, this is ignored. Also line sync is the only supported mode for this bridge, therefore, the front porch is always ignored. The most controversial patch is probably "drm/bridge: add dsi_lp11_notify mechanism" which is needed for (1) above. Some time ago there was a series [1] to add a manual power-up, which was abandoned and which didn't suite the needs for this bridge anyway. Also, this will gradually change the tc_ prefix to tc358775_ while the functions are refactored. The bridge was successfully tested on a Mediatek MT8195 SoC with the following panels: - Innolux G101ICE - AUO G121EAN01.0 - Innolux G156HCE (dual-link LVDS) [1] https://lore.kernel.org/r/20231016165355.1327217-1-dmitry.baryshkov@linaro.org/ Signed-off-by: Michael Walle --- Michael Walle (20): drm/bridge: add dsi_lp11_notify mechanism drm/mediatek: dsi: provide LP-11 mode during .pre_enable drm/mediatek: dsi: add support for .dsi_lp11_notity() drm/bridge: tc358775: fix regulator supply id drm/bridge: tc358775: add crtc modes fixup drm/bridge: tc358775: redefine LV_MX() drm/bridge: tc358775: use regmap instead of open coded access functions drm/bridge: tc358775: remove error message if regulator is missing drm/bridge: tc358775: remove complex vsdelay calculation drm/bridge: tc358775: simplify lvds_link property drm/bridge: tc358775: reformat weird indentation drm/bridge: tc358775: correctly configure LVDS clock drm/bridge: tc358775: split the init code drm/bridge: tc358775: configure PLL depending on the LVDS clock drm/bridge: tc358775: dynamically configure DSI link settings drm/bridge: tc358775: use proper defines to configure LVDS timings drm/bridge: tc358775: move bridge power up/down into functions drm/bridge: tc358775: fix the power-up/down delays drm/bridge: tc358775: fix power-up sequencing drm/bridge: tc358775: use devm_drm_bridge_add() drivers/gpu/drm/bridge/Kconfig | 1 + drivers/gpu/drm/bridge/tc358775.c | 601 ++++++++++++++++++++----------------- drivers/gpu/drm/drm_bridge.c | 16 + drivers/gpu/drm/mediatek/mtk_dsi.c | 8 +- include/drm/drm_bridge.h | 12 + 5 files changed, 355 insertions(+), 283 deletions(-) --- base-commit: 9221b2819b8a4196eecf5476d66201be60fbcf29 change-id: 20240506-tc358775-fix-powerup-53f490043179