From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [RESEND PATCH v3 06/11] drm: add DT bindings documentation for atmel-hlcdc-dc driver Date: Mon, 21 Jul 2014 15:22:05 +0200 Message-ID: <2695740.osZpCPIB8S@avalon> References: <1404751384-5077-1-git-send-email-boris.brezillon@free-electrons.com> <20140721143428.078b68a4@bbrezillon> <20140721125522.GE15238@ulmo> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0480747321==" Return-path: In-Reply-To: <20140721125522.GE15238@ulmo> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Thierry Reding Cc: Mark Rutland , devicetree@vger.kernel.org, Nicolas Ferre , dri-devel@lists.freedesktop.org, Alexandre Belloni , Bo Shen , Lee Jones , Jean-Jacques Hiblot , Samuel Ortiz , Tim Niemeyer , Jean-Christophe Plagniol-Villard , linux-pwm@vger.kernel.org, Pawel Moll , Ian Campbell , Rob Herring , Andrew Victor , linux-arm-kernel@lists.infradead.org, Thomas Petazzoni , Kumar Gala List-Id: devicetree@vger.kernel.org --===============0480747321== Content-Type: multipart/signed; boundary="nextPart1692194.q5RyYzooKS"; micalg="pgp-sha1"; protocol="application/pgp-signature" --nextPart1692194.q5RyYzooKS Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="us-ascii" Hi Thierry, On Monday 21 July 2014 14:55:23 Thierry Reding wrote: > On Mon, Jul 21, 2014 at 02:34:28PM +0200, Boris BREZILLON wrote: > > On Mon, 21 Jul 2014 14:16:42 +0200 Laurent Pinchart wrote: > >> On Monday 21 July 2014 14:12:47 Thierry Reding wrote: > >>> On Mon, Jul 21, 2014 at 11:57:37AM +0200, Boris BREZILLON wrote: [snip] > >>>> The new drm_display_info structure should look like this [2] (ex= cept > >>>> that color_formats and bpc have not be removed yet), and [1] is = just > >>>> here to show how the video_bus_format enum would look like. > >>>>=20 > >>>> [1] http://code.bulix.org/rfd0yx-86557 > >>>> [2] http://code.bulix.org/7n03b4-86556 > >>>=20 > >>> Quoting from your paste: > >>> =09+ const enum video_bus_format *bus_formats; > >>> =09+ int nbus_formats; > >>>=20 > >>> Do we really need more than one? > >>=20 > >> We do if we want to replace the color_formats and bpc fields. > >=20 > > Yes, that's what I was about to answer :-). >=20 > Maybe we don't need to replace color_formats and bpc field immediatel= y. > That could be done in a follow-up patch. We don't need to replace them right now, but we should at least agree o= n how=20 to replace them. Introducing a new field that would need to be replaced= in the=20 near future when removing color_formats and bpc would be a waste of tim= e. =2D-=20 Regards, Laurent Pinchart --nextPart1692194.q5RyYzooKS Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAABAgAGBQJTzRP9AAoJEIkPb2GL7hl1CRwIAKsndmFK9LfGyqybt0xELJL3 0hFgaQ217OVUnQAZIdRsV9ggUykJb8QDkkQ8PB9Wj3ozpTY2GawkPTxTkeEwnhgx mTR06286JIpstXd3sXHnHsnxaDZrTqCKShTmp2o3Tjm+5qpuAuS7bghPu5SVdwJG NLjnx/77UwBvPfwQ1F6cZne1Qb5GgbckBqYdXeJDcEOHGxH0iJJd/M34LwTjL/AG DXONP/ZINX7S/yRLX71kAG/btPNanEc8to70gLRdpkke+M55bsuU+AhDa28uhkpu scW+SWJ5ckflZVLMZZ/M6Il+Ub5w39kyx8v//qit4DPeyjyx0FHk/CuuDdfNFWo= =BXe8 -----END PGP SIGNATURE----- --nextPart1692194.q5RyYzooKS-- --===============0480747321== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============0480747321==--