From patchwork Wed Jan 10 14:12:25 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 124099 Delivered-To: patch@linaro.org Received: by 10.140.22.227 with SMTP id 90csp5325313qgn; Wed, 10 Jan 2018 06:12:41 -0800 (PST) X-Google-Smtp-Source: ACJfBoudIrzpcasp0Qy1YH60PUpYlkVQ/bic5nXuVpuV+tJyb/7HMhtsXdgCrBnMuIF52MFfC7y0 X-Received: by 10.99.120.203 with SMTP id t194mr14307130pgc.53.1515593561173; Wed, 10 Jan 2018 06:12:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1515593561; cv=none; d=google.com; s=arc-20160816; b=DlDIejAyigNpNppiQzVS+jAtJHd1ZGUhY5nYhrflRTftkK6u8KdLLDyLm/4T4hOort bXrh0PxhsFkFeb6p0KzY2x1t1gNDNVgMisN3oBQKMYuML760O2utTf1AptdjEAY83bGJ +nN8yQ+0AKeB8ubvBkbP47S1OqHE8QUcm6rTVMenTYe4h6mIzXmIgq6HscBdkTu1FiGg 3WQj82ounYMKyoHgTOio5hMonOeGzx9SkmcVyKB12bDRnFFIuiUAw7qvlpE5xjmYJwCr sFJa6+Lo/hHc9EkuL9VsbG+nFxecO+FtYn5UFDjdEKN8rc+1QR20Exyag08PtH/ZVyG0 KguA== 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=a3GxRSLsi92lr9MRWmcSjlgI0kR6yT7vAMfgYN3NRgCuQzkyW6+75A1u96W5skOCOb 3QeuAkP8gf35t5TVVKvZ5zNPbY3NfzcRmqwlEnwfLfFYHigZm+wX83fcNk0NlbrUKm21 7rAcJjlmhCSQ0D30aNFfg27M0hqVmvs4ax98FHLrB6gGaqw0VGzgZeh57JkiK7I6RhOY 7x4Xxc02e57ASx+Y1ro2m1K5c3hoo7tIoCIGQGPpKpFm7OMsa1vDXr9rD7bBIL3gX3cW 4uXz1Ku23yUR7Yurq8Py4WnzczhcRAC9tK1gYec+cvsFVS6iqgtPxIF824Oluhmy42qL VXbw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=OoQGAgQ7; 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 z3si1363420pgr.12.2018.01.10.06.12.40 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 10 Jan 2018 06:12:41 -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=OoQGAgQ7; 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 2B5666E2D4; Wed, 10 Jan 2018 14:12:40 +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 924536E2D4 for ; Wed, 10 Jan 2018 14:12:38 +0000 (UTC) Received: by mail-lf0-x241.google.com with SMTP id u2so5546833lff.7 for ; Wed, 10 Jan 2018 06:12:38 -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=OoQGAgQ7YVdktIkrYSflqviBrncZfB6MWiFW6DNZtBonu/GWLsRZ8tzdqzsCTyE7cl ac6SXImsIm2G8QIKCy+LJ3qxWcjtSDx0ApYKL1XVTVLVXp7yFDWyrSMHn9zWkCm2yGH2 qMIplGddyb2rGzAv2UCNYgspzFu8CaQMJ8/XM= 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=RBAuHXCFaQ21gnajPzIhUW1zwM7GzXTcxh+Lcy6lUmyLAEsd3/iMNap8AcnJ/LDUTK 0A9MoRxFCsUmxyvdn0LDE8DPPOvDKZ0YYsArOBNgSDKpnJAmQFBgoExQhgpLWNK1+G5U yx0Z0cvJeoskWbSXKcCcB2V8M2Ij2uXKqpcnRi8scls1w0RdTJOIe743u7t8SsiTA+zD 8uOXtCqWCA/sHPEMXKfGLziZyl+QP0BFaFXKdYwQ+lRVSUiyIeO1OuSH0E2C36I1vShz C7EjHW1zAHG4cw/1ypqJTcwnY487ZN/OGzpac0UpM8BDHNomxrsFjP78KoRx0psnvtDc vNWQ== X-Gm-Message-State: AKwxytem+6FELinIRTx7JlGSwNTG0unebNKU2u+itt01qo6AtLunoaAo R7Mg+UO59y30PcH8a4tOj/B77A== X-Received: by 10.25.161.19 with SMTP id k19mr9964169lfe.127.1515593556952; Wed, 10 Jan 2018 06:12:36 -0800 (PST) Received: from genomnajs.ideon.se ([85.235.10.227]) by smtp.gmail.com with ESMTPSA id u72sm1019258lfi.64.2018.01.10.06.12.35 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 10 Jan 2018 06:12:36 -0800 (PST) From: Linus Walleij To: Archit Taneja , Andrzej Hajda , Laurent Pinchart Subject: [PATCH 3/4 v6] drm/bridge: Add timing support to dumb VGA DAC Date: Wed, 10 Jan 2018 15:12:25 +0100 Message-Id: <20180110141226.27826-3-linus.walleij@linaro.org> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180110141226.27826-1-linus.walleij@linaro.org> References: <20180110141226.27826-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);