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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id AED16C43334 for ; Tue, 28 Jun 2022 15:01:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1347311AbiF1PBe (ORCPT ); Tue, 28 Jun 2022 11:01:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37926 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346312AbiF1PBd (ORCPT ); Tue, 28 Jun 2022 11:01:33 -0400 Received: from gloria.sntech.de (gloria.sntech.de [185.11.138.130]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 09CBE2F64E; Tue, 28 Jun 2022 08:01:30 -0700 (PDT) Received: from ip5b412258.dynamic.kabel-deutschland.de ([91.65.34.88] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1o6Ci1-0005Gn-Sy; Tue, 28 Jun 2022 17:00:57 +0200 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: David Airlie , Daniel Vetter , Rob Herring , Krzysztof Kozlowski , Thierry Reding , Sam Ravnborg , Eugeniy Paltsev , Vinod Koul , Liam Girdwood , Mark Brown , Serge Semin , Daniel Lezcano , Palmer Dabbelt , Palmer Dabbelt , linux-riscv@lists.infradead.org Cc: Thomas Gleixner , Paul Walmsley , Albert Ou , Conor Dooley , Masahiro Yamada , Damien Le Moal , Geert Uytterhoeven , Niklas Cassel , Dillon Min , Heng Sia , Jose Abreu , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org, alsa-devel@alsa-project.org, linux-spi@vger.kernel.org, linux-riscv@lists.infradead.org, Conor Dooley Subject: Re: [PATCH v2 01/16] dt-bindings: display: convert ilitek,ili9341.txt to dt-schema Date: Tue, 28 Jun 2022 17:00:56 +0200 Message-ID: <3361801.QJadu78ljV@diego> In-Reply-To: <20220627194003.2395484-2-mail@conchuod.ie> References: <20220627194003.2395484-1-mail@conchuod.ie> <20220627194003.2395484-2-mail@conchuod.ie> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Precedence: bulk List-ID: X-Mailing-List: linux-spi@vger.kernel.org Hi Conor, Am Montag, 27. Juni 2022, 21:39:49 CEST schrieb Conor Dooley: > From: Conor Dooley > > A dt-schema binding for the Ilitek ili9341 was created as > panel/ilitek,ili9341.yaml but the txt binding was ignored in the > process. Move the remaining items in the txt binding to the yaml one & > delete it. > > The example in the txt binding has a spi-max-frequency which disagrees > with the yaml replacement (and its own documentation) so change that to > conform with the binding. There are no users in tree of the Adafruit > yx240qv29 to check against. > > Link: https://cdn-learn.adafruit.com/assets/assets/000/046/879/original/SPEC-YX240QV29-T_Rev.A__1_.pdf > Signed-off-by: Conor Dooley in your v1 you already got a [0] Reviewed-by: Rob Herring for this patch Looking at the diffstat, which didn't change, you should include such tags in followup revisions, to prevent reviewers from double efforts. Heiko [0] https://lore.kernel.org/r/20220627232054.GA3155668-robh@kernel.org > --- > .../bindings/display/ilitek,ili9341.txt | 27 ----------- > .../display/panel/ilitek,ili9341.yaml | 48 +++++++++++++------ > 2 files changed, 34 insertions(+), 41 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/display/ilitek,ili9341.txt > > diff --git a/Documentation/devicetree/bindings/display/ilitek,ili9341.txt b/Documentation/devicetree/bindings/display/ilitek,ili9341.txt > deleted file mode 100644 > index 169b32e4ee4e..000000000000 > --- a/Documentation/devicetree/bindings/display/ilitek,ili9341.txt > +++ /dev/null > @@ -1,27 +0,0 @@ > -Ilitek ILI9341 display panels > - > -This binding is for display panels using an Ilitek ILI9341 controller in SPI > -mode. > - > -Required properties: > -- compatible: "adafruit,yx240qv29", "ilitek,ili9341" > -- dc-gpios: D/C pin > -- reset-gpios: Reset pin > - > -The node for this driver must be a child node of a SPI controller, hence > -all mandatory properties described in ../spi/spi-bus.txt must be specified. > - > -Optional properties: > -- rotation: panel rotation in degrees counter clockwise (0,90,180,270) > -- backlight: phandle of the backlight device attached to the panel > - > -Example: > - display@0{ > - compatible = "adafruit,yx240qv29", "ilitek,ili9341"; > - reg = <0>; > - spi-max-frequency = <32000000>; > - dc-gpios = <&gpio0 9 GPIO_ACTIVE_HIGH>; > - reset-gpios = <&gpio0 8 GPIO_ACTIVE_HIGH>; > - rotation = <270>; > - backlight = <&backlight>; > - }; > diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml > index 6058948a9764..94ca92878434 100644 > --- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml > +++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml > @@ -23,6 +23,7 @@ properties: > - enum: > # ili9341 240*320 Color on stm32f429-disco board > - st,sf-tc240t-9370-t > + - adafruit,yx240qv29 > - const: ilitek,ili9341 > > reg: true > @@ -47,31 +48,50 @@ properties: > vddi-led-supply: > description: Voltage supply for the LED driver (1.65 .. 3.3 V) > > -additionalProperties: false > +unevaluatedProperties: false > > required: > - compatible > - reg > - dc-gpios > - - port > + > +if: > + properties: > + compatible: > + contains: > + enum: > + - st,sf-tc240t-9370-t > +then: > + required: > + - port > > examples: > - |+ > + #include > spi { > #address-cells = <1>; > #size-cells = <0>; > panel: display@0 { > - compatible = "st,sf-tc240t-9370-t", > - "ilitek,ili9341"; > - reg = <0>; > - spi-3wire; > - spi-max-frequency = <10000000>; > - dc-gpios = <&gpiod 13 0>; > - port { > - panel_in: endpoint { > - remote-endpoint = <&display_out>; > - }; > - }; > - }; > + compatible = "st,sf-tc240t-9370-t", > + "ilitek,ili9341"; > + reg = <0>; > + spi-3wire; > + spi-max-frequency = <10000000>; > + dc-gpios = <&gpiod 13 0>; > + port { > + panel_in: endpoint { > + remote-endpoint = <&display_out>; > + }; > + }; > + }; > + display@1{ > + compatible = "adafruit,yx240qv29", "ilitek,ili9341"; > + reg = <1>; > + spi-max-frequency = <10000000>; > + dc-gpios = <&gpio0 9 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&gpio0 8 GPIO_ACTIVE_HIGH>; > + rotation = <270>; > + backlight = <&backlight>; > }; > + }; > ... >