From patchwork Fri Dec 15 12:10:43 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 122078 Delivered-To: patch@linaro.org Received: by 10.140.22.227 with SMTP id 90csp385884qgn; Fri, 15 Dec 2017 04:11:58 -0800 (PST) X-Google-Smtp-Source: ACJfBovzW0ELd5jjC0W8htqMcYb7+I9MWzDfG8VBq0v2IZCrmjdM3Y35MFRtMVqxrj3abmHt8Ldn X-Received: by 10.98.209.8 with SMTP id z8mr13201002pfg.113.1513339918101; Fri, 15 Dec 2017 04:11:58 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1513339918; cv=none; d=google.com; s=arc-20160816; b=ps2A4e2k0E+9r+xRkB+hlAhSOjikQBm2fuVfvtYDMN+oc33FZXlZemGql3a9hXau5H MaoG5mm8Ij8jbyHB3/7XNCAZYuSGts6SbaTOP4IE7znqfglQ8m3Hlf7swmsDtYUKPMnv rakoh2WbigAIJU6QR18D7oYJV/FwtWEswP6JLEe79/nWNfticFzdNe2tnEUbJHKqNU9E 3ycxLKXmXQ51H96bSoaSZ3hE9gSBKxlpKHwCXCZhuDHRBFUz8U83Pr9MvV3RaVVhle39 NZKfbIf9ST/5peKJ2eo7N0H1guiXGtGMSXQTXWXNQw1W11/SNSeKaH+z1xoO47qLZh+m R8YA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:cc:message-id:date:subject:to:from :dkim-signature:delivered-to:arc-authentication-results; bh=+jN4YZMJu8Upd0wd6+9mNT/tSKOAyhIzMQxXUq/Ddnk=; b=JAFoU1KlBhXvvUVoijhAwtkyjKr3tctw5ZMHGopxAjMwxe4oiNPs/HCfaOvJhKIFrO 152EhxxGjpQoN1vUS4BKC1dxXltcQ0z1cFyTlClQ2Rl5xGmN2hpSIlOBFk9W7hsUxZRi ffHv+NQtdOGAYPwvo9DzIRILmsVNmi2SIYhGQWIgq/9153rD4nALN6hYVWkRddSuFbOP MU+Sa0Bmu1k2tiq0Vz8BqP6X1hWDF6QhzzHPEIG1rvBE8TgPSNENRv9JQevG/8Hb3NkE NNsWa/tuAXizxfSfcZ74qx3gw4eEQbv+hXeFhWvulFo70hkACidXvalb/WHY9/lCQFgZ 3rkw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=O8r+lhMM; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from gabe.freedesktop.org (gabe.freedesktop.org. [131.252.210.177]) by mx.google.com with ESMTPS id 1si4839940plw.770.2017.12.15.04.11.57 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 15 Dec 2017 04:11:58 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) client-ip=131.252.210.177; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=O8r+lhMM; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id ADADD6E7DC; Fri, 15 Dec 2017 12:11:26 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-lf0-x244.google.com (mail-lf0-x244.google.com [IPv6:2a00:1450:4010:c07::244]) by gabe.freedesktop.org (Postfix) with ESMTPS id 510B66E7DC for ; Fri, 15 Dec 2017 12:11:23 +0000 (UTC) Received: by mail-lf0-x244.google.com with SMTP id i2so10217447lfe.9 for ; Fri, 15 Dec 2017 04:11:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=hhEX5TUMMMLywGY9j2++GxewDS3XOJTupNDi6m8O/wI=; b=O8r+lhMMY9m15DfrwcBBIhyii6VBAIXImcKoj4m9BRndrngvZ99MfYCnSmgCDGDuw+ oUZ1ZnWLDVfWMxmdYumBFsKAIY1qHfGebAjdK5oVjoayHx7d+j+7vGgmqZhLw37H3kJs hY0r7VU64t4VFvhfckkPB52dvsTcG9TRxvFuY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=hhEX5TUMMMLywGY9j2++GxewDS3XOJTupNDi6m8O/wI=; b=amQwGw3YnaaI3ijbESybBhLiVg29fLC/tt8iCpcuc9ptmzJVhcklxd3i+Zn9JX8qC4 o7+Vctl1DT2D1ccqtjahwVFpTQFx53fxOydB2cEx2+IZGEs86wY8QzAEl0RDl7VtLabH NEcXZNGL6ViMQjsnp1DAEAUzqImuyjQST5uMaNlaYHMon1HgVoa/2oQtvp/YSTCY6iV3 574Z3yZKAIFogb3G1ZZMHJTFZCYbnkkfE1GG+PAoWL5QtWCucOmubMGk5v10I9ma6Gi4 RW99z4SPK9RhGaylHpPOU11t7Sfu36Ym577Uz8c2spVWUIqt9oP7J2jbL9E90RWiJBfh Y9nQ== X-Gm-Message-State: AKGB3mLoGtfmelAGuMRVEW0EMh7vVKu4UxNoxptEDIitmX3v+9DwSS9c TD3T/X/FKV2vbjoNPbVXovZWYg== X-Received: by 10.25.161.132 with SMTP id k126mr5369214lfe.110.1513339881060; Fri, 15 Dec 2017 04:11:21 -0800 (PST) Received: from localhost.localdomain ([90.229.2.39]) by smtp.gmail.com with ESMTPSA id z81sm1289345lff.80.2017.12.15.04.11.13 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 15 Dec 2017 04:11:20 -0800 (PST) From: Linus Walleij To: Archit Taneja , Andrzej Hajda , Laurent Pinchart Subject: [PATCH 0/4 v5] Support bridge timings Date: Fri, 15 Dec 2017 13:10:43 +0100 Message-Id: <20171215121047.3650-1-linus.walleij@linaro.org> X-Mailer: git-send-email 2.14.3 Cc: linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" This patch set is in response to Laurent's mail: https://www.spinics.net/lists/dri-devel/msg155618.html So in summary: - The connector is apparently not the right abstraction to carry the detailed timings specification between DRI drivers and bridge drivers. - Instead put detailed timing data into the bridge itself as an optional information pointer. - Add fields to specify triggering edge on the clock, setup and hold time for the bridge. - Augment the dumb VGA driver with this data, supporting a few ADV and TI variants. - Augment the PL111 driver to use this data to figure out if it needs to clock out display data on the negative edge rather than the positive edge because the current clocking out on the positive edge obviously partly misses the setup->hold window, as can be observed in annoying green flicker. Linus Walleij (4): drm/bridge: Add bindings for TI THS8134 drm/bridge: Provide a way to embed timing info in bridges drm/bridge: Add timing support to dumb VGA DAC drm/pl111: Support handling bridge timings .../bridge/{ti,ths8135.txt => ti,ths813x.txt} | 13 +++-- drivers/gpu/drm/bridge/dumb-vga-dac.c | 61 ++++++++++++++++++++-- drivers/gpu/drm/pl111/Kconfig | 1 + drivers/gpu/drm/pl111/pl111_display.c | 35 +++++++++++-- drivers/gpu/drm/pl111/pl111_drv.c | 20 +++---- include/drm/drm_bridge.h | 20 +++++++ 6 files changed, 130 insertions(+), 20 deletions(-) rename Documentation/devicetree/bindings/display/bridge/{ti,ths8135.txt => ti,ths813x.txt} (69%)