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=-3.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED 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 CE7C1C5517A for ; Tue, 10 Nov 2020 05:53:31 +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 3663820781 for ; Tue, 10 Nov 2020 05:53:31 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3663820781 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 EA0938982C; Tue, 10 Nov 2020 05:53:29 +0000 (UTC) Received: from asavdk3.altibox.net (asavdk3.altibox.net [109.247.116.14]) by gabe.freedesktop.org (Postfix) with ESMTPS id 12E4C8982C for ; Tue, 10 Nov 2020 05:53:29 +0000 (UTC) Received: from ravnborg.org (unknown [188.228.123.71]) (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 077672003B; Tue, 10 Nov 2020 06:53:25 +0100 (CET) Date: Tue, 10 Nov 2020 06:53:24 +0100 From: Sam Ravnborg To: Liu Ying Subject: Re: [PATCH] dt-bindings: display: panel-simple: Allow optional 'ports' property Message-ID: <20201110055324.GA1993169@ravnborg.org> References: <1604477017-17642-1-git-send-email-victor.liu@nxp.com> <20201104104735.GA1581328@ravnborg.org> <9a187307aa4f39c44a117f2b99a7f7fd4817e129.camel@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <9a187307aa4f39c44a117f2b99a7f7fd4817e129.camel@nxp.com> X-CMAE-Score: 0 X-CMAE-Analysis: v=2.3 cv=VbvZwmh9 c=1 sm=1 tr=0 a=S6zTFyMACwkrwXSdXUNehg==:117 a=S6zTFyMACwkrwXSdXUNehg==:17 a=kj9zAlcOel0A:10 a=Todku-sVb2SKgMi1d8kA: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: devicetree@vger.kernel.org, David Airlie , dri-devel@lists.freedesktop.org, Rob Herring , Thierry Reding Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hi Liu Ying, On Tue, Nov 10, 2020 at 10:37:27AM +0800, Liu Ying wrote: > Hi Sam, > > On Wed, 2020-11-04 at 11:47 +0100, Sam Ravnborg wrote: > > Hi Liu Ying > > > > On Wed, Nov 04, 2020 at 04:03:37PM +0800, Liu Ying wrote: > > > Some simple panels have dual LVDS interfaces which receive even and > > > odd > > > pixels respectively, like 'nlt,nl192108ac18-02d' and > > > 'koe,tx26d202vm0bwa'. > > > So, let's allow optional 'ports' property so that pixel order can > > > be got > > > via drm_of_lvds_get_dual_link_pixel_order() if it's child nodes > > > 'port@0' > > > and 'port@1' contain 'dual-lvds-even-pixels' and 'dual-lvds-odd- > > > pixels' > > > properties respectively. > > > > A panel with dual LVDS interfaces is no longer in the "simple" > > category. > > The panel-simple binding shall be limited to the simple pnales only. > > > > This is also why we have for example panel-simple-dsi binding. > > > > Please consider either a binding dedicated for the dual port displays > > or > > Thanks for your review. > > Does a new 'panel-simple-lvds-dual-ports.yaml' binding look ok? > I assume it would inherit all properties of panel-simple.yaml and all > existing compatibles of LVDS panels with dual ports in > panel-simple.yaml can be moved to it. Yes, that would be fine. If possible we shall have similar panels described by the same binding. Try to send out the binding for review early before starting to move too much around - so we can get an early look at it at see if this is indeed the right direction to go. Sam _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel