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=-6.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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 1F59DC38A29 for ; Wed, 15 Apr 2020 00:54:45 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E906B20737 for ; Wed, 15 Apr 2020 00:54:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="eVHUoiht" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E906B20737 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 6DDC06E7E5; Wed, 15 Apr 2020 00:54:44 +0000 (UTC) Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by gabe.freedesktop.org (Postfix) with ESMTPS id A38BD6E7E5 for ; Wed, 15 Apr 2020 00:54:42 +0000 (UTC) Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 8A422521; Wed, 15 Apr 2020 02:54:39 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1586912079; bh=t/s3cFaprX7TQgkKtf3BVyw0MwfDpYj0dFVoO/TDuXw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=eVHUoihtAmBgramO/GhT99Seqkul5d/u62tGrXDHYQvQSqbUU7Sw+ZUl0suUDOBjH Ib2qE7AMpG3vFcOfNBOrzXsQYevvUVThKSZpee18Yn8tRP5LgXr/QGWHYm+YK8gz1t UQJHwhWVButreXqgrcjjJoQkFuUrkhU+iHWlrva4= Date: Wed, 15 Apr 2020 03:54:27 +0300 From: Laurent Pinchart To: Rob Herring Subject: Re: [PATCH 2/4] dt-bindings: display: bridge: Convert simple-bridge bindings to YAML Message-ID: <20200415005427.GY19819@pendragon.ideasonboard.com> References: <20200405232318.26833-1-laurent.pinchart+renesas@ideasonboard.com> <20200405232318.26833-3-laurent.pinchart+renesas@ideasonboard.com> <20200414220000.GA31265@bogus> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200414220000.GA31265@bogus> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Laurent Pinchart , dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, Jacopo Mondi , Maxime Ripard Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hi Rob, On Tue, Apr 14, 2020 at 05:00:00PM -0500, Rob Herring wrote: > On Mon, Apr 06, 2020 at 02:23:16AM +0300, Laurent Pinchart wrote: > > The simple-bridge driver supports multiple simple or dumb bridges, > > covered by different compatible strings but otherwise identical DT > > bindings. Some of those bridges have undocumented bindings, while others > > are documented in text form in separate files. Group them all in a > > single binding and convert it to YAML. > > > > Signed-off-by: Laurent Pinchart > > --- > > .../bindings/display/bridge/adi,adv7123.txt | 50 ---------- > > .../bindings/display/bridge/dumb-vga-dac.txt | 50 ---------- > > .../display/bridge/simple-bridge.yaml | 99 +++++++++++++++++++ > > .../bindings/display/bridge/ti,ths813x.txt | 51 ---------- > > 4 files changed, 99 insertions(+), 151 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/display/bridge/adi,adv7123.txt > > delete mode 100644 Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt > > create mode 100644 Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml > > delete mode 100644 Documentation/devicetree/bindings/display/bridge/ti,ths813x.txt > > > > diff --git a/Documentation/devicetree/bindings/display/bridge/adi,adv7123.txt b/Documentation/devicetree/bindings/display/bridge/adi,adv7123.txt > > deleted file mode 100644 > > index a6b2b2b8f3d9..000000000000 > > --- a/Documentation/devicetree/bindings/display/bridge/adi,adv7123.txt > > +++ /dev/null > > @@ -1,50 +0,0 @@ > > -Analog Device ADV7123 Video DAC > > -------------------------------- > > - > > -The ADV7123 is a digital-to-analog converter that outputs VGA signals from a > > -parallel video input. > > - > > -Required properties: > > - > > -- compatible: Should be "adi,adv7123" > > - > > -Optional properties: > > - > > -- psave-gpios: Power save control GPIO > > Not documented in the new schema. Did you intend to change to > 'enable-gpios'? Oops, I forgot to mention that, my bad. The psave GPIO isn't supported by the driver, and isn't used in any upstream DT files, so I thought it would be safe to drop it, and add support it later (possibly through the enable GPIO) if/when needed. I'll mention it in the commit message. -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel