From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757353AbdCXPVk (ORCPT ); Fri, 24 Mar 2017 11:21:40 -0400 Received: from fllnx210.ext.ti.com ([198.47.19.17]:50145 "EHLO fllnx210.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756965AbdCXPVc (ORCPT ); Fri, 24 Mar 2017 11:21:32 -0400 Subject: Re: [PATCHv2 10/10] ARM: dts: n950: add display support To: Tony Lindgren References: <20170304235021.27055-1-sre@kernel.org> <20170305004309.28259-1-sre@kernel.org> <20170305004309.28259-10-sre@kernel.org> <20170324142939.GY10760@atomide.com> <20170324151202.GZ10760@atomide.com> CC: Sebastian Reichel , Aaro Koskinen , Laurent Pinchart , David Airlie , , , From: Tomi Valkeinen Message-ID: <38bb9030-5244-1751-1606-4d6598139bf0@ti.com> Date: Fri, 24 Mar 2017 17:20:31 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <20170324151202.GZ10760@atomide.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="LFmXUCSMXLEdjUMosFeE3VMkDqeaiDmRX" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --LFmXUCSMXLEdjUMosFeE3VMkDqeaiDmRX Content-Type: multipart/mixed; boundary="vhrkcaivDiHuohpROda6RHUUen0gcLQsq"; protected-headers="v1" From: Tomi Valkeinen To: Tony Lindgren Cc: Sebastian Reichel , Aaro Koskinen , Laurent Pinchart , David Airlie , linux-omap@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Message-ID: <38bb9030-5244-1751-1606-4d6598139bf0@ti.com> Subject: Re: [PATCHv2 10/10] ARM: dts: n950: add display support References: <20170304235021.27055-1-sre@kernel.org> <20170305004309.28259-1-sre@kernel.org> <20170305004309.28259-10-sre@kernel.org> <20170324142939.GY10760@atomide.com> <20170324151202.GZ10760@atomide.com> In-Reply-To: <20170324151202.GZ10760@atomide.com> --vhrkcaivDiHuohpROda6RHUUen0gcLQsq Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 24/03/17 17:12, Tony Lindgren wrote: > * Tomi Valkeinen [170324 08:01]: >> On 24/03/17 16:29, Tony Lindgren wrote: >>> * Sebastian Reichel [170304 16:45]: >>>> Add basic panel support for the Nokia N950. It must be tweaked a >>>> little bit later, since the panel was built into the device >>>> upside-down. Also the first 5 and the last 5 pixels are covered >>>> by plastic. >>> >>> This one seems safe to apply separately to shrink the pending >>> patches a bit. Please confirm if that's the case and I'll apply >>> this one. >> >> No, I think there are too many open questions to apply this one. And w= e >> need to move to common DRM drivers at some point (hopefully soon) so t= he >> bindings must be aligned with those if and when possible. >=20 > Hmm so what are the open questions and do you have in mind for > the panel bindings? Well, until we have driver that supports the panel without problems, we can't be sure if we have everything in the binding. At least the TE was missing or broken. Normal videotimings don't really make sense for DSI command mode panels, so I'm not overly enthusiastic in having them in the bindings. Possibly that's the easy way to go, but I'm not sure yet. There are the "TODO" parts in the bindings, that's a clear sign of open questions. I haven't looked at how the other DRM drivers handle DSI command mode panels. That's something to study, before adding anything new to the omap specific bindings. > Naturally we still need to support the current bindings too. Do the current bindings work? I guess they did work on omapfb... Do these new bindings work with omapfb? Generally speaking, I've been bitten badly enough with DT bindings that weren't up to the task after all, and then changing the later was a major pain (or working around it). So I'd rather not merge anything to DT if there are any unclarities, and a driver not working which uses those bindings is an unclarity. Especially if the only reason to merge is to get a single patch out from a bigger patch series, without actually enabling anything. Tomi --vhrkcaivDiHuohpROda6RHUUen0gcLQsq-- --LFmXUCSMXLEdjUMosFeE3VMkDqeaiDmRX Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJY1TlAAAoJEPo9qoy8lh71jdcP/jpxtaNvv2tOfWLv2ZTbZcr1 iGjFJDO3eiumVSEHnCSQ/G8AsC1R1xi/LGwgkZFmPhw5CYkOwlRjaRB2LSrq/6hm 0GRdJXgP7oRbRhia9g1q/jGa2bu2DTnhZYGHIYdS172pV1ZEZMt6NE+0+aF0L1KQ PCE1//Elg1u3xGCccNsUXm2+1Qdjj6MKFYhk131IrMiu1RyjOO6h5G23v9kedt85 6J0pirGbSza6poQcmoyczOGnlh97LlTevHcbqPjjf2lxI1qrYzFSD4zYb+xfWMsw KWxvpOcRktBQAAvM+mfQyTB5RGmRvFKaAJ7weZj/9Zn9tR10lqq6qF1n76Mn7oug IfFvkT+N/AM70aQ8sL/uE4JvVHs7IJiTSRPHCxARK7ssxTT+6lDX3VukvDUKqz5g MavvCHjPEDpHJUMcHp6seTQs9O5QfvQTjgFgQ3zbeJGtG7j8k9kQeGaFy5GfucaZ poXpoKz0LfuthHxCn3IC/6qYv7zsJkn5dbd9sgRBwTxhK8kxkrdG2+Gx763emFnn iuwAFxocAi5jOwbRM8UkCdb+9v8XVjTgVCCxSAX+0YZ1zcBxG9NsJvIzdqYu7KXD +gDxAUB/HaaXI6SSgFoRCMomVgs50/879252gT89d4qmA3QDzSudxlCwUmzufj/G GsylcSbNV3eZk++KFumq =3azz -----END PGP SIGNATURE----- --LFmXUCSMXLEdjUMosFeE3VMkDqeaiDmRX--