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 09:05:28 +0100 Message-ID: <20121122080528.GA32579@avionic-0098.adnet.avionic-design.de> References: <1353512389-16261-1-git-send-email-przanoni@gmail.com> <20121121154742.GA1026@avionic-0098.adnet.avionic-design.de> <20121122064851.GB22143@avionic-0098.adnet.avionic-design.de> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0660165010==" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Paulo Zanoni List-Id: dri-devel@lists.freedesktop.org --===============0660165010== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="mYCpIKhGyMATD0i+" Content-Disposition: inline --mYCpIKhGyMATD0i+ Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 22, 2012 at 09:00:24AM +0100, Rafa=C5=82 Mi=C5=82ecki wrote: > 2012/11/22 Thierry Reding : > > On Wed, Nov 21, 2012 at 05:08:12PM +0100, 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 c= an > >> > 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'll have to rely on somebody else to do the testing since I don't have > > an HDMI capable hardware except Tegra to run this on. But yes, I had > > planned to convert Tegra and at least one other driver for reference. > > But I guess while at it I could just as well convert all of them. >=20 > I'll take a look at radeon (I'll try to convert it to the new > functions) over weekend. Okay, great! I think for radeon things should be the easiest since it doesn't currently fill in anything but the colorspace field. Judging by the bug report that Paulo mentioned this will probably not be enough for some hardware, but should be enough according to the specification. Having more data in the AVI infoframe shouldn't hurt, though. Thierry --mYCpIKhGyMATD0i+ Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) iQIcBAEBAgAGBQJQrdzIAAoJEN0jrNd/PrOh9j8P/R6Re91e1esm5CQI3RzuDACa 9Tsqm0DNsN0VfG5uvv1fhCQ/w2kshj+2b6v9S8x5HxJtvc8ZH66+tIOvsvPwk75D nUbMd67eVZp5sMqnthK7PF43mDL/UY++Slz2A3PMO1PUJfjhAJRv6tp0M48ZHy7U 59ozxS8p3XUhHa9qmB1G1pwRh1Nyzvm9DdcOYFi5cFn3u+i1MMi9bbtNNbySbhDE 7TEvVJrv5cm3sAtEFSzKm0eAsUp5sxPMD9oznIs6d5TeNhPRX83sgMiLKjV4K8X4 whNJHQgPOB/kV8AVk9zMhxGN05S6UVkaf2bWJuFs4ZW315OCp4AgB1EIgsxh0oc5 O5bZBfD1Hf4BhaMHECwumPBi756G0XnwgexfCbRn69KGyHznbqDpy/ZwA7+ZOh6y 6foaW1TqLxxxC4pwdNkv1jd5PDkpmm2d93rZH0yU2rFL1QgOXH2p1l6aUyb/+UgJ fs8yfLDj+e0nmwPYEpfng7PxXlkMM+4Yb3wgyuMQs18F51fP4A8yxNO5LDcmnnTd 0C4NQIaFzwkKaUhpBLwhJBmjk/oLL9jZldF1iEr1LAzmWYiwn+P/XIq2vdGctEtC ewM8sr8ko4Nd5cPebeoNgcekKkN8m1mDtB4tdmulcSB/IhbZsPhyEcfX2VDf5LQ/ VhzCDExU4HhDCGnTcayD =qv60 -----END PGP SIGNATURE----- --mYCpIKhGyMATD0i+-- --===============0660165010== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============0660165010==--