[{"id":1776648,"web_url":"http://patchwork.ozlabs.org/comment/1776648/","msgid":"<20170927215638.tjbllqq2twdvuivj@rob-hp-laptop>","list_archive_url":null,"date":"2017-09-27T21:56:38","subject":"Re: [RESEND RFC PATCH 4/7] dt-bindings: Document Allwinner DWC HDMI\n\tTX node","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Wed, Sep 20, 2017 at 10:01:21PM +0200, Jernej Skrabec wrote:\n> Add documentation about Allwinner DWC HDMI TX node, found in H3 SoC.\n> \n> Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net>\n> ---\n>  .../bindings/display/sunxi/sun4i-drm.txt           | 158 ++++++++++++++++++++-\n>  1 file changed, 157 insertions(+), 1 deletion(-)\n> \n> diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt\n> index 92512953943e..cb6aee5c486f 100644\n> --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt\n> +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt\n> @@ -60,6 +60,40 @@ Required properties:\n>      first port should be the input endpoint. The second should be the\n>      output, usually to an HDMI connector.\n>  \n> +DWC HDMI TX Encoder\n> +-----------------------------\n> +\n> +The HDMI transmitter is a Synopsys DesignWare HDMI 1.4 TX controller IP\n> +with Allwinner's own PHY IP. It supports audio and video outputs and CEC.\n> +\n> +These DT bindings follow the Synopsys DWC HDMI TX bindings defined in\n> +Documentation/devicetree/bindings/display/bridge/dw_hdmi.txt with the\n> +following device-specific properties.\n> +\n> +Required properties:\n> +\n> +  - compatible: value must be one of:\n> +    * \"allwinner,sun8i-h3-dw-hdmi\"\n> +  - reg: two pairs of base address and size of memory-mapped region, first\n> +    for controller and second for PHY\n> +    registers.\n> +  - reg-io-width: See dw_hdmi.txt. Shall be 1.\n> +  - interrupts: HDMI interrupt number\n> +  - clocks: phandles to the clocks feeding the HDMI encoder\n> +    * iahb: the HDMI interface clock\n> +    * isfr: the HDMI module clock\n> +    * ddc: the HDMI ddc clock\n> +  - clock-names: the clock names mentioned above\n> +  - resets: phandles to the reset controllers driving the encoder\n> +    * hdmi: the reset line for the HDMI\n> +    * ddc: the reset line for the DDC\n> +  - reset-names: the reset names mentioned above\n> +\n> +  - ports: A ports node with endpoint definitions as defined in\n> +    Documentation/devicetree/bindings/media/video-interfaces.txt. The\n> +    first port should be the input endpoint. The second should be the\n> +    output, usually to an HDMI connector.\n> +\n>  TV Encoder\n>  ----------\n>  \n> @@ -255,7 +289,7 @@ Required properties:\n>    - allwinner,pipelines: list of phandle to the display engine\n>      frontends (DE 1.0) or mixers (DE 2.0) available.\n>  \n> -Example:\n> +Example 1:\n>  \n>  panel: panel {\n>  \tcompatible = \"olimex,lcd-olinuxino-43-ts\";\n> @@ -455,3 +489,125 @@ display-engine {\n>  \tcompatible = \"allwinner,sun5i-a13-display-engine\";\n>  \tallwinner,pipelines = <&fe0>;\n>  };\n> +\n> +Example 2:\n> +\n> +connector {\n> +\tcompatible = \"hdmi-connector\";\n> +\ttype = \"a\";\n> +\n> +\tport {\n> +\t\thdmi_con_in: endpoint {\n> +\t\t\tremote-endpoint = <&hdmi_out_con>;\n> +\t\t};\n> +\t};\n> +};\n> +\n> +de: display-engine {\n> +\tcompatible = \"allwinner,sun8i-h3-display-engine\";\n> +\tallwinner,pipelines = <&mixer0>;\n> +};\n> +\n> +hdmi: hdmi@1ee0000 {\n> +\tcompatible = \"allwinner,h3-dw-hdmi\";\n> +\treg = <0x01ee0000 0x10000>,\n> +\t      <0x01ef0000 0x10000>;\n> +\treg-io-width = <1>;\n> +\tinterrupts = <GIC_SPI 88 IRQ_TYPE_LEVEL_HIGH>;\n> +\tclocks = <&ccu CLK_BUS_HDMI>, <&ccu CLK_HDMI>,\n> +\t\t <&ccu CLK_HDMI_DDC>;\n> +\tclock-names = \"iahb\", \"isfr\", \"ddc\";\n> +\tresets = <&ccu RST_BUS_HDMI0>, <&ccu RST_BUS_HDMI1>;\n> +\treset-names = \"hdmi\", \"ddc\";\n> +\n> +\tports {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <0>;\n> +\n> +\t\thdmi_in: port@0 {\n> +\t\t\t#address-cells = <1>;\n> +\t\t\t#size-cells = <0>;\n> +\t\t\treg = <0>;\n> +\n> +\t\t\thdmi_in_tcon0: endpoint@0 {\n> +\t\t\t\treg = <0>;\n\nYou don't need reg when there's only one.\n\nOtherwise,\n\nAcked-by: Rob Herring <robh@kernel.org>\n--\nTo unsubscribe from this list: send the line \"unsubscribe devicetree\" in\nthe body of a message to majordomo@vger.kernel.org\nMore majordomo info at  http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"<devicetree-owner@vger.kernel.org>","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=<UNKNOWN>)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3y2Wr75hkQz9t67\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tThu, 28 Sep 2017 07:56:55 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1752065AbdI0V4m (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tWed, 27 Sep 2017 17:56:42 -0400","from mail-pg0-f68.google.com ([74.125.83.68]:33935 \"EHLO\n\tmail-pg0-f68.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1752011AbdI0V4l (ORCPT\n\t<rfc822; devicetree@vger.kernel.org>); Wed, 27 Sep 2017 17:56:41 -0400","by mail-pg0-f68.google.com with SMTP id u18so10493367pgo.1;\n\tWed, 27 Sep 2017 14:56:40 -0700 (PDT)","from localhost ([70.35.39.2]) by smtp.gmail.com with ESMTPSA id\n\tu73sm21833715pfi.58.2017.09.27.14.56.39\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tWed, 27 Sep 2017 14:56:39 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=nEGIR8jF0rUAHgGKmGvh9ttTyjfjYBmfWHgFoK3LKvY=;\n\tb=EikyjYJ5JNvXbm+B358xkiqa1CmaJPkoKqhoR2SlO0MEAnJkmYMCg30L3UrvmTKGWP\n\tFA7h24yDiqPod1r69O6sOAhHZoqeaCLxnxFfzgbigq6mU8KVWyykbJm4Sa4Ue3CXSOtO\n\t10Eu7apG5VsVS/+guDcikbMGo4KV9gpIMYbtVPDMrBLknoP3VZs77RC+ZOa/UM3IZ9fP\n\tIrRuD1E1jM7/jB+QzM6tDduD3JMwhAAbDY3o0LeaA7/7CsBCgMsRiGRmKsxFDpqMC/wA\n\tBgEE2yypoxlcj1GDEg39yyCVxjLX5pD+Qn/WLpiPiF/2cp8bE8urBiIm/3f0zrps0+J9\n\tH4rQ==","X-Gm-Message-State":"AHPjjUgKM2LcGR5iEM+6kWXpDq6BAlsPJ/A25iYcMfhwEfLByJFdPsOp\n\tDcKSYRHzwW0aDk8UyFSPVg==","X-Google-Smtp-Source":"AOwi7QByo9s4fTIR8BoJWuNg94Yxu7oSVWGT1fbb9gFnaEF3LumlkRvYmHP8wezztT1awFRTLiUcUg==","X-Received":"by 10.159.247.140 with SMTP id e12mr2328374pls.340.1506549400392;\n\tWed, 27 Sep 2017 14:56:40 -0700 (PDT)","Date":"Wed, 27 Sep 2017 16:56:38 -0500","From":"Rob Herring <robh@kernel.org>","To":"Jernej Skrabec <jernej.skrabec@siol.net>","Cc":"maxime.ripard@free-electrons.com, wens@csie.org,\n\tLaurent.pinchart@ideasonboard.com, hans.verkuil@cisco.com,\n\tnarmstrong@baylibre.com, dri-devel@lists.freedesktop.org,\n\tdevicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\tlinux-kernel@vger.kernel.org, linux-clk@vger.kernel.org,\n\ticenowy@aosc.io, linux-sunxi@googlegroups.com,\n\tlinux-media@vger.kernel.org","Subject":"Re: [RESEND RFC PATCH 4/7] dt-bindings: Document Allwinner DWC HDMI\n\tTX node","Message-ID":"<20170927215638.tjbllqq2twdvuivj@rob-hp-laptop>","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>\n\t<20170920200124.20457-5-jernej.skrabec@siol.net>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170920200124.20457-5-jernej.skrabec@siol.net>","User-Agent":"NeoMutt/20170113 (1.7.2)","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<devicetree.vger.kernel.org>","X-Mailing-List":"devicetree@vger.kernel.org"}}]