From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH] drm/i915: set the AVI VIC of the HDMI mode Date: Thu, 22 Nov 2012 07:50:55 +0100 Message-ID: <20121122065055.GC22143@avionic-0098.adnet.avionic-design.de> References: <1353512389-16261-1-git-send-email-przanoni@gmail.com> <20121121154742.GA1026@avionic-0098.adnet.avionic-design.de> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1776337074==" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Paulo Zanoni List-Id: dri-devel@lists.freedesktop.org --===============1776337074== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="JgQwtEuHJzHdouWu" Content-Disposition: inline --JgQwtEuHJzHdouWu Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Nov 21, 2012 at 05:09:31PM +0100, Daniel Vetter wrote: > On Wed, Nov 21, 2012 at 5:08 PM, Daniel Vetter wrote: > > On Wed, Nov 21, 2012 at 4:47 PM, Thierry Reding > > wrote: > >> Oh great, so I copied that table for nothing. Thanks for Cc'ing, I can > >> reuse that in the HDMI infoframe series. > > > > Wrt the infoframe series, I think it'd be awesome if you could convert > > i915 and radeon (iirc the existing drivers with the "best" avi > > infoframe support) over to the new code. This gives some nice > > validation, both by testing on actual hw and that the interface is > > sane, since it'll be used by 2-3 different drivers then. > > > > I think the best way is to pick the infoframe implementation you best > > like from one of these, move it into the helper, then improve it until > > you're happy. And then convert over 1-2 other drivers. At least that's > > been my approach for the recent dp helper refactoring. >=20 > And if you could use that opportunity to integrate the kerneldoc for > edid/eld and the new infoframe code into the drm docbook, that would > be rather awesome ;-) I suck at documentation =3D), but I'll see what I can do. Thierry --JgQwtEuHJzHdouWu Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) iQIcBAEBAgAGBQJQrctPAAoJEN0jrNd/PrOhVuEP/j7XigBQ5Cc1jG7ijfJIz327 S/QmDWt7ANQuf7c3uGaoBrF9oxBbjotSzC7//FFjV4c1GAx6aKTq4BYpEvSIlyoc GkKEy4JQ5L3cVGp5HUafZlp6KinmB115XROxPZsrg0ONznaTlsZ1SyeFTwGUTKjw b9edl3wV9Ix0rbuHYNB0LHMT6bEIn2WVeMZvg97e3zT2asKUpvdqxtnNufYSzmrg yfZ0XAW+qv6dqWWYUSnm0JlSMoPjtnSr6zaqSQMhVUCm4w3NUuKIyxOaVdXgAZOR gLQiADJCqd2TWKY18Rq4jDLwguOPOgXM/xOPmAJM+XaQnt2dfm4TpDTJzbj5v1sk 5HC7y8vmyRI6hTomnDMyGMWeLjZeVykYQEeU8B7JF4CWxjUeb9siCUHEKTsqzXR/ Nf7cKQEPiejsw59DSTDQsuG34BQMzF63bqyTrx6IQDGEwkrxIuLRdIU0kowMN57b fK4rvhowY7kYPDWCFf9QBZLOI3PUYL20WVktyfNsB7rOxb34I39yD7ttRIIQlpF/ l5Fj2SCliBEqK5pJNI+f8M5v9GjbjSVp1w+p6Z8aMU/qzNIxy49wiOMkBo7cmhmQ RwUTJTnaOBJdcO4E9pzqZlt/kMgQFid92PXsQnSdrg61+mF8YwVbnuussTmGsZth 4KkJ9Hkigcgpgm+7Zxi+ =S2Lm -----END PGP SIGNATURE----- --JgQwtEuHJzHdouWu-- --===============1776337074== 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 --===============1776337074==--