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=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 8DB0DC18E5B for ; Mon, 16 Mar 2020 08:53:19 +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 6DBF520719 for ; Mon, 16 Mar 2020 08:53:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6DBF520719 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ravnborg.org 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 DD354897F9; Mon, 16 Mar 2020 08:53:18 +0000 (UTC) Received: from asavdk3.altibox.net (asavdk3.altibox.net [109.247.116.14]) by gabe.freedesktop.org (Postfix) with ESMTPS id 821D2897F9 for ; Mon, 16 Mar 2020 08:53:17 +0000 (UTC) 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 asavdk3.altibox.net (Postfix) with ESMTPS id A42282002E; Mon, 16 Mar 2020 09:53:12 +0100 (CET) Date: Mon, 16 Mar 2020 09:53:11 +0100 From: Sam Ravnborg To: Tomi Valkeinen Subject: Re: [PATCH v1 35/36] dt-bindings: display: convert lgphilips,lb035q02 to DT Schema Message-ID: <20200316085311.GA19201@ravnborg.org> References: <20200315134416.16527-1-sam@ravnborg.org> <20200315134416.16527-36-sam@ravnborg.org> <20200316082607.GA18510@ravnborg.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) X-CMAE-Score: 0 X-CMAE-Analysis: v=2.3 cv=eMA9ckh1 c=1 sm=1 tr=0 a=UWs3HLbX/2nnQ3s7vZ42gw==:117 a=UWs3HLbX/2nnQ3s7vZ42gw==:17 a=jpOVt7BSZ2e4Z31A5e1TngXxSK0=:19 a=kj9zAlcOel0A:10 a=nj24DLa8QVhGY9zhrUcA:9 a=CjuIK1q_8ugA:10 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: Maxime Ripard , Nikolaus Schaller , Jonathan Bakker , Sandeep Panda , dri-devel@lists.freedesktop.org, Paul Cercueil , Andrzej Hajda , Thierry Reding , Laurent Pinchart , Chris Zhong , Marco Franchi , Stefan Mavrodiev , Jerry Han , Hoegeun Kwon , Tony Lindgren , Nickey Yang , Jagan Teki , Robert Chiras , Vinay Simha BN , Marek Belisko , Heiko Schocher , Brian Masney , devicetree@vger.kernel.org, Guido Gunther , Mark Brown , Alexandre Courbot , Werner Johansson , Purism Kernel Team , Peter Ujfalusi , Lin Huang , Douglas Anderson , linux-spi@vger.kernel.org, Peter Rosin Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hi Tomi. On Mon, Mar 16, 2020 at 10:42:45AM +0200, Tomi Valkeinen wrote: > On 16/03/2020 10:26, Sam Ravnborg wrote: > > > > Isn't this also compatible with panel-simple bindings? 'label' is the only > > > one not in panel-simple, but that's optional and has never been used by the > > > panel driver. > > The panel is a SPI slave - which is not too obvious from the old > > binding. > > > > The new DT Schema includes spi/spi-slave.yaml to give the binding > > proper access to the spi slave properties. > > > > That would not be possible with panel-simple binding as no further > > properties are allowed with the panel-simple binding. > > > > I hope this explains why there is a dedicated binding for this panel. > > Hmm, but how is this different than, say, DSI panels? There are DSI panels > in panel-simple bindings, and those might require DSI bus parameters ('reg' > in the minimum). We have panel-simple-dsi for DSI simple based panels. This binding includes the reg property. If we have included DSI panels in panel-simple.yaml, and we likely have by accident, then they should be moved to panel-simple-dsi.yaml. If they requires anything else then they shall have their own binding. panel-simple.yaml and panel-simple.dsi.yaml are on purpose only for the simple panels and they have: "additionalProperties: false" to avoid that a lot of extra sneaks in. I actually considered shortly a panel-simple-spi.yaml, but the few panels I looked at had different names for the power-supply so that did not fly. I did not check them all - we have today (with this patch-set) 9 bindings that references spi-slave.yaml. Sam _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel