Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp7685245ybi; Mon, 22 Jul 2019 18:31:52 -0700 (PDT) X-Google-Smtp-Source: APXvYqzWv6hC+MEFprnWpn0w74WU1CMt+7l0b6yoEbFR8yKgZrFb/+lTCiMTQTK7bgyMfGfuZsAm X-Received: by 2002:a62:b411:: with SMTP id h17mr2958273pfn.99.1563845512836; Mon, 22 Jul 2019 18:31:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1563845512; cv=none; d=google.com; s=arc-20160816; b=J71y6in2niRBdRJvvuwAvN6YFvJG33tqo1SGpufyTtNZ75nYBvNEo5BrsQUuiAoub0 kNqUH4d8k9gN87O0X9mAwO3Fi5JGt4fuCf9RnmpdWRe51sjVSc4XVgwMly6vm91m9RGn KXPIfG83VQ2mS7t/BFEgMVimH8iJw5iGHxY6FfXqMZQjeZB9buA4+XzQfsrp1Ufgw7S2 UKZgNjzg2rS8j8NXCn5VeDYEPKreAOZ1ecV2C1RKptzfqmpD6RRzU6LBAYSLbfs/yuWk ehUp0x1HKsJfy4o/zHBh5tmWZUUBXHM+1Mf6ZEqe4hg3V6iwGF010vkGLCbidl8dzCaR 3UoA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=aC1MLxLbjDg5tOzPiyVil8Be4/KhIGolhfOdnXDjtBE=; b=Tc+i6fl44ypEaE4Ai2G71VpylhZcQSnSvSLbKqIS4OSgjtxKiq/5qxOXFk34NBWrru f2mghHna+GX2f8P+bIkU93f/GkOnlu3IU/nkht0Bi7O+TXxMIis1BKJflpcNVCsztbB9 8BaK4biEGYuvIA8zfN9WeqTk5m8v4oRPM0yp3jg0RndgFl6l7FTtoL/LVgTuubrAzoEZ nhtV1MGggQ+MEUjlG+jUlyLFsUZnnJR0RrEefadXol1ze9b92rnX6VTjjcnhgkiE+JMC +uTlkVBryIY5JrXaqAya95/69BfjEyaLdlLGwsvlPxo77xfXSy1KtDghoBC8PIV6nxkz qnZA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b="a/SPgKgt"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h16si4271255pgl.318.2019.07.22.18.31.36; Mon, 22 Jul 2019 18:31:52 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b="a/SPgKgt"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730224AbfGVSrn (ORCPT + 99 others); Mon, 22 Jul 2019 14:47:43 -0400 Received: from mail-ot1-f65.google.com ([209.85.210.65]:43532 "EHLO mail-ot1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727821AbfGVSrn (ORCPT ); Mon, 22 Jul 2019 14:47:43 -0400 Received: by mail-ot1-f65.google.com with SMTP id j11so17066038otp.10; Mon, 22 Jul 2019 11:47:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=aC1MLxLbjDg5tOzPiyVil8Be4/KhIGolhfOdnXDjtBE=; b=a/SPgKgtMHEdYvP1ujLe27UWsqf7cBVVME1tq3pKmJc6b83inBBubiAdSQckX+VKbG NMiOeacyFUgN0Qt19RTEwwPT60/w8OyT7FsZutliKE1iXT7iAB5PXEBx7BYKXiNZfH8q MF7lGbAelaec9OuYlnC4Mp9B7PfHQpJ7fUA99cfrk5ijZizuyFAhxRAvDekrMOOz3Gen 7qV5snjmYlHafHWT2AQK/LeRCwUiqhg0Vlar9IN/RRt8Ktr5nA+9wRIxopDqu+I+xDwV Jc3FCInW4B1XPn3GIvYd+4BICjqmE09mOcrVpVYrSXl+ID0WjJJqVUA2r0N2FDhpOdQS 2oGw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=aC1MLxLbjDg5tOzPiyVil8Be4/KhIGolhfOdnXDjtBE=; b=emjPx4TfJc/f8pK7KxpFMzoCkuzPJKDJwAgJuf1rX2wuq44WdHqSH5NDgR/sWTEgk0 yVEppwWY7cvsyunb4VQ2l26Hpjq2NBr/xAzb9l18oAYg4tfyw/SDpURPm8c+00BtqNU1 26pjtyLTHpMxKpAD+C4CMADedcjj2b/6IKrdjfccn0poh4dVrd0PIqgZSW4ANtI0cxMR RnQ4VFoz2rWfpTkuoNPT8ksJZxCfxLARBD8otNQJvy9EMBE/8TDcTNeV2XGi7fTXyViW 6OZDPiaPPnQK3R0scFi/Tpk91feYvcDQD8yhA1Mys6XP3/4Rk0U5Wk1hd9qPEaa3gznn MD4Q== X-Gm-Message-State: APjAAAVK3Kznw9mG4OI7ULmSAL+xoN2BWJFrQbdSBFMFRzuzeLzGxpfp r2PyMnXulMi+kkau2Iwf23h9tCDP2q3GrzJPwo8= X-Received: by 2002:a9d:4109:: with SMTP id o9mr29072586ote.353.1563821262309; Mon, 22 Jul 2019 11:47:42 -0700 (PDT) MIME-Version: 1.0 References: <20190722150414.9F97668B20@verein.lst.de> In-Reply-To: <20190722150414.9F97668B20@verein.lst.de> From: Vasily Khoruzhick Date: Mon, 22 Jul 2019 11:47:33 -0700 Message-ID: Subject: Re: [PATCH v2 0/7] Add anx6345 DP/eDP bridge for Olimex Teres-I To: Torsten Duwe Cc: Maxime Ripard , Chen-Yu Tsai , Rob Herring , Mark Rutland , Thierry Reding , David Airlie , Daniel Vetter , Andrzej Hajda , Laurent Pinchart , Icenowy Zheng , Sean Paul , Harald Geyer , Greg Kroah-Hartman , Thomas Gleixner , dri-devel , devicetree , arm-linux , linux-kernel , <"Bcc:duwe"@lst.de> Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 22, 2019 at 8:04 AM Torsten Duwe wrote: > > ANX6345 LVTTL->eDP video bridge, driver with device tree bindings. > > 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. No connector spec, no "quirks"[1], > just plain EDID at work. You still need a panel to put backlight in there. Otherwise backlight will stay on when display is turned off. > > * 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 > > -------------- > [1] I hesitate to associate information about a connected panel with that term. > But should it be neccessary for maximum power saving (e.g. pinebooks), > it would be good to have something here, regardless of nomenclature. > > Torsten