From patchwork Fri Jan 12 07:48:53 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 124338 Delivered-To: patch@linaro.org Received: by 10.140.22.227 with SMTP id 90csp2041608qgn; Fri, 12 Jan 2018 05:08:43 -0800 (PST) X-Google-Smtp-Source: ACJfBovo6xoifi+/0lJaW7I/Zt9lbA4YGaIPkKuRNRmthTX+jUUDXKbOUiliLFUbKyKrO09cK6fG X-Received: by 10.84.236.13 with SMTP id q13mr14937965plk.445.1515762523176; Fri, 12 Jan 2018 05:08:43 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1515762523; cv=none; d=google.com; s=arc-20160816; b=iKzTeOLEPrdVc2v0lLCDcVsSYbhirLGvN9Rbuhydg9EmgumpvTdK61glDVGa41Cs0b Uf1VSuPVFweqYeGS4WIDXYtjNcjfHx2ZVMEos42UHa0RT4FpqRIB5cl6S9lmReTcX/5T YxJHg2uNnkw8/uojprNKOw/0bVprjtPOfUQMK4y8wuw6oG8K9ZaZ4iyogGagy/wWqUbv mrKAhpoGwe2eRjoWzilFdxtV/tCKvKcUqlmPJ/TdeqzExTyoGRn69OGCZakzGIi8BzE+ MO3vpvi/hWYkqAjxqZUJr1QEL4QaTeiPKXi6SbH7FP2QFvnqrVRyQRahyqxgGcZHTE8F efAA== 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:references:in-reply-to:message-id:date :subject:to:from:dkim-signature:delivered-to :arc-authentication-results; bh=VxN3C+koALL1AxJmbGZcuwLfROrqoDVyV9QvZEyhC4o=; b=srL1044scDLz/c5HJTNevOdTdVQBXksVFXrRP8lK3NJQdbu3HshpYozGW6Nm4tSuRQ f1ueIuY2aVcHq4yxB3iQqJmFbK6073Sl+6noDxlIhJvT1FoGaQxPBvUzBXWl+XjIFt13 06rfhJr5YeEmCvTpfEeF8ljlz7OmdT5POQ6jB1ctg71tSk1FrSn+Obyt2Xx11QDl3LVl XyrkTZD6p0mGMkl5xcGp5LuU3I9bb3RsNYIUuM+JgwuAc3qPZ/yybCSsuUTxw/Tog1yZ 3QdBoA+QHmN0dT9Cytzm/U+zlY894JKYWsCZnO0vyYjE5LrXyBqTLNmwgFpXtZjLOW4t NnGg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=SAxUja+m; 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 u6si8191676pls.598.2018.01.12.05.08.42 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 12 Jan 2018 05:08:43 -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=SAxUja+m; 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 01A806E875; Fri, 12 Jan 2018 13:08:12 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-lf0-x241.google.com (mail-lf0-x241.google.com [IPv6:2a00:1450:4010:c07::241]) by gabe.freedesktop.org (Postfix) with ESMTPS id C90356E876 for ; Fri, 12 Jan 2018 13:08:10 +0000 (UTC) Received: by mail-lf0-x241.google.com with SMTP id j143so5897446lfg.0 for ; Fri, 12 Jan 2018 05:08:10 -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:in-reply-to:references; bh=z2JcY7odEzBpM5y/UJZlDlPtURq803oKvAZHv3wupm4=; b=SAxUja+mQxZ72OPkOtig/NgcW5qK2X/pwtFzXbtNbblFrSwS7JGe8Z2Y99gVEQgNgT uYSaTj7Xc2s1C/uCs5mUnMiMJz+yDyg6ZgW5K423nmYt4NXVA3TQ3zteA9wwOoQNIFlJ Kbdu1knWSylHM29TCrEldP5xhdNiVDNQ2xAeU= 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:in-reply-to :references; bh=z2JcY7odEzBpM5y/UJZlDlPtURq803oKvAZHv3wupm4=; b=g1Xy1t5h4NOV2PJMqAN+MpWKdFgf/c101fX+ruHAR1VgqDUez+K+0v5zZQaJhz+uL5 fR/MfAbZFoSTlqu8aQZf8F0plFP6TNT+6aKYJZiCl9Lu+Kqz0nrITIqkjzcHRkN6QavR w1HNP1TdtpGirk1fKqW3euAHD3mZFN7dJkMnN/BNNoNZB5+wm03gzuDkiMu4mQSAtL/W AKmRwI7gd/EtRAsc0wG6TEO6byXevsx0lY/MQ6HQfQUhCQgdb14MGwCNuyVLDJquAYup EuWsY2ggt/is1zN4O1/jRekdV+5+yM5DF9Y1bxp0SzJoalUmjAVj7M9jik+kwh2/POWZ eqqA== X-Gm-Message-State: AKwxytcQxEf4IvW8MefmYYp8uG+twKZ8Pa6NgodC3SaMZ3Gx40aE5sO9 4ktlpw6n/uOnkz+uQ6ABalE4zp36STM= X-Received: by 10.25.225.215 with SMTP id l84mr7260680lfk.110.1515743608270; Thu, 11 Jan 2018 23:53:28 -0800 (PST) Received: from localhost.localdomain (c-cb7471d5.014-348-6c756e10.cust.bredbandsbolaget.se. [213.113.116.203]) by smtp.gmail.com with ESMTPSA id a69sm4115039ljf.54.2018.01.11.23.53.27 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 11 Jan 2018 23:53:27 -0800 (PST) From: Linus Walleij To: Archit Taneja , Andrzej Hajda , Laurent Pinchart Subject: [PATCH 3/4 v7] drm/bridge: Add timing support to dumb VGA DAC Date: Fri, 12 Jan 2018 08:48:53 +0100 Message-Id: <20180112074854.9560-3-linus.walleij@linaro.org> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180112074854.9560-1-linus.walleij@linaro.org> References: <20180112074854.9560-1-linus.walleij@linaro.org> Cc: Bartosz Golaszewski , dri-devel@lists.freedesktop.org, Maxime Ripard , linux-arm-kernel@lists.infradead.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 extends the dumb VGA DAC bridge to handle the THS8134A and THS8134B VGA DACs in addition to those already handled. We assign the proper timing data to the pointer inside the bridge struct so display controllers that need to align their timings to the bridge can pick it up and work from there. Cc: Bartosz Golaszewski Cc: Maxime Ripard Reviewed-by: Laurent Pinchart Signed-off-by: Linus Walleij --- ChangeLog v5->v6: - Use DRM_BUS_FLAG_PIXDATA_[POS|NEG]EDGE to indicate the sampling edge of the clock signal. - Skip intermediate variable for timings. - Leave timings as NULL for really dumb VGA DACs. - Collect Laurent's Review tag. ChangeLog v4->v5: - Rewrite the support using the new concept of defining fine-granular sampling (setup+hold) timing definitions stored in the bridge timings struct. ChangeLog v3->v4: - Actually have the code syntactically correct and compiling :( (Kconfig mistake.) (...) AS usr/initramfs_data.o AR usr/built-in.o CC drivers/gpu/drm/bridge/dumb-vga-dac.o AR drivers/gpu/drm/bridge/built-in.o AR drivers/gpu/drm/built-in.o AR drivers/gpu/built-in.o AR drivers/built-in.o (...) ChangeLog v2->v3: - Move const specifier. - Cut one line of code assigning bus flags. - Preserve the "ti,ths8135" compatible for elder device trees. ChangeLog v1->v2: - Alphabetize includes - Use a u32 with the bus polarity flags and just encode the polarity using the DRM define directly. - Rename vendor_data to vendor_info. - Simplify assignment of the flag as it is just a simple u32 now. - Probe all TI variants on the "ti,ths813x" wildcard for now, we only need to know that the device is in this family to set the clock edge flag right. --- drivers/gpu/drm/bridge/dumb-vga-dac.c | 59 +++++++++++++++++++++++++++++++++-- 1 file changed, 56 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c b/drivers/gpu/drm/bridge/dumb-vga-dac.c index de5e7dee7ad6..498d5948d1a8 100644 --- a/drivers/gpu/drm/bridge/dumb-vga-dac.c +++ b/drivers/gpu/drm/bridge/dumb-vga-dac.c @@ -11,6 +11,7 @@ */ #include +#include #include #include @@ -204,6 +205,7 @@ static int dumb_vga_probe(struct platform_device *pdev) vga->bridge.funcs = &dumb_vga_bridge_funcs; vga->bridge.of_node = pdev->dev.of_node; + vga->bridge.timings = of_device_get_match_data(&pdev->dev); drm_bridge_add(&vga->bridge); @@ -222,10 +224,61 @@ static int dumb_vga_remove(struct platform_device *pdev) return 0; } +/* + * We assume the ADV7123 DAC is the "default" for historical reasons + * Information taken from the ADV7123 datasheet, revision D. + * NOTE: the ADV7123EP seems to have other timings and need a new timings + * set if used. + */ +static const struct drm_bridge_timings default_dac_timings = { + /* Timing specifications, datasheet page 7 */ + .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE, + .setup_time_ps = 500, + .hold_time_ps = 1500, +}; + +/* + * Information taken from the THS8134, THS8134A, THS8134B datasheet named + * "SLVS205D", dated May 1990, revised March 2000. + */ +static const struct drm_bridge_timings ti_ths8134_dac_timings = { + /* From timing diagram, datasheet page 9 */ + .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE, + /* From datasheet, page 12 */ + .setup_time_ps = 3000, + /* I guess this means latched input */ + .hold_time_ps = 0, +}; + +/* + * Information taken from the THS8135 datasheet named "SLAS343B", dated + * May 2001, revised April 2013. + */ +static const struct drm_bridge_timings ti_ths8135_dac_timings = { + /* From timing diagram, datasheet page 14 */ + .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE, + /* From datasheet, page 16 */ + .setup_time_ps = 2000, + .hold_time_ps = 500, +}; + static const struct of_device_id dumb_vga_match[] = { - { .compatible = "dumb-vga-dac" }, - { .compatible = "adi,adv7123" }, - { .compatible = "ti,ths8135" }, + { + .compatible = "dumb-vga-dac", + .data = NULL, + }, + { + .compatible = "adi,adv7123", + .data = &default_dac_timings, + }, + { + .compatible = "ti,ths8135", + .data = &ti_ths8135_dac_timings, + }, + { + .compatible = "ti,ths8134", + .data = &ti_ths8134_dac_timings, + }, {}, }; MODULE_DEVICE_TABLE(of, dumb_vga_match);