On Wed, Nov 21, 2012 at 05:09:31PM +0100, Daniel Vetter wrote: > On Wed, Nov 21, 2012 at 5:08 PM, Daniel Vetter <daniel at ffwll.ch> wrote: > > On Wed, Nov 21, 2012 at 4:47 PM, Thierry Reding > > <thierry.reding at avionic-design.de> 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. > > 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 =), but I'll see what I can do. Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20121122/cd88111f/attachment.pgp>
