From patchwork Mon Nov 4 10:34:23 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Torsten Duwe X-Patchwork-Id: 11226329 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id B047913A4 for ; Mon, 4 Nov 2019 19:35:28 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 9860E2089C for ; Mon, 4 Nov 2019 19:35:28 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9860E2089C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id D07F36E838; Mon, 4 Nov 2019 19:35:11 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from verein.lst.de (verein.lst.de [213.95.11.211]) by gabe.freedesktop.org (Postfix) with ESMTPS id 32B1C6E2D3 for ; Mon, 4 Nov 2019 11:04:06 +0000 (UTC) Received: by verein.lst.de (Postfix, from userid 2005) id F319F68BE1; Mon, 4 Nov 2019 12:04:00 +0100 (CET) From: Torsten Duwe Date: Mon, 4 Nov 2019 11:34:23 +0100 Subject: [PATCH v5 0/7] Add anx6345 DP/eDP bridge for Olimex Teres-I To: Maxime Ripard , Chen-Yu Tsai , Rob Herring , Mark Rutland , Thierry Reding , David Airlie , Daniel Vetter , Andrzej Hajda , Laurent Pinchart , Icenowy Zheng , Sean Paul , Vasily Khoruzhick , Harald Geyer , Greg Kroah-Hartman , Thomas Gleixner Message-Id: <20191104110400.F319F68BE1@verein.lst.de> X-Mailman-Approved-At: Mon, 04 Nov 2019 19:35:10 +0000 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" ANX6345 LVTTL->eDP video bridge, driver with device tree bindings. Changes since v4: * enforce DT ports to be nodes and forbid additionalProperties. Changes since v3: * converted binding schema file to json-schema ("YAML") It now validates (itself and the dts) like a charm ;-) Input port0 is mandatory, output port1 is optional. * Enric Balletbo i Serra waived analogix-anx6345 module ownership to icenowy Since they both agree, I won't interfere. Changes from v2: * use SPDX-IDs throughout * removed the panel output again, as it was not what Maxime had in mind. At least the Teres-I does very well without. * binding clarifications and cosmetic changes as suggested by Andrzej Changes from v1: * fixed up copyright information. Most code changes are only moves and thus retain copyright and module ownership. Even the new analogix-anx6345.c originates from the old 1495-line analogix-anx78xx.c, with 306 insertions and 987 deletions (ignoring the trivial anx78xx -> anx6345 replacements) 306 new vs. 508 old... * fixed all minor formatting issues brought up * merged previously separate new analogix_dp_i2c module into existing analogix_dp * split additional defines into a preparatory patch * renamed the factored-out common functions anx_aux_* -> anx_dp_aux_*, because anx_...aux_transfer was exported globally. Besides, it is now GPL-only exported. * moved chip ID read into a separate function. * keep the chip powered after a successful probe. (There's a good chance that this is the only display during boot!) * updated the binding document: LVTTL input is now required, only the output side description is optional. Laurent: I have also looked into the drm_panel_bridge infrastructure, but it's not that trivial to convert these drivers to it. Changes from the respective previous versions: * the reset polarity is corrected in DT and the driver; things should be clearer now. * as requested, add a panel (the known innolux,n116bge) and connect the ports. * renamed dvdd?? to *-supply to match the established scheme * trivial update to the #include list, to make it compile in 5.2 Icenowy Zheng (4): drm/bridge: move ANA78xx driver to analogix subdirectory drm/bridge: split some definitions of ANX78xx to dedicated headers drm/bridge: extract some Analogix I2C DP common code drm/bridge: Add Analogix anx6345 support Torsten Duwe (3): drm/bridge: Prepare Analogix anx6345 support dt-bindings: Add ANX6345 DP/eDP transmitter binding arm64: dts: allwinner: a64: enable ANX6345 bridge on Teres-I .../bindings/display/bridge/anx6345.yaml | 92 +++ .../boot/dts/allwinner/sun50i-a64-teres-i.dts | 45 +- drivers/gpu/drm/bridge/Kconfig | 10 - drivers/gpu/drm/bridge/Makefile | 4 +- drivers/gpu/drm/bridge/analogix-anx78xx.h | 710 ------------------ drivers/gpu/drm/bridge/analogix/Kconfig | 22 + drivers/gpu/drm/bridge/analogix/Makefile | 4 +- drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 793 +++++++++++++++++++++ .../drm/bridge/{ => analogix}/analogix-anx78xx.c | 146 +--- drivers/gpu/drm/bridge/analogix/analogix-anx78xx.h | 255 +++++++ .../gpu/drm/bridge/analogix/analogix-i2c-dptx.c | 165 +++++ .../gpu/drm/bridge/analogix/analogix-i2c-dptx.h | 258 +++++++ .../drm/bridge/analogix/analogix-i2c-txcommon.h | 236 ++++++ 13 files changed, 1868 insertions(+), 872 deletions(-) create mode 100644 Documentation/devicetree/bindings/display/bridge/anx6345.yaml delete mode 100644 drivers/gpu/drm/bridge/analogix-anx78xx.h create mode 100644 drivers/gpu/drm/bridge/analogix/analogix-anx6345.c rename drivers/gpu/drm/bridge/{ => analogix}/analogix-anx78xx.c (90%) create mode 100644 drivers/gpu/drm/bridge/analogix/analogix-anx78xx.h create mode 100644 drivers/gpu/drm/bridge/analogix/analogix-i2c-dptx.c create mode 100644 drivers/gpu/drm/bridge/analogix/analogix-i2c-dptx.h create mode 100644 drivers/gpu/drm/bridge/analogix/analogix-i2c-txcommon.h Signed-off-by: Icenowy Zheng Signed-off-by: Vasily Khoruzhick Signed-off-by: Torsten Duwe Reviewed-by: Andrzej Hajda