From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 60B42C54FD0 for ; Mon, 27 Apr 2020 09:09:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3D6822064C for ; Mon, 27 Apr 2020 09:09:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726434AbgD0JJF (ORCPT ); Mon, 27 Apr 2020 05:09:05 -0400 Received: from asavdk4.altibox.net ([109.247.116.15]:43156 "EHLO asavdk4.altibox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726183AbgD0JJF (ORCPT ); Mon, 27 Apr 2020 05:09:05 -0400 Received: from ravnborg.org (unknown [158.248.194.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by asavdk4.altibox.net (Postfix) with ESMTPS id 1762580618; Mon, 27 Apr 2020 11:09:01 +0200 (CEST) Date: Mon, 27 Apr 2020 11:09:00 +0200 From: Sam Ravnborg To: Ricardo =?iso-8859-1?Q?Ca=F1uelo?= Cc: devicetree@vger.kernel.org, enric.balletbo@collabora.com, robh+dt@kernel.org, kernel@collabora.com, dri-devel@lists.freedesktop.org Subject: Re: [PATCH v2] dt-bindings: display: anx7814.txt: convert to yaml Message-ID: <20200427090900.GA25058@ravnborg.org> References: <20200427084658.27962-1-ricardo.canuelo@collabora.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20200427084658.27962-1-ricardo.canuelo@collabora.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CMAE-Score: 0 X-CMAE-Analysis: v=2.3 cv=MOBOZvRl c=1 sm=1 tr=0 a=UWs3HLbX/2nnQ3s7vZ42gw==:117 a=UWs3HLbX/2nnQ3s7vZ42gw==:17 a=8nJEP1OIZ-IA:10 a=QX4gbG5DAAAA:8 a=7gkXJVJtAAAA:8 a=gEfo2CItAAAA:8 a=e5mUnYsNAAAA:8 a=9PyzFoNYW9ryUBNcHcwA:9 a=Ngot1CxQ-7BNPHMX:21 a=ceCSG8EcjXA1U1zs:21 a=wPNLvfGTeEIA:10 a=AbAUZ8qAyYyZVLSsDulk:22 a=E9Po1WZjFZOl8hwRPBS3:22 a=sptkURWiP4Gy88Gu7hUp:22 a=Vxmtnl_E_bksehYqCbjh:22 Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Ricardo. On Mon, Apr 27, 2020 at 10:46:58AM +0200, Ricardo Cañuelo wrote: > This converts the Analogix ANX7814 bridge DT binding to yaml. Port > definitions and descriptions were expanded, apart from that it's a > direct translation from the original binding. > > Signed-off-by: Ricardo Cañuelo One comment below, with that fixed: Acked-by: Sam Ravnborg > --- > Changes in v2 (suggested by Enric Balletbo): > > - File name change: use full compatible string. > - Binding description removed. > - #address-cells and #size-cells properties removed from ports node. > - Example node renamed: anx7814 -> bridge. > > Tested with: > make dt_binding_check ARCH=arm64 DT_SCHEMA_FILES=<.../analogix,anx7814.yaml> > make dtbs_check ARCH=arm64 DT_SCHEMA_FILES=<.../analogix,anx7814.yaml> > > .../display/bridge/analogix,anx7814.yaml | 124 ++++++++++++++++++ > .../bindings/display/bridge/anx7814.txt | 42 ------ > 2 files changed, 124 insertions(+), 42 deletions(-) > create mode 100644 Documentation/devicetree/bindings/display/bridge/analogix,anx7814.yaml > delete mode 100644 Documentation/devicetree/bindings/display/bridge/anx7814.txt > > diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,anx7814.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,anx7814.yaml > new file mode 100644 > index 000000000000..5459336ac2f5 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7814.yaml > @@ -0,0 +1,124 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/bridge/analogix,anx7814.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analogix ANX7814 SlimPort (Full-HD Transmitter) > + > +maintainers: > + - Enric Balletbo i Serra > + > +properties: > + compatible: > + enum: > + - analogix,anx7808 > + - analogix,anx7812 > + - analogix,anx7814 > + - analogix,anx7818 > + > + reg: > + maxItems: 1 > + description: I2C address of the device. > + > + interrupts: > + maxItems: 1 > + description: Should contain the INTP interrupt. > + > + hpd-gpios: > + maxItems: 1 > + description: Which GPIO to use for hpd. > + > + pd-gpios: > + maxItems: 1 > + description: Which GPIO to use for power down. > + > + reset-gpios: > + maxItems: 1 > + description: Which GPIO to use for reset. > + > + dvdd10-supply: > + maxItems: 1 > + description: Regulator for 1.0V digital core power. > + > + ports: > + type: object > + description: > + A node containing input and output port nodes with endpoint > + definitions as documented in > + Documentation/devicetree/bindings/media/video-interfaces.txt > + Documentation/devicetree/bindings/graph.txt > + > + properties: > + port@0: > + type: object > + description: Video port for HDMI input. > + > + properties: > + reg: > + const: 0 > + > + port@1: > + type: object > + description: > + Video port for SlimPort, DisplayPort, eDP or MyDP output. > + > + properties: > + reg: > + const: 1 > + > + required: > + - port@0 > + - port@1 > + > +required: > + - compatible > + - reg > + - interrupts > + - hpd-gpios > + - pd-gpios > + - reset-gpios > + - ports > + > +additionalProperties: false > + > +examples: > + - | > + #include > + #include > + > + i2c0 { Name this i2c > + #address-cells = <1>; > + #size-cells = <0>; > + > + anx7814: bridge@38 { > + compatible = "analogix,anx7814"; > + reg = <0x38>; > + interrupt-parent = <&gpio0>; > + interrupts = <99 IRQ_TYPE_LEVEL_LOW>; /* INTP */ > + hpd-gpios = <&pio 36 GPIO_ACTIVE_HIGH>; > + pd-gpios = <&pio 33 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&pio 98 GPIO_ACTIVE_HIGH>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + anx7814_in: endpoint { > + remote-endpoint = <&hdmi0_out>; > + }; > + }; > + > + port@1 { > + reg = <1>; > + anx7814_out: endpoint { > + remote-endpoint = <&edp_out>; > + }; > + }; > + }; > + }; > + }; > + > +... > diff --git a/Documentation/devicetree/bindings/display/bridge/anx7814.txt b/Documentation/devicetree/bindings/display/bridge/anx7814.txt > deleted file mode 100644 > index 17258747fff6..000000000000 > --- a/Documentation/devicetree/bindings/display/bridge/anx7814.txt > +++ /dev/null > @@ -1,42 +0,0 @@ > -Analogix ANX7814 SlimPort (Full-HD Transmitter) > ------------------------------------------------ > - > -The ANX7814 is an ultra-low power Full-HD (1080p60) SlimPort transmitter > -designed for portable devices. > - > -Required properties: > - > - - compatible : Must be one of: > - "analogix,anx7808" > - "analogix,anx7812" > - "analogix,anx7814" > - "analogix,anx7818" > - - reg : I2C address of the device > - - interrupts : Should contain the INTP interrupt > - - hpd-gpios : Which GPIO to use for hpd > - - pd-gpios : Which GPIO to use for power down > - - reset-gpios : Which GPIO to use for reset > - > -Optional properties: > - > - - dvdd10-supply : Regulator for 1.0V digital core power. > - - Video port for HDMI input, using the DT bindings defined in [1]. > - > -[1]: Documentation/devicetree/bindings/media/video-interfaces.txt > - > -Example: > - > - anx7814: anx7814@38 { > - compatible = "analogix,anx7814"; > - reg = <0x38>; > - interrupt-parent = <&gpio0>; > - interrupts = <99 IRQ_TYPE_LEVEL_LOW>; /* INTP */ > - hpd-gpios = <&pio 36 GPIO_ACTIVE_HIGH>; > - pd-gpios = <&pio 33 GPIO_ACTIVE_HIGH>; > - reset-gpios = <&pio 98 GPIO_ACTIVE_HIGH>; > - port { > - anx7814_in: endpoint { > - remote-endpoint = <&hdmi0_out>; > - }; > - }; > - }; > -- > 2.18.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel